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/http/http_client.hpp b/http/http_client.hpp
index 5f352d3..1664d01 100644
--- a/http/http_client.hpp
+++ b/http/http_client.hpp
@@ -120,13 +120,15 @@
const boost::asio::ip::tcp::endpoint& endpoint) {
if (ec)
{
- BMCWEB_LOG_ERROR << "Connect " << endpoint
+ BMCWEB_LOG_ERROR << "Connect "
+ << endpoint.address().to_string()
<< " failed: " << ec.message();
self->state = ConnState::connectFailed;
self->handleConnState();
return;
}
- BMCWEB_LOG_DEBUG << "Connected to: " << endpoint;
+ BMCWEB_LOG_DEBUG << "Connected to: "
+ << endpoint.address().to_string();
self->state = ConnState::connected;
self->handleConnState();
});
@@ -186,7 +188,7 @@
BMCWEB_LOG_DEBUG << "recvMessage() bytes transferred: "
<< bytesTransferred;
BMCWEB_LOG_DEBUG << "recvMessage() data: "
- << self->parser->get();
+ << self->parser->get().body();
unsigned int respCode = self->parser->get().result_int();
BMCWEB_LOG_DEBUG << "recvMessage() Header Response Code: "