diff options
author | Ed Tanous <ed@tanous.net> | 2024-05-02 02:23:54 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-05-02 02:29:09 +0300 |
commit | 36a59eafed32c0063d444fc0722f899c0024092d (patch) | |
tree | 5415dc04035412de4d7450887184c393462b7c2f | |
parent | 25b54dba775b31021a3a4677eb79e9771bcb97f7 (diff) | |
download | bmcweb-master.tar.xz |
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.cpp | 2 |
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); |