Allow boost url and url_view to be added to json

The latest version of nlohmann seems to have support for adding any
arbitrary iterable object as an array in json.  Unfortunately, because
boost::urls::url produces at iterable of unsigned char, this means that
trying to encode urls leads to something like:

"@odata.id": [
47,
114,
101,
100,
102
]

Which is super unhelpful in that it does this implicitly.  Given this
behavior, there are two options here, make it so that code doesn't
compile, or rely on the adl_serializer to just do the expected thing.

This patchset opts for the later, to simply to the reasonable behavior,
and call string() on the url before loading it into the json.

Tested: Unit tests passing.  Fixes bug in subsequent patchset.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Id2f49bc8bd7153a0ad0c0fa8be2e13ce7c538e7f

1 files changed, 9 insertions, 0 deletions

diff --git a/http/ut/utility_test.cpp b/http/ut/utility_test.cpp
index bab21f96c0..1600e7f54f 100644
--- a/http/ut/utility_test.cpp
+++ b/http/ut/utility_test.cpp
@@ -247,5 +247,14 @@ TEST(Router, ParameterTagging)
               crow::black_magic::getParameterTag("<uint><double><int>"));
 }
 
+TEST(URL, JsonEncoding)
+{
+    using nlohmann::json;
+
+    std::string urlString = "/foo";
+    EXPECT_EQ(json(boost::urls::url(urlString)), urlString);
+    EXPECT_EQ(json(boost::urls::url_view(urlString)), urlString);
+}
+
 } // namespace
 } // namespace crow::utility