From 74c2d96f248c698f81472aa3a1c68145f716c185 Mon Sep 17 00:00:00 2001 From: Philippe Antoine Date: Fri, 1 Jul 2022 15:56:29 +0200 Subject: [PATCH] mqtt: do not log reason_codes if there is none Ticket: #5167 --- rust/src/mqtt/logger.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/rust/src/mqtt/logger.rs b/rust/src/mqtt/logger.rs index d7b343931ed4..22fc7de07927 100644 --- a/rust/src/mqtt/logger.rs +++ b/rust/src/mqtt/logger.rs @@ -233,11 +233,13 @@ fn log_mqtt(tx: &MQTTTransaction, flags: u32, js: &mut JsonBuilder) -> Result<() log_mqtt_header(js, &msg.header)?; js.set_uint("message_id", unsuback.message_id as u64)?; if let Some(codes) = &unsuback.reason_codes { - js.open_array("reason_codes")?; - for t in codes { - js.append_uint(*t as u64)?; + if codes.len() > 0 { + js.open_array("reason_codes")?; + for t in codes { + js.append_uint(*t as u64)?; + } + js.close()?; // reason_codes } - js.close()?; // reason_codes } js.close()?; // unsuback }