From 36a59eafed32c0063d444fc0722f899c0024092d Mon Sep 17 00:00:00 2001 From: Ed Tanous Date: Wed, 1 May 2024 16:23:54 -0700 Subject: Fix regression in LogService 25b54db introduced an inadvertent inversion in options. Admittedly, this option (redfish-dbus-entries) chose to override URLs instead of creating new ones, which makes it incompatible. Normally we'd require unique URIs for each entry to make this error not possible, and I believe that this is the only instance of us registering two handlers for one url/verb. Tested: Romulus now boots and functions in qemu. GET /redfish/v1 returns results. Change-Id: I6125a5a0242b6cfc54ff19866665227c97f45d0a Signed-off-by: Ed Tanous --- redfish-core/src/redfish.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/redfish-core/src/redfish.cpp b/redfish-core/src/redfish.cpp index e7a58c1f41..47d6fc2d07 100644 --- a/redfish-core/src/redfish.cpp +++ b/redfish-core/src/redfish.cpp @@ -130,7 +130,7 @@ RedfishService::RedfishService(App& app) requestRoutesFaultLogDumpClear(app); } - if constexpr (BMCWEB_REDFISH_DBUS_LOG) + if constexpr (!BMCWEB_REDFISH_DBUS_LOG) { requestRoutesJournalEventLogEntryCollection(app); requestRoutesJournalEventLogEntry(app); -- cgit v1.2.3