Prevent serialization in LogProtobufMessage if log level is not met

[Molter73]

Description

This is a small PR to prevent protobuf message serialization on the stdout clients when log level would discard the result.

This changed was triggered by this comment: #2063 (comment)

Checklist

• Investigated and inspected CI test results
• Updated documentation accordingly

Automated testing

• Added unit tests
• Added integration tests
• Added regression tests

If any of these don't apply, please comment below.

Testing Performed

Was not able to test this locally, help would be appreciated.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 27.50%. Comparing base (4de6015) to head (f517932).

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
collector/lib/Utility.cpp	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2077      +/-   ##
==========================================
- Coverage   27.51%   27.50%   -0.01%     
==========================================
  Files          93       93              
  Lines        5728     5730       +2     
  Branches     2534     2535       +1     
==========================================
  Hits         1576     1576              
- Misses       3485     3487       +2     
  Partials      667      667              

Flag	Coverage Δ
collector-unit-tests	27.50% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JoukoVirtanen]

Did the following testing. Applied the following diff

diff --git a/collector/lib/NetworkStatusNotifier.cpp b/collector/lib/NetworkStatusNotifier.cpp
index 27efaa6ad..5abe8e046 100644
--- a/collector/lib/NetworkStatusNotifier.cpp
+++ b/collector/lib/NetworkStatusNotifier.cpp
@@ -342,6 +342,7 @@ void NetworkStatusNotifier::AddConnections(::google::protobuf::RepeatedPtrField<
 
     added_events++;
     updates->AddAllocated(conn_proto);
+    LogProtobufMessage(*conn_proto);
   }
 
   for (const auto& [id, events] : rate_limited_containers) {
diff --git a/collector/lib/Utility.cpp b/collector/lib/Utility.cpp
index 92f0a0b03..10e017df2 100644
--- a/collector/lib/Utility.cpp
+++ b/collector/lib/Utility.cpp
@@ -255,7 +255,9 @@ std::optional<std::string> SanitizedUTF8(std::string_view str) {
 void LogProtobufMessage(const google::protobuf::Message& msg) {
   using namespace google::protobuf::util;
 
+  CLOG(INFO) << "In LogProtobufMessage";
   if (!logging::CheckLogLevel(logging::LogLevel::DEBUG)) {
+    CLOG(INFO) << "Leaving early";
     return;
   }

Built collector and ran an integration test

make -C integration-tests TestRuntimeConfigFile

It doesn't matter which one is run. Checked the collector logs

[INFO    2025/04/09 23:03:40] (Utility.cpp:258) In LogProtobufMessage
[DEBUG   2025/04/09 23:03:40] (Utility.cpp:267) GRPC: {"remoteAddress":{"port":53,"ipNetwork":"CAgICCA="},"protocol":"L4_PROTOCOL_TCP","role":"ROLE_CLIENT","containerId":"63f007f892f5"}

When running with log level set to info

COLLECTOR_LOG_LEVEL=info make -C integration-tests TestRuntimeConfigFile

[INFO    2025/04/09 23:20:43] In LogProtobufMessage
[INFO    2025/04/09 23:20:43] Leaving early

[JoukoVirtanen]

LGTM!