From 51b2f284f086a9b6a60744a730f4599b68a7535c Mon Sep 17 00:00:00 2001 From: Grant Limberg Date: Wed, 16 Jul 2025 09:05:54 -0700 Subject: [PATCH] call DB::eraseNetwork when it should be One of those i'm not sure how this ever worked situations --- controller/DB.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/controller/DB.cpp b/controller/DB.cpp index 1e8aef5ec..827f549d1 100644 --- a/controller/DB.cpp +++ b/controller/DB.cpp @@ -456,6 +456,9 @@ void DB::_networkChanged(nlohmann::json& old, nlohmann::json& networkConfig, boo auto provider = opentelemetry::trace::Provider::GetTracerProvider(); auto tracer = provider->GetTracer("db"); auto span = tracer->StartSpan("db::_networkChanged"); + span->SetAttribute("old_network_config", old.dump()); + span->SetAttribute("network_config", networkConfig.dump()); + span->SetAttribute("notify_listeners", notifyListeners); auto scope = tracer->WithActiveSpan(span); Metrics::db_network_change++; @@ -517,6 +520,8 @@ void DB::_networkChanged(nlohmann::json& old, nlohmann::json& networkConfig, boo std::cerr << "Error deauthorizing members on network delete: " << e.what() << std::endl; } + this->eraseNetwork(networkId); + // delete the network std::unique_lock l(_networks_l); _networks.erase(networkId);