Don't rely on operator << for object logging
In the upcoming fmt patch, we remove the use of streams, and a number of
our logging statements are relying on them. This commit changes them to
no longer rely on operator>> or operator+ to build their strings. This
alone isn't very useful, but in the context of the next patch makes the
automation able to do a complete conversion of all log statements
automatically.
Tested: enabled logging on local and saw log statements print to console
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I0e5dc2cf015c6924037e38d547535eda8175a6a1
diff --git a/include/ibm/management_console_rest.hpp b/include/ibm/management_console_rest.hpp
index a9e0ec4..8e30746 100644
--- a/include/ibm/management_console_rest.hpp
+++ b/include/ibm/management_console_rest.hpp
@@ -136,7 +136,7 @@
// Form the file path
loc /= fileID;
- BMCWEB_LOG_DEBUG << "Writing to the file: " << loc;
+ BMCWEB_LOG_DEBUG << "Writing to the file: " << loc.string();
// Check if the same file exists in the directory
bool fileExists = std::filesystem::exists(loc, ec);
@@ -305,7 +305,7 @@
"/var/lib/bmcweb/ibm-management-console/configfiles/" + fileID);
if (!std::filesystem::exists(loc))
{
- BMCWEB_LOG_ERROR << loc << "Not found";
+ BMCWEB_LOG_ERROR << loc.string() << "Not found";
asyncResp->res.result(boost::beast::http::status::not_found);
asyncResp->res.jsonValue["Description"] = resourceNotFoundMsg;
return;