Make get_lines_from_stream work for unix newlines

Author: dodomorandi

src/ftp.rs

@@ -445,24 +445,29 @@ impl FtpStream {
     }
 
     /// Consume a stream and return a vector of lines
-    async fn get_lines_from_stream<R>(mut data_stream: R) -> Result<Vec<String>>
+    async fn get_lines_from_stream<R>(data_stream: R) -> Result<Vec<String>>
     where
         R: AsyncBufRead + Unpin,
     {
         let mut lines: Vec<String> = Vec::new();
-        let mut line = String::new();
+
+        let mut lines_stream = data_stream.lines();
         loop {
-            match data_stream.read_to_string(&mut line).await {
-                Ok(0) => break,
-                Ok(_) => lines.extend(
-                    line.split("\r\n")
-                        .map(String::from)
-                        .filter(|s| !s.is_empty()),
-                ),
-                Err(err) => return Err(FtpError::ConnectionError(err)),
-            };
+            let line = lines_stream
+                .next_line()
+                .await
+                .map_err(FtpError::ConnectionError)?;
+
+            match line {
+                Some(line) => {
+                    if line.is_empty() {
+                        continue;
+                    }
+                    lines.push(line);
+                }
+                None => break Ok(lines),
+            }
         }
-        Ok(lines)
     }
 
     /// Execute `LIST` command which returns the detailed file listing in human readable format.