Document cargo install in README.md

Encoding errors are replaced with � when sent to the ntfy server.

Cargo.lock

@@ -592,7 +592,7 @@ dependencies = [
 
 [[package]]
 name = "ntfy-run"
-version = "0.1.0"
+version = "0.1.2"
 dependencies = [
  "clap",
  "itertools",

Cargo.toml

@@ -1,7 +1,19 @@
 [package]
 name = "ntfy-run"
-version = "0.1.0"
+version = "0.1.2"
 edition = "2021"
+authors = ["Quantum <me@quantum5.ca>"]
+description = "ntfy-run is a tool to run a command, capture its output, and send it to a ntfy server."
+readme = "README.md"
+homepage = "https://github.com/quantum5/ntfy-run"
+repository = "https://github.com/quantum5/ntfy-run"
+license = "GPL-3.0-or-later"
+keywords = ["ntfy", "cron", "notifications", "utility"]
+categories = ["command-line-interface"]
+
+exclude = [
+    ".github/*"
+]
 
 [dependencies]
 clap = { version = "4.5.20", features = ["derive", "env"] }

README.md

@@ -16,7 +16,13 @@ sudo wget https://github.com/quantum5/ntfy-run/releases/latest/download/ntfy-run
 sudo chmod a+x /usr/local/bin/ntfy-run
 ```
 
-Alternatively, build it yourself:
+You can also install the latest release with `cargo`:
+
+```bash
+cargo install ntfy-run
+```
+
+Finally, you can build the latest Git version yourself:
 
 ```bash
 git clone https://github.com/quantum5/ntfy-run.git

src/main.rs

@@ -1,12 +1,13 @@
-use crate::runner::CaptureError;
+use crate::runner::{CaptureError, CapturedOutput};
 use clap::Parser;
-use runner::CapturedOutput;
+use std::process::exit;
 
 mod quote;
 mod runner;
+mod tap_stream;
 
 #[derive(Parser)]
-/// Tool to run a command, capture its output, and send it to ntfy.
+#[command(version, about)]
 struct Cli {
     /// URL of the ntfy server and topic, e.g. https://ntfy.sh/topic
     #[arg(short = 'n', long = "ntfy-url", env = "NTFY_URL", alias = "url")]
@@ -99,7 +100,8 @@ fn format_post_body(output: CapturedOutput) -> String {
     }];
 
     if !output.errors.is_empty() {
-        fragments.push("==================== Errors ====================".to_string());
+        fragments.push("".to_string());
+        fragments.push("========== Errors ==========".to_string());
         for error in &output.errors {
             fragments.push(match error {
                 CaptureError::Spawn(error) => format!("Spawn error: {}", error),
@@ -108,19 +110,18 @@ fn format_post_body(output: CapturedOutput) -> String {
                 CaptureError::Wait(error) => format!("Error while waiting for process: {}", error),
             });
         }
-        fragments.push("\n".to_string());
     }
 
     if !output.stdout.is_empty() {
-        fragments.push("==================== STDOUT ====================".to_string());
+        fragments.push("".to_string());
-        fragments.push(output.stdout);
+        fragments.push("========== STDOUT ==========".to_string());
-        fragments.push("\n".to_string());
+        fragments.push(String::from_utf8_lossy(output.stdout.trim_ascii_end()).into_owned());
     }
 
     if !output.stderr.is_empty() {
-        fragments.push("==================== STDERR ====================".to_string());
+        fragments.push("".to_string());
-        fragments.push(output.stderr);
+        fragments.push("========== STDERR ==========".to_string());
-        fragments.push("\n".to_string());
+        fragments.push(String::from_utf8_lossy(output.stderr.trim_ascii_end()).into_owned());
     }
 
     fragments.join("\n")
@@ -209,7 +210,13 @@ async fn main() {
     };
 
     match request.send().await.and_then(|r| r.error_for_status()) {
-        Ok(_) => (),
+        Ok(_) => exit(match status {
-        Err(error) => eprintln!("Failed to send request to ntfy: {}", error),
+            Some(code) => code.code().unwrap_or(255),
+            None => 255,
+        }),
+        Err(error) => {
+            eprintln!("Failed to send request to ntfy: {}", error);
+            exit(37)
+        }
     }
 }

src/runner.rs

@@ -1,19 +1,18 @@
+use crate::tap_stream::{ReadOrWrite, TapStream};
 use std::process::{ExitStatus, Stdio};
-use tokio::io::{AsyncBufReadExt, BufReader};
+use tokio::{io, process::Command, select};
-use tokio::process::Command;
-use tokio::select;
 
 pub enum CaptureError {
-    Spawn(std::io::Error),
+    Spawn(io::Error),
-    Stdout(std::io::Error),
+    Stdout(io::Error),
-    Stderr(std::io::Error),
+    Stderr(io::Error),
-    Wait(std::io::Error),
+    Wait(io::Error),
 }
 
 pub struct CapturedOutput {
     pub status: Option<ExitStatus>,
-    pub stdout: String,
+    pub stdout: Vec<u8>,
-    pub stderr: String,
+    pub stderr: Vec<u8>,
     pub errors: Vec<CaptureError>,
 }
 
@@ -38,49 +37,50 @@ pub async fn run_forward_and_capture(cmdline: &Vec<String>) -> CapturedOutput {
         Err(error) => {
             return CapturedOutput {
                 status: None,
-                stdout: "".to_string(),
+                stdout: vec![],
-                stderr: "".to_string(),
+                stderr: vec![],
                 errors: vec![CaptureError::Spawn(error)],
             }
         }
     };
 
-    let mut stdout_stream = BufReader::new(child.stdout.take().unwrap()).lines();
+    let mut stdout_tap = TapStream::new(child.stdout.take().unwrap(), io::stdout());
-    let mut stderr_stream = BufReader::new(child.stderr.take().unwrap()).lines();
+    let mut stderr_tap = TapStream::new(child.stderr.take().unwrap(), io::stderr());
 
-    let mut stdout_buffer = Vec::new();
+    let mut stdout = vec![];
-    let mut stderr_buffer = Vec::new();
+    let mut stderr = vec![];
     let mut errors = Vec::new();
 
+    let mut stdout_eof = false;
+    let mut stderr_eof = false;
+    let mut maybe_status: Option<ExitStatus> = None;
+
     let status = loop {
         select! {
-            line = stdout_stream.next_line() => match line {
+            result = stdout_tap.step(), if !stdout_eof => match result {
-                Ok(Some(line)) => {
+                Ok(ReadOrWrite::Read(bytes)) => stdout.extend_from_slice(bytes),
-                    println!("{}", line);
+                Ok(ReadOrWrite::Written) => (),
-                    stdout_buffer.push(line);
+                Ok(ReadOrWrite::EOF) => stdout_eof = true,
-                },
-                Ok(None) => (),
                 Err(error) => errors.push(CaptureError::Stdout(error)),
             },
-            line = stderr_stream.next_line() => match line {
+            result = stderr_tap.step(), if !stderr_eof => match result {
-                Ok(Some(line)) => {
+                Ok(ReadOrWrite::Read(bytes)) => stderr.extend_from_slice(bytes),
-                    eprintln!("{}", line);
+                Ok(ReadOrWrite::Written) => (),
-                    stderr_buffer.push(line);
+                Ok(ReadOrWrite::EOF) => stderr_eof = true,
-                },
-                Ok(None) => (),
                 Err(error) => errors.push(CaptureError::Stderr(error)),
             },
-            status = child.wait() => match status {
+            status = child.wait(), if maybe_status.is_none() => match status {
-                Ok(status) => break status,
+                Ok(status) => maybe_status = Some(status),
                 Err(error) => errors.push(CaptureError::Wait(error)),
-            }
+            },
+            else => break maybe_status.unwrap(),
         }
     };
 
     CapturedOutput {
         status: Some(status),
-        stdout: stdout_buffer.join("\n").to_string(),
+        stdout,
-        stderr: stderr_buffer.join("\n").to_string(),
+        stderr,
         errors,
     }
 }

src/tap_stream.rs

@@ -0,0 +1,52 @@
+use std::io;
+use tokio::io::{AsyncRead, AsyncReadExt, AsyncWrite, AsyncWriteExt};
+
+const BUF_SIZE: usize = 16384;
+
+pub struct TapStream<R: AsyncRead + Unpin, W: AsyncWrite + Unpin> {
+    source: R,
+    target: W,
+    buffer: [u8; BUF_SIZE],
+    buf_start: usize,
+    buf_end: usize,
+}
+
+pub enum ReadOrWrite<'a> {
+    Read(&'a [u8]),
+    Written,
+    EOF,
+}
+
+impl<R: AsyncRead + Unpin, W: AsyncWrite + Unpin> TapStream<R, W> {
+    pub fn new(source: R, target: W) -> TapStream<R, W> {
+        TapStream {
+            source,
+            target,
+            buffer: [0; BUF_SIZE],
+            buf_start: 0,
+            buf_end: 0,
+        }
+    }
+
+    pub async fn step(&mut self) -> io::Result<ReadOrWrite> {
+        if self.buf_start == self.buf_end {
+            let bytes = self.source.read(&mut self.buffer[..]).await?;
+            self.buf_start = 0;
+            self.buf_end = bytes;
+
+            if bytes == 0 {
+                Ok(ReadOrWrite::EOF)
+            } else {
+                Ok(ReadOrWrite::Read(&self.buffer[0..bytes]))
+            }
+        } else {
+            let bytes = self
+                .target
+                .write(&self.buffer[self.buf_start..self.buf_end])
+                .await?;
+
+            self.buf_start += bytes;
+            Ok(ReadOrWrite::Written)
+        }
+    }
+}