]> granicus.if.org Git - apache/commitdiff
too-long worker schemes and/or hostnames are no longer fatal errors
authorJim Jagielski <jim@apache.org>
Wed, 7 Feb 2018 15:38:23 +0000 (15:38 +0000)
committerJim Jagielski <jim@apache.org>
Wed, 7 Feb 2018 15:38:23 +0000 (15:38 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1823482 13f79535-47bb-0310-9956-ffa450edef68

CHANGES
docs/log-message-tags/next-number
modules/proxy/proxy_util.c

diff --git a/CHANGES b/CHANGES
index bdd3073ea66f850253ba76ba93c5dcd82ed1d7d6..34f8404d1082cb8df17b80704c52a49cd8f41bc2 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.5.1
 
+  *) mod_proxy: Worker schemes and hostnames which are too large are no
+     longer fatal errors; it is logged and the truncated values are stored.
+
   *) core: adding AP_DECLARE for ap_parse_vhost_addrs() and minor bumb mmn. Resolves
      building mod_ssl on Windows. [Stefan Eissing, Gregg Smith]
 
index ddbc9f7456f94f6b3f7f8d8277920b0e41f93033..da690282fcb4025f6ceae3ec5ec925c84a4e2fb7 100644 (file)
@@ -1 +1 @@
-10117
+10119
index ee20176cd3dab5a3913cffeb715475118ab37f5b..c006fc7ec1b515ce41a950a7d5ec7d53af962fb0 100644 (file)
@@ -1724,10 +1724,12 @@ PROXY_DECLARE(char *) ap_proxy_define_worker(apr_pool_t *p,
         "Alert! worker name (%s) too long; truncated to: %s", ptr, wshared->name);
     }
     if (PROXY_STRNCPY(wshared->scheme, uri.scheme) != APR_SUCCESS) {
-        return apr_psprintf(p, "worker scheme (%s) too long", uri.scheme);
+        ap_log_error(APLOG_MARK, APLOG_ERR, 0, ap_server_conf, APLOGNO(010117)
+        "Alert! worker scheme (%s) too long; truncated to: %s", uri.scheme, wshared->scheme);
     }
     if (PROXY_STRNCPY(wshared->hostname, uri.hostname) != APR_SUCCESS) {
-        return apr_psprintf(p, "worker hostname (%s) too long", uri.hostname);
+        ap_log_error(APLOG_MARK, APLOG_ERR, 0, ap_server_conf, APLOGNO(010118)
+        "Alert! worker hostname (%s) too long; truncated to: %s", uri.hostname, wshared->hostname);
     }
     wshared->flush_packets = flush_off;
     wshared->flush_wait = PROXY_FLUSH_WAIT;