commit 82b286fb77833c70bd670f3b428e49f6d9416d06
author Ed Tanous <ed@tanous.net> Tue May 06 13:29:48 2025 -0700
committer Ed Tanous <ed@tanous.net> Tue Jun 03 21:58:41 2025 +0000
tree 11f5411c1d7078e394e42f4a135ee26e0a065dfa
parent a49436930f9295bb74908dbd002409ee96325ea6

Remove implicit conversions

Since 2020, nlohmann has recognized that implicit conversions to and
from json are an issue.  Many bugs have been caused at both development
time and runtime due to unexpected implicit conversions from json to
std::string/int/bool.  This commit disables implicit conversions using
JSON_USE_IMPLICIT_CONVERSIONS [1].  This option will become the default
in the future.  That comment was written 3 years ago at this point, so
we should prepare.

Tested:
Redfish service validator passes.

[1] https://json.nlohmann.me/api/macros/json_use_implicit_conversions/
Change-Id: Id6cc47b9bbf8889e4777fd6d77ec992f3139962c
Signed-off-by: Ed Tanous <etanous@nvidia.com>

meson.build

diff --git a/meson.build b/meson.build
index e28a1b1..0038115 100644
--- a/meson.build
+++ b/meson.build
@@ -190,6 +190,7 @@
             '-DBOOST_URL_NO_SOURCE_LOCATION',
             '-DBOOST_SPIRIT_X3_NO_RTTI',
             '-DJSON_NOEXCEPTION',
+            '-DJSON_USE_IMPLICIT_CONVERSIONS=0',
             '-DOPENSSL_NO_FILENAMES',
             '-DSDBUSPLUS_DISABLE_BOOST_COROUTINES',
         ],