This is an automated email from the ASF dual-hosted git repository.

bmahler pushed a commit to branch 1.8.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 0ce4e87a8137d95ce73122cbce3d6c9e2018f621
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/posix/libevent/libevent_ssl_socket.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/3rdparty/libprocess/src/posix/libevent/libevent_ssl_socket.cpp 
b/3rdparty/libprocess/src/posix/libevent/libevent_ssl_socket.cpp
index 29a1bf7..9f37301 100644
--- a/3rdparty/libprocess/src/posix/libevent/libevent_ssl_socket.cpp
+++ b/3rdparty/libprocess/src/posix/libevent/libevent_ssl_socket.cpp
@@ -1172,8 +1172,8 @@ 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 << "'"
+                           << " failed: " << hostname.error();
             } else {
               VLOG(2) << "Accepting from " << hostname.get();
               peer_hostname = hostname.get();

Reply via email to