summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEd Tanous <ed@tanous.net>2024-05-02 02:23:54 +0300
committerEd Tanous <ed@tanous.net>2024-05-02 02:29:09 +0300
commit36a59eafed32c0063d444fc0722f899c0024092d (patch)
tree5415dc04035412de4d7450887184c393462b7c2f
parent25b54dba775b31021a3a4677eb79e9771bcb97f7 (diff)
downloadbmcweb-master.tar.xz
Fix regression in LogServiceHEADmaster
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 <ed@tanous.net>
-rw-r--r--redfish-core/src/redfish.cpp2
1 files changed, 1 insertions, 1 deletions
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);