From 6c44bfa3fb257f1a0a014306339ce691d8a2b88c Mon Sep 17 00:00:00 2001 From: Arne Welzel Date: Mon, 14 Apr 2025 14:06:59 +0200 Subject: [PATCH] broker: Include log event identifier in stderr output --- src/broker/Manager.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/broker/Manager.cc b/src/broker/Manager.cc index a77f050ede..5cb63149c0 100644 --- a/src/broker/Manager.cc +++ b/src/broker/Manager.cc @@ -1167,8 +1167,11 @@ void Manager::ProcessLogEvents() { event_mgr.Enqueue(::Broker::internal_log_event, std::move(args)); } if ( bstate->stderrSeverity >= severity ) { - fprintf(stderr, "[BROKER/%s] %s\n", severity_names_tbl[static_cast(severity)], - event->description.c_str()); + // Formatting the event->identifier string_view using "%.*s" - the explicit + // precision ".*" allows specifying the length of the following char* argument + // as string_views in general are not guaranteed to be null terminated. + fprintf(stderr, "[BROKER/%s] %.*s: %s\n", severity_names_tbl[static_cast(severity)], + static_cast(event->identifier.size()), event->identifier.data(), event->description.c_str()); } } }