Merge pull request #810 from CertainLach/fix/syslog-off-by-one

fix: off-by-one in syslog.rs

Author: squell

src/log/syslog.rs

@@ -55,12 +55,9 @@ impl Write for SysLogWriter {
                     mid -= 1;
                 }
 
-                // index of last whitespace before byte cutoff
-                let ascii_utf8_len = 1;
                 mid = message[..mid]
                     .rfind(|c: char| c.is_ascii_whitespace())
-                    .unwrap_or(mid)
-                    + ascii_utf8_len;
+                    .unwrap_or(mid);
 
                 let left = &message[..mid];
                 let right = &message[mid..];
@@ -132,4 +129,15 @@ mod tests {
 
         logger.log(&record);
     }
+
+    #[test]
+    fn can_truncate_syslog_with_no_spaces() {
+        let logger = Syslog;
+        let record = log::Record::builder()
+            .args(format_args!("iwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercasesiwillhandlecornercases"))
+            .level(log::Level::Info)
+            .build();
+
+        logger.log(&record);
+    }
 }