Re: [pbs-devel] [PATCH proxmox-backup v2 1/2] rest-server/worker-task: replace newlines with '\n' in task result

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

On 11.10.21 14:14, Dominik Csapak wrote:
> we parse the task result from the last line, so we should not print a
> new line in the task result, else we get an 'unknown' task state
> 

isn't the issue that we already print the newline in the file_logger's log method
this is using underneath and we could get two newlines (i.e. a last empty line)
which then failed to parse?

> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
> ---
> no changes
>  proxmox-rest-server/src/worker_task.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/proxmox-rest-server/src/worker_task.rs b/proxmox-rest-server/src/worker_task.rs
> index 51394549..a8899ab9 100644
> --- a/proxmox-rest-server/src/worker_task.rs
> +++ b/proxmox-rest-server/src/worker_task.rs
> @@ -494,7 +494,7 @@ impl TaskState {
>          match self {
>              TaskState::Error { message, .. } => format!("TASK ERROR: {}", message),
>              other => format!("TASK {}", other),
> -        }
> +        }.replace('\n', "\\n") // no newline in task result

Why not `.trim_end()` ? A literal \n seems rather odd to me..

Comment suggestion:

// our consumer already add \n where required, so avoid double-newline

>      }
>  
>      fn from_endtime_and_message(endtime: i64, s: &str) -> Result<Self, Error> {
>