From a08b1ff56ffa35d3fca828882bc9fc86d92f18e7 Mon Sep 17 00:00:00 2001 From: Jon Siwek Date: Wed, 29 Apr 2020 15:25:03 -0700 Subject: [PATCH] Add warning message for unknown Broker statuses There's now a couple placeholder/unimplemented status values in Broker related to upcoming routing features that we don't want to handle explicitly for compatibility reasons, but also don't want the compiler warning about unhandled values in the switch. --- CHANGES | 9 +++++++++ VERSION | 2 +- aux/broker | 2 +- src/broker/Manager.cc | 4 ++++ 4 files changed, 15 insertions(+), 2 deletions(-) diff --git a/CHANGES b/CHANGES index afbbe7e1b0..5f40755c6c 100644 --- a/CHANGES +++ b/CHANGES @@ -1,4 +1,13 @@ +3.2.0-dev.445 | 2020-04-29 15:25:03 -0700 + + * Add warning message for unknown Broker statuses (Jon Siwek, Corelight) + + There's now a couple placeholder/unimplemented status values in Broker + related to upcoming routing features that we don't want to handle + explicitly for compatibility reasons, but also don't want the compiler + warning about unhandled values in the switch. + 3.2.0-dev.443 | 2020-04-28 17:10:38 -0700 * GH-941: Fix build when configured to use libmaxminddb (Jon Siwek, Corelight) diff --git a/VERSION b/VERSION index 0196cb0a94..0d2171ff80 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -3.2.0-dev.443 +3.2.0-dev.445 diff --git a/aux/broker b/aux/broker index 3375d5703e..8d84b0268a 160000 --- a/aux/broker +++ b/aux/broker @@ -1 +1 @@ -Subproject commit 3375d5703e0c805455649fff4bf30827f63dbf74 +Subproject commit 8d84b0268a3dcd2edd2ccccdac16e3279f52b729 diff --git a/src/broker/Manager.cc b/src/broker/Manager.cc index 3e44fef00d..f2b4d10c33 100644 --- a/src/broker/Manager.cc +++ b/src/broker/Manager.cc @@ -1229,6 +1229,10 @@ void Manager::ProcessStatus(broker::status stat) --peer_count; event = Broker::peer_lost; break; + + default: + reporter->Warning("Unhandled Broker status: %s", to_string(stat).data()); + break; } if ( ! event )