diff options
author | Ed Tanous <ed@tanous.net> | 2024-05-10 01:48:09 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-05-10 02:16:06 +0300 |
commit | 83328316857b42960edd46cadf06902d2d4ba00b (patch) | |
tree | 1329cd1fe0eb4084411f8d9459a52f8cf3f0ed14 /http/parsing.hpp | |
parent | 3eaecac33685ca3297d53e041ab5c5116bc18d25 (diff) | |
download | bmcweb-83328316857b42960edd46cadf06902d2d4ba00b.tar.xz |
Fix lesser used options
25b54dba775b31021a3a4677eb79e9771bcb97f7 missed several cases where we
had ifndef instead of ifdef. because these weren't the defaults, these
don't show up as failures when testing.
Tested: Redfish service validator passes. Inspection primarily.
Mechanical change.
Change-Id: I3f6915a97eb44d071795aed76476c6bee7e8ed27
Signed-off-by: Ed Tanous <ed@tanous.net>
Diffstat (limited to 'http/parsing.hpp')
-rw-r--r-- | http/parsing.hpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/http/parsing.hpp b/http/parsing.hpp index cf813537ed..d59621104c 100644 --- a/http/parsing.hpp +++ b/http/parsing.hpp @@ -32,9 +32,10 @@ inline JsonParseResult parseRequestAsJson(const crow::Request& req, req.getHeaderValue(boost::beast::http::field::content_type))) { BMCWEB_LOG_WARNING("Failed to parse content type on request"); -#ifndef BMCWEB_INSECURE_IGNORE_CONTENT_TYPE - return JsonParseResult::BadContentType; -#endif + if constexpr (!BMCWEB_INSECURE_IGNORE_CONTENT_TYPE) + { + return JsonParseResult::BadContentType; + } } jsonOut = nlohmann::json::parse(req.body(), nullptr, false); if (jsonOut.is_discarded()) |