This is an automated email from the ASF dual-hosted git repository. bmahler pushed a commit to branch 1.5.x in repository https://gitbox.apache.org/repos/asf/mesos.git
commit fa8e09c66e42f5a94e4b0e2ad9b3bc9de4d2aa6c Author: Benjamin Mahler <[email protected]> AuthorDate: Mon Oct 21 19:57:54 2019 -0400 Logged failed TLS reverse DNS lookups as warnings for 'legacy' scheme. These were getting logged at VLOG(2), whereas we want all networking related errors to be logged as warnings (or errors if appropriate). Review: https://reviews.apache.org/r/71643 --- 3rdparty/libprocess/src/libevent_ssl_socket.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/3rdparty/libprocess/src/libevent_ssl_socket.cpp b/3rdparty/libprocess/src/libevent_ssl_socket.cpp index 405a5e4..d82fac1 100644 --- a/3rdparty/libprocess/src/libevent_ssl_socket.cpp +++ b/3rdparty/libprocess/src/libevent_ssl_socket.cpp @@ -1134,8 +1134,9 @@ void LibeventSSLSocketImpl::accept_SSL_callback(AcceptRequest* request) } if (hostname.isError()) { - VLOG(2) << "Could not determine hostname of peer: " - << hostname.error(); + LOG(WARNING) << "Reverse DNS lookup for" + << " '" << request->ip.get() << "'" + << " failed: " << hostname.error(); } else { VLOG(2) << "Accepting from " << hostname.get(); peer_hostname = hostname.get();
