From 869e4601b4634d07b444e796e69d55b7459a1df3 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Mon, 11 Oct 2021 14:56:28 +0200 Subject: [PATCH] api daemons: fix sending log-reopen command send_command serializes everything so it cannot be used to send a raw, optimized command. Normally that means we get an error like > 'unable to parse parameters (expected json object)' when used that way. Switch over to send_raw_command which does not re-serializes the command. Fixes: 45b8a032 ("refactor send_command") Originally-by: Dominik Csapak Signed-off-by: Thomas Lamprecht --- src/bin/proxmox-backup-proxy.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/bin/proxmox-backup-proxy.rs b/src/bin/proxmox-backup-proxy.rs index d6194bc9..f75023c0 100644 --- a/src/bin/proxmox-backup-proxy.rs +++ b/src/bin/proxmox-backup-proxy.rs @@ -737,11 +737,11 @@ async fn command_reopen_logfiles() -> Result<(), Error> { // only care about the most recent daemon instance for each, proxy & api, as other older ones // should not respond to new requests anyway, but only finish their current one and then exit. let sock = server::our_ctrl_sock(); - let f1 = server::send_command(sock, "{\"command\":\"api-access-log-reopen\"}\n"); + let f1 = server::send_raw_command(sock, "{\"command\":\"api-access-log-reopen\"}\n"); let pid = server::read_pid(buildcfg::PROXMOX_BACKUP_API_PID_FN)?; let sock = server::ctrl_sock_from_pid(pid); - let f2 = server::send_command(sock, "{\"command\":\"api-access-log-reopen\"}\n"); + let f2 = server::send_raw_command(sock, "{\"command\":\"api-access-log-reopen\"}\n"); match futures::join!(f1, f2) { (Err(e1), Err(e2)) => Err(format_err!("reopen commands failed, proxy: {}; api: {}", e1, e2)),