[pbs-devel] [PATCH proxmox-backup] server/worker_task: remove all trailing newlines when reading task status

[Dominik Csapak]

sometimes we may write multiple newlines at the end of tasks,
for example if an error message contains multiple newlines itself

simply ignore all newlines at the end of task logs instead of only one

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
it happened to me during tape cleaning testing, a permission denied
error had two newlines at the end, and i did not find the source of
them... anyway i think we should ignore all of them

 src/server/worker_task.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/server/worker_task.rs b/src/server/worker_task.rs
index 967814c9..a4b2a35f 100644
--- a/src/server/worker_task.rs
+++ b/src/server/worker_task.rs
@@ -191,7 +191,7 @@ pub fn upid_read_status(upid: &UPID) -> Result<TaskState, Error> {
     file.read_to_end(&mut data)?;
 
     // task logs should end with newline, we do not want it here
-    if !data.is_empty() && data[data.len()-1] == b'\n' {
+    while !data.is_empty() && data[data.len()-1] == b'\n' {
         data.pop();
     }
 
-- 
2.20.1