You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by kb...@apache.org on 2011/12/24 07:22:49 UTC

svn commit: r1222915 - /httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c

Author: kbrand
Date: Sat Dec 24 06:22:49 2011
New Revision: 1222915

URL: http://svn.apache.org/viewvc?rev=1222915&view=rev
Log:
REMOTE_ADDR is now r->useragent_ip, which is not/never available at
SSL_CB_HANDSHAKE_DONE time. ap_log_cerror() already logs c->client_ip
for us, however, so we can simply drop REMOTE_ADDR from the message.

Modified:
    httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c

Modified: httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c?rev=1222915&r1=1222914&r2=1222915&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c (original)
+++ httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c Sat Dec 24 06:22:49 2011
@@ -1879,9 +1879,7 @@ static void log_tracing_state(const SSL 
      */
     if (where & SSL_CB_HANDSHAKE_DONE) {
         ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, c, APLOGNO(02041)
-                      "Connection: Client IP: %s, Protocol: %s, "
-                      "Cipher: %s (%s/%s bits)",
-                      ssl_var_lookup(NULL, s, c, NULL, "REMOTE_ADDR"),
+                      "Protocol: %s, Cipher: %s (%s/%s bits)",
                       ssl_var_lookup(NULL, s, c, NULL, "SSL_PROTOCOL"),
                       ssl_var_lookup(NULL, s, c, NULL, "SSL_CIPHER"),
                       ssl_var_lookup(NULL, s, c, NULL, "SSL_CIPHER_USEKEYSIZE"),