summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--.clang-tidy1
-rw-r--r--http/http_request.hpp6
-rw-r--r--http/utility.hpp30
-rw-r--r--http/websocket.hpp4
-rw-r--r--meson.build13
-rw-r--r--redfish-core/include/event_service_manager.hpp2
-rw-r--r--redfish-core/include/redfish_aggregator.hpp9
-rw-r--r--redfish-core/include/utils/collection.hpp2
-rw-r--r--redfish-core/include/utils/query_param.hpp45
-rw-r--r--redfish-core/lib/metric_report.hpp12
-rw-r--r--redfish-core/lib/metric_report_definition.hpp9
-rw-r--r--redfish-core/lib/redfish_v1.hpp5
-rw-r--r--redfish-core/lib/virtual_media.hpp8
-rw-r--r--redfish-core/src/error_messages.cpp35
-rw-r--r--subprojects/boost-url.wrap6
-rw-r--r--subprojects/boost.wrap8
-rw-r--r--test/http/utility_test.cpp10
17 files changed, 79 insertions, 126 deletions
diff --git a/.clang-tidy b/.clang-tidy
index 8af0822656..7d9979fafe 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -9,7 +9,6 @@ bugprone-branch-clone,
bugprone-copy-constructor-init,
bugprone-dangling-handle,
bugprone-dynamic-static-initializers,
-bugprone-exception-escape,
bugprone-fold-init-type,
bugprone-forward-declaration-namespace,
bugprone-forwarding-reference-overload,
diff --git a/http/http_request.hpp b/http/http_request.hpp
index deaba31c41..d203077e10 100644
--- a/http/http_request.hpp
+++ b/http/http_request.hpp
@@ -116,16 +116,14 @@ struct Request
private:
bool setUrlInfo()
{
- auto result = boost::urls::parse_relative_ref(
- boost::urls::string_view(target().data(), target().size()));
+ auto result = boost::urls::parse_relative_ref(target());
if (!result)
{
return false;
}
urlView = *result;
- url = std::string_view(urlView.encoded_path().data(),
- urlView.encoded_path().size());
+ url = urlView.encoded_path();
return true;
}
};
diff --git a/http/utility.hpp b/http/utility.hpp
index b1b38da6f7..4f5cea74fd 100644
--- a/http/utility.hpp
+++ b/http/utility.hpp
@@ -4,7 +4,9 @@
#include <openssl/crypto.h>
#include <boost/callable_traits.hpp>
+#include <boost/url/parse.hpp>
#include <boost/url/url.hpp>
+#include <boost/url/url_view.hpp>
#include <nlohmann/json.hpp>
#include <array>
@@ -616,13 +618,13 @@ class UrlSegmentMatcherVisitor
public:
UrlParseResult operator()(std::string& output)
{
- output = std::string_view(segment.data(), segment.size());
+ output = segment;
return UrlParseResult::Continue;
}
UrlParseResult operator()(std::string_view expected)
{
- if (std::string_view(segment.data(), segment.size()) == expected)
+ if (segment == expected)
{
return UrlParseResult::Continue;
}
@@ -634,13 +636,12 @@ class UrlSegmentMatcherVisitor
return UrlParseResult::Done;
}
- explicit UrlSegmentMatcherVisitor(
- const boost::urls::string_value& segmentIn) :
+ explicit UrlSegmentMatcherVisitor(std::string_view segmentIn) :
segment(segmentIn)
{}
private:
- const boost::urls::string_value& segment;
+ std::string_view segment;
};
inline bool readUrlSegments(const boost::urls::url_view& urlView,
@@ -766,9 +767,8 @@ inline bool validateAndSplitUrl(std::string_view destUrl, std::string& urlProto,
std::string& host, uint16_t& port,
std::string& path)
{
- boost::string_view urlBoost(destUrl.data(), destUrl.size());
boost::urls::result<boost::urls::url_view> url =
- boost::urls::parse_uri(urlBoost);
+ boost::urls::parse_uri(destUrl);
if (!url)
{
return false;
@@ -781,11 +781,9 @@ inline bool validateAndSplitUrl(std::string_view destUrl, std::string& urlProto,
port = setPortDefaults(url.value());
- host = std::string_view(url->encoded_host().data(),
- url->encoded_host().size());
+ host = url->encoded_host();
- path = std::string_view(url->encoded_path().data(),
- url->encoded_path().size());
+ path = url->encoded_path();
if (path.empty())
{
path = "/";
@@ -793,15 +791,13 @@ inline bool validateAndSplitUrl(std::string_view destUrl, std::string& urlProto,
if (url->has_fragment())
{
path += '#';
- path += std::string_view(url->encoded_fragment().data(),
- url->encoded_fragment().size());
+ path += url->encoded_fragment();
}
if (url->has_query())
{
path += '?';
- path += std::string_view(url->encoded_query().data(),
- url->encoded_query().size());
+ path += url->encoded_query();
}
return true;
@@ -819,7 +815,7 @@ struct adl_serializer<boost::urls::url>
// NOLINTNEXTLINE(readability-identifier-naming)
static void to_json(json& j, const boost::urls::url& url)
{
- j = url.string();
+ j = url.buffer();
}
};
@@ -829,7 +825,7 @@ struct adl_serializer<boost::urls::url_view>
// NOLINTNEXTLINE(readability-identifier-naming)
static void to_json(json& j, const boost::urls::url_view& url)
{
- j = url.string();
+ j = url.buffer();
}
};
} // namespace nlohmann
diff --git a/http/websocket.hpp b/http/websocket.hpp
index ecaa1d2f9a..3e4ef0c40c 100644
--- a/http/websocket.hpp
+++ b/http/websocket.hpp
@@ -218,8 +218,8 @@ class ConnectionImpl : public Connection
}
if (closeHandler)
{
- std::string_view reason = ws.reason().reason;
- closeHandler(*this, std::string(reason));
+ std::string reason{ws.reason().reason.c_str()};
+ closeHandler(*this, reason);
}
return;
}
diff --git a/meson.build b/meson.build
index 4c03ef9613..967b30933f 100644
--- a/meson.build
+++ b/meson.build
@@ -168,8 +168,8 @@ if (cxx.get_id() == 'gcc' and cxx.version().version_compare('>8.0'))
'-Wduplicated-cond',
'-Wduplicated-branches',
'-Wlogical-op',
- '-Wunused-parameter',
'-Wnull-dereference',
+ '-Wunused-parameter',
'-Wdouble-promotion',
'-Wshadow',
'-Wno-psabi',
@@ -241,7 +241,6 @@ cxx.get_supported_arguments([
'-DBOOST_ASIO_NO_DEPRECATED',
'-DBOOST_ASIO_SEPARATE_COMPILATION',
'-DBOOST_BEAST_SEPARATE_COMPILATION',
- '-DBOOST_BEAST_USE_STD_STRING_VIEW',
'-DBOOST_EXCEPTION_DISABLE',
'-DBOOST_URL_NO_SOURCE_LOCATION',
'-DJSON_NOEXCEPTION',
@@ -285,21 +284,13 @@ else
endif
bmcweb_dependencies += nlohmann_json
-boost = dependency('boost',version : '>=1.80.0', required : false, include_type: 'system')
+boost = dependency('boost',version : '>=1.81.0', required : false, include_type: 'system')
if not boost.found()
boost = subproject('boost', required: true).get_variable('boost_dep')
boost = boost.as_system('system')
endif
bmcweb_dependencies += boost
-if cxx.has_header('boost/url/url_view.hpp')
- boost_url = declare_dependency()
-else
- boost_url = subproject('boost-url', required: true).get_variable('boost_url_dep')
- boost_url = boost_url.as_system('system')
-endif
-bmcweb_dependencies += boost_url
-
if get_option('tests').enabled()
gtest = dependency('gtest', main: true,disabler: true, required : false)
gmock = dependency('gmock', required : false)
diff --git a/redfish-core/include/event_service_manager.hpp b/redfish-core/include/event_service_manager.hpp
index 7f3b43a8da..6f1e91b2d9 100644
--- a/redfish-core/include/event_service_manager.hpp
+++ b/redfish-core/include/event_service_manager.hpp
@@ -519,7 +519,7 @@ class Subscription : public persistent_data::UserSubscription
{
if (std::find(metricReportDefinitions.begin(),
metricReportDefinitions.end(),
- mrdUri.string()) == metricReportDefinitions.end())
+ mrdUri.buffer()) == metricReportDefinitions.end())
{
return;
}
diff --git a/redfish-core/include/redfish_aggregator.hpp b/redfish-core/include/redfish_aggregator.hpp
index 99da8ee1e0..8170b6436b 100644
--- a/redfish-core/include/redfish_aggregator.hpp
+++ b/redfish-core/include/redfish_aggregator.hpp
@@ -312,7 +312,7 @@ class RedfishAggregator
BMCWEB_LOG_ERROR << "Port value out of range";
return;
}
- url.set_port(static_cast<uint16_t>(*propVal));
+ url.set_port(std::to_string(static_cast<uint16_t>(*propVal)));
}
else if (prop.first == "AuthType")
@@ -435,8 +435,7 @@ class RedfishAggregator
}
// We didn't recognize the prefix and need to return a 404
- boost::urls::string_value name = req.urlView.segments().back();
- std::string_view nameStr(name.data(), name.size());
+ std::string_view nameStr = req.urlView.segments().back();
messages::resourceNotFound(asyncResp->res, "", nameStr);
}
@@ -461,9 +460,7 @@ class RedfishAggregator
// don't need to write an error code
if (isCollection == AggregationType::Resource)
{
- boost::urls::string_value name =
- sharedReq->urlView.segments().back();
- std::string_view nameStr(name.data(), name.size());
+ std::string_view nameStr = sharedReq->urlView.segments().back();
messages::resourceNotFound(asyncResp->res, "", nameStr);
}
return;
diff --git a/redfish-core/include/utils/collection.hpp b/redfish-core/include/utils/collection.hpp
index 97f622d8f4..f11730279d 100644
--- a/redfish-core/include/utils/collection.hpp
+++ b/redfish-core/include/utils/collection.hpp
@@ -29,7 +29,7 @@ inline void
const char* subtree = "/xyz/openbmc_project/inventory")
{
BMCWEB_LOG_DEBUG << "Get collection members for: "
- << collectionPath.string();
+ << collectionPath.buffer();
crow::connections::systemBus->async_method_call(
[collectionPath, aResp{std::move(aResp)}](
const boost::system::error_code ec,
diff --git a/redfish-core/include/utils/query_param.hpp b/redfish-core/include/utils/query_param.hpp
index fec5384b7e..bfe1001029 100644
--- a/redfish-core/include/utils/query_param.hpp
+++ b/redfish-core/include/utils/query_param.hpp
@@ -16,7 +16,6 @@
#include <boost/beast/http/status.hpp>
#include <boost/beast/http/verb.hpp>
#include <boost/url/params_view.hpp>
-#include <boost/url/string.hpp>
#include <nlohmann/json.hpp>
#include <algorithm>
@@ -374,80 +373,78 @@ inline bool getSelectParam(std::string_view value, Query& query)
return true;
}
-inline std::optional<Query>
- parseParameters(const boost::urls::params_view& urlParams,
- crow::Response& res)
+inline std::optional<Query> parseParameters(boost::urls::params_view urlParams,
+ crow::Response& res)
{
Query ret;
for (const boost::urls::params_view::value_type& it : urlParams)
{
- std::string_view key(it.key.data(), it.key.size());
- std::string_view value(it.value.data(), it.value.size());
- if (key == "only")
+ if (it.key == "only")
{
if (!it.value.empty())
{
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
return std::nullopt;
}
ret.isOnly = true;
}
- else if (key == "$expand" && bmcwebInsecureEnableQueryParams)
+ else if (it.key == "$expand" && bmcwebInsecureEnableQueryParams)
{
- if (!getExpandType(value, ret))
+ if (!getExpandType(it.value, ret))
{
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
return std::nullopt;
}
}
- else if (key == "$top")
+ else if (it.key == "$top")
{
- QueryError topRet = getTopParam(value, ret);
+ QueryError topRet = getTopParam(it.value, ret);
if (topRet == QueryError::ValueFormat)
{
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
return std::nullopt;
}
if (topRet == QueryError::OutOfRange)
{
messages::queryParameterOutOfRange(
- res, value, "$top", "0-" + std::to_string(Query::maxTop));
+ res, it.value, "$top",
+ "0-" + std::to_string(Query::maxTop));
return std::nullopt;
}
}
- else if (key == "$skip")
+ else if (it.key == "$skip")
{
- QueryError topRet = getSkipParam(value, ret);
+ QueryError topRet = getSkipParam(it.value, ret);
if (topRet == QueryError::ValueFormat)
{
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
return std::nullopt;
}
if (topRet == QueryError::OutOfRange)
{
messages::queryParameterOutOfRange(
- res, value, key,
+ res, it.value, it.key,
"0-" + std::to_string(std::numeric_limits<size_t>::max()));
return std::nullopt;
}
}
- else if (key == "$select")
+ else if (it.key == "$select")
{
- if (!getSelectParam(value, ret))
+ if (!getSelectParam(it.value, ret))
{
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
return std::nullopt;
}
}
else
{
// Intentionally ignore other errors Redfish spec, 7.3.1
- if (key.starts_with("$"))
+ if (it.key.starts_with("$"))
{
// Services shall return... The HTTP 501 Not Implemented
// status code for any unsupported query parameters that
// start with $ .
- messages::queryParameterValueFormatError(res, value, key);
+ messages::queryParameterValueFormatError(res, it.value, it.key);
res.result(boost::beast::http::status::not_implemented);
return std::nullopt;
}
diff --git a/redfish-core/lib/metric_report.hpp b/redfish-core/lib/metric_report.hpp
index 33c8c9c826..0fb3a68e5c 100644
--- a/redfish-core/lib/metric_report.hpp
+++ b/redfish-core/lib/metric_report.hpp
@@ -42,16 +42,12 @@ inline bool fillReport(nlohmann::json& json, const std::string& id,
const TimestampReadings& timestampReadings)
{
json["@odata.type"] = "#MetricReport.v1_3_0.MetricReport";
- json["@odata.id"] =
- crow::utility::urlFromPieces("redfish", "v1", "TelemetryService",
- "MetricReports", id)
- .string();
+ json["@odata.id"] = crow::utility::urlFromPieces(
+ "redfish", "v1", "TelemetryService", "MetricReports", id);
json["Id"] = id;
json["Name"] = id;
- json["MetricReportDefinition"]["@odata.id"] =
- crow::utility::urlFromPieces("redfish", "v1", "TelemetryService",
- "MetricReportDefinitions", id)
- .string();
+ json["MetricReportDefinition"]["@odata.id"] = crow::utility::urlFromPieces(
+ "redfish", "v1", "TelemetryService", "MetricReportDefinitions", id);
const auto& [timestamp, readings] = timestampReadings;
json["Timestamp"] = redfish::time_utils::getDateTimeUintMs(timestamp);
diff --git a/redfish-core/lib/metric_report_definition.hpp b/redfish-core/lib/metric_report_definition.hpp
index bc1b894bfd..e6308eb404 100644
--- a/redfish-core/lib/metric_report_definition.hpp
+++ b/redfish-core/lib/metric_report_definition.hpp
@@ -34,16 +34,13 @@ inline void
{
asyncResp->res.jsonValue["@odata.type"] =
"#MetricReportDefinition.v1_3_0.MetricReportDefinition";
- asyncResp->res.jsonValue["@odata.id"] =
- crow::utility::urlFromPieces("redfish", "v1", "TelemetryService",
- "MetricReportDefinitions", id)
- .string();
+ asyncResp->res.jsonValue["@odata.id"] = crow::utility::urlFromPieces(
+ "redfish", "v1", "TelemetryService", "MetricReportDefinitions", id);
asyncResp->res.jsonValue["Id"] = id;
asyncResp->res.jsonValue["Name"] = id;
asyncResp->res.jsonValue["MetricReport"]["@odata.id"] =
crow::utility::urlFromPieces("redfish", "v1", "TelemetryService",
- "MetricReports", id)
- .string();
+ "MetricReports", id);
asyncResp->res.jsonValue["Status"]["State"] = "Enabled";
asyncResp->res.jsonValue["ReportUpdates"] = "Overwrite";
diff --git a/redfish-core/lib/redfish_v1.hpp b/redfish-core/lib/redfish_v1.hpp
index fa3ca363a0..95db449a7c 100644
--- a/redfish-core/lib/redfish_v1.hpp
+++ b/redfish-core/lib/redfish_v1.hpp
@@ -38,12 +38,11 @@ inline void redfish404(App& app, const crow::Request& req,
BMCWEB_LOG_ERROR << "404 on path " << path;
- boost::urls::string_value name = req.urlView.segments().back();
- std::string_view nameStr(name.data(), name.size());
+ std::string name = req.urlView.segments().back();
// Note, if we hit the wildcard route, we don't know the "type" the user was
// actually requesting, but giving them a return with an empty string is
// still better than nothing.
- messages::resourceNotFound(asyncResp->res, "", nameStr);
+ messages::resourceNotFound(asyncResp->res, "", name);
}
inline void redfish405(App& app, const crow::Request& req,
diff --git a/redfish-core/lib/virtual_media.hpp b/redfish-core/lib/virtual_media.hpp
index af38163fb8..402cbbd11d 100644
--- a/redfish-core/lib/virtual_media.hpp
+++ b/redfish-core/lib/virtual_media.hpp
@@ -32,12 +32,12 @@ namespace redfish
inline std::string getTransferProtocolTypeFromUri(const std::string& imageUri)
{
boost::urls::result<boost::urls::url_view> url =
- boost::urls::parse_uri(boost::string_view(imageUri));
+ boost::urls::parse_uri(imageUri);
if (!url)
{
return "None";
}
- boost::string_view scheme = url->scheme();
+ std::string_view scheme = url->scheme();
if (scheme == "smb")
{
return "CIFS";
@@ -294,7 +294,7 @@ enum class TransferProtocol
inline std::optional<TransferProtocol>
getTransferProtocolFromUri(const boost::urls::url_view& imageUri)
{
- boost::string_view scheme = imageUri.scheme();
+ std::string_view scheme = imageUri.scheme();
if (scheme == "smb")
{
return TransferProtocol::smb;
@@ -401,7 +401,7 @@ inline bool
return false;
}
boost::urls::result<boost::urls::url_view> url =
- boost::urls::parse_uri(boost::string_view(imageUrl));
+ boost::urls::parse_uri(imageUrl);
if (!url)
{
messages::actionParameterValueFormatError(res, imageUrl, "Image",
diff --git a/redfish-core/src/error_messages.cpp b/redfish-core/src/error_messages.cpp
index 5f055bfca7..bee3140685 100644
--- a/redfish-core/src/error_messages.cpp
+++ b/redfish-core/src/error_messages.cpp
@@ -206,8 +206,7 @@ void malformedJSON(crow::Response& res)
*/
nlohmann::json resourceMissingAtURI(const boost::urls::url_view& arg1)
{
- std::array<std::string_view, 1> args{
- std::string_view{arg1.data(), arg1.size()}};
+ std::array<std::string_view, 1> args{arg1.buffer()};
return getLog(redfish::registries::base::Index::resourceMissingAtURI, args);
}
@@ -293,9 +292,8 @@ void unrecognizedRequestBody(crow::Response& res)
nlohmann::json resourceAtUriUnauthorized(const boost::urls::url_view& arg1,
std::string_view arg2)
{
- return getLog(
- redfish::registries::base::Index::resourceAtUriUnauthorized,
- std::to_array({std::string_view{arg1.data(), arg1.size()}, arg2}));
+ return getLog(redfish::registries::base::Index::resourceAtUriUnauthorized,
+ std::to_array<std::string_view>({arg1.buffer(), arg2}));
}
void resourceAtUriUnauthorized(crow::Response& res,
@@ -524,10 +522,9 @@ void propertyValueOutOfRange(crow::Response& res, std::string_view arg1,
*/
nlohmann::json resourceAtUriInUnknownFormat(const boost::urls::url_view& arg1)
{
- std::string_view arg1str{arg1.data(), arg1.size()};
return getLog(
redfish::registries::base::Index::resourceAtUriInUnknownFormat,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
void resourceAtUriInUnknownFormat(crow::Response& res,
@@ -701,9 +698,8 @@ void resourceTypeIncompatible(crow::Response& res, std::string_view arg1,
nlohmann::json resetRequired(const boost::urls::url_view& arg1,
std::string_view arg2)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::resetRequired,
- std::to_array({arg1str, arg2}));
+ std::to_array<std::string_view>({arg1.buffer(), arg2}));
}
void resetRequired(crow::Response& res, const boost::urls::url_view& arg1,
@@ -786,8 +782,7 @@ nlohmann::json propertyValueResourceConflict(std::string_view arg1,
{
return getLog(
redfish::registries::base::Index::propertyValueResourceConflict,
- std::to_array(
- {arg1, arg2, std::string_view{arg3.data(), arg3.size()}}));
+ std::to_array<std::string_view>({arg1, arg2, arg3.buffer()}));
}
void propertyValueResourceConflict(crow::Response& res, std::string_view arg1,
@@ -852,9 +847,8 @@ void propertyValueIncorrect(crow::Response& res, std::string_view arg1,
*/
nlohmann::json resourceCreationConflict(const boost::urls::url_view& arg1)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::resourceCreationConflict,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
void resourceCreationConflict(crow::Response& res,
@@ -1005,9 +999,8 @@ void resourceNotFound(crow::Response& res, std::string_view arg1,
*/
nlohmann::json couldNotEstablishConnection(const boost::urls::url_view& arg1)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::couldNotEstablishConnection,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
void couldNotEstablishConnection(crow::Response& res,
@@ -1131,10 +1124,9 @@ void actionParameterNotSupported(crow::Response& res, std::string_view arg1,
nlohmann::json sourceDoesNotSupportProtocol(const boost::urls::url_view& arg1,
std::string_view arg2)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(
redfish::registries::base::Index::sourceDoesNotSupportProtocol,
- std::to_array({arg1str, arg2}));
+ std::to_array<std::string_view>({arg1.buffer(), arg2}));
}
void sourceDoesNotSupportProtocol(crow::Response& res,
@@ -1192,9 +1184,8 @@ void accountRemoved(crow::Response& res)
*/
nlohmann::json accessDenied(const boost::urls::url_view& arg1)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::accessDenied,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
void accessDenied(crow::Response& res, const boost::urls::url_view& arg1)
@@ -1360,9 +1351,8 @@ void noValidSession(crow::Response& res)
*/
nlohmann::json invalidObject(const boost::urls::url_view& arg1)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::invalidObject,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
void invalidObject(crow::Response& res, const boost::urls::url_view& arg1)
@@ -1707,9 +1697,8 @@ void queryParameterOutOfRange(crow::Response& res, std::string_view arg1,
nlohmann::json passwordChangeRequired(const boost::urls::url_view& arg1)
{
- std::string_view arg1str(arg1.data(), arg1.size());
return getLog(redfish::registries::base::Index::passwordChangeRequired,
- std::to_array({arg1str}));
+ std::to_array<std::string_view>({arg1.buffer()}));
}
/**
diff --git a/subprojects/boost-url.wrap b/subprojects/boost-url.wrap
deleted file mode 100644
index d19baba12f..0000000000
--- a/subprojects/boost-url.wrap
+++ /dev/null
@@ -1,6 +0,0 @@
-[wrap-git]
-directory = boost-url-git
-
-revision = d740a92d38e3a8f4d5b2153f53b82f1c98e312ab
-url = https://github.com/CPPAlliance/url.git
-patch_directory = boost-url
diff --git a/subprojects/boost.wrap b/subprojects/boost.wrap
index 75735f4d0a..1e44ebfece 100644
--- a/subprojects/boost.wrap
+++ b/subprojects/boost.wrap
@@ -1,9 +1,9 @@
[wrap-file]
-directory = boost_1_80_0
+directory = boost_1_81_0
-source_url = https://downloads.yoctoproject.org/mirror/sources/boost_1_80_0.tar.bz2
-source_hash = 1e19565d82e43bc59209a168f5ac899d3ba471d55c7610c677d4ccf2c9c500c0
-source_filename = 1_80_0.tar.bz2
+source_url = https://boostorg.jfrog.io/artifactory/main/release/1.81.0/source/boost_1_81_0.tar.bz2
+source_hash = 941c568e7ac79aa448ac28c98a5ec391fd1317170953c487bcf977c6ee6061ce
+source_filename = 1_81_0.tar.bz2
patch_directory = boost
diff --git a/test/http/utility_test.cpp b/test/http/utility_test.cpp
index 58a28654e1..2e0c82e4d7 100644
--- a/test/http/utility_test.cpp
+++ b/test/http/utility_test.cpp
@@ -84,16 +84,16 @@ TEST(Utility, Base64EncodeDecodeString)
TEST(Utility, UrlFromPieces)
{
boost::urls::url url = urlFromPieces("redfish", "v1", "foo");
- EXPECT_EQ(std::string_view(url.data(), url.size()), "/redfish/v1/foo");
+ EXPECT_EQ(url.buffer(), "/redfish/v1/foo");
url = urlFromPieces("/", "badString");
- EXPECT_EQ(std::string_view(url.data(), url.size()), "/%2f/badString");
+ EXPECT_EQ(url.buffer(), "/%2F/badString");
url = urlFromPieces("bad?tring");
- EXPECT_EQ(std::string_view(url.data(), url.size()), "/bad%3ftring");
+ EXPECT_EQ(url.buffer(), "/bad%3Ftring");
url = urlFromPieces("/", "bad&tring");
- EXPECT_EQ(std::string_view(url.data(), url.size()), "/%2f/bad&tring");
+ EXPECT_EQ(url.buffer(), "/%2F/bad&tring");
}
TEST(Utility, readUrlSegments)
@@ -232,7 +232,7 @@ TEST(AppendUrlFromPieces, PiecesAreAppendedViaDelimiters)
appendUrlPieces(url, "/", "bad&tring");
EXPECT_EQ(std::string_view(url.data(), url.size()),
- "/redfish/v1/foo/bar/%2f/bad&tring");
+ "/redfish/v1/foo/bar/%2F/bad&tring");
}
} // namespace
generated by cgit v1.2.3 (git 2.39.0) at 2024-06-01 15:00:53 +0300