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/webassets.hpp b/include/webassets.hpp
index 8f41f2f..36c3f8d 100644
--- a/include/webassets.hpp
+++ b/include/webassets.hpp
@@ -57,7 +57,7 @@
std::filesystem::recursive_directory_iterator dirIter(rootpath, ec);
if (ec)
{
- BMCWEB_LOG_ERROR << "Unable to find or open " << rootpath
+ BMCWEB_LOG_ERROR << "Unable to find or open " << rootpath.string()
<< " static file hosting disabled";
return;
}
@@ -117,7 +117,7 @@
{
// Got a duplicated path. This is expected in certain
// situations
- BMCWEB_LOG_DEBUG << "Got duplicated path " << webpath;
+ BMCWEB_LOG_DEBUG << "Got duplicated path " << webpath.string();
continue;
}
const char* contentType = nullptr;
@@ -137,7 +137,7 @@
if (contentType == nullptr)
{
BMCWEB_LOG_ERROR << "Cannot determine content-type for "
- << absolutePath << " with extension "
+ << absolutePath.string() << " with extension "
<< extension;
}