You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by co...@apache.org on 2019/01/23 18:25:50 UTC

svn commit: r1851954 - /httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c

Author: covener
Date: Wed Jan 23 18:25:50 2019
New Revision: 1851954

URL: http://svn.apache.org/viewvc?rev=1851954&view=rev
Log:
revert of r808895 somehow missed mod_proxy_ftp

r808895 added `ServerTokens OFF` so some modules were
switched from ap_get_server_banner() to 
ap_get_server_description() so they would never be empty.  

But ap_get_server_description() is documnented as being for local 
use only, so it was not really a suitable replacement much less 
w/ `ServerTokens OFF` subsequently taken out.



Modified:
    httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c?rev=1851954&r1=1851953&r2=1851954&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_ftp.c Wed Jan 23 18:25:50 2019
@@ -1954,7 +1954,7 @@ static int proxy_ftp_handler(request_rec
 
     apr_rfc822_date(dates, r->request_time);
     apr_table_setn(r->headers_out, "Date", dates);
-    apr_table_setn(r->headers_out, "Server", ap_get_server_description());
+    apr_table_setn(r->headers_out, "Server", ap_get_server_banner());
 
     /* set content-type */
     if (dirlisting) {