Add an intelligent error message should no proxy submodules be

valid to handle a request.
PR: 8407
Obtained from:
Submitted by:
Reviewed by:


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@94785 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 9 insertions, 3 deletions

diff --git a/CHANGES b/CHANGES
index 46b7726e9b..be75fd6f2f 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,8 @@
 Changes with Apache 2.0.36
 
+  *) Add an intelligent error message should no proxy submodules be
+     valid to handle a request. PR 8407 [Graham Leggett]
+
   *) Major improvements in concurrent processing for AB by enabling
      non-blocking connect()s and preventing APR from doing blocking
      read()s. Also implement fatal error checking for apr_recv().
@@ -23,6 +26,7 @@ Changes with Apache 2.0.36
   *) Fix perchild MPM so that it uses ap_gname2id for groups instead of
      ap_uname2id. [Scott Lamb <slamb@slamb.org>]
 
+>>>>>>> 1.728
   *) Fix AcceptPathInfo. PR 8234  [Cliff Woolley]
 
   *) [Security] Added the APLOG_TOCLIENT flag to ap_log_rerror() to
diff --git a/modules/proxy/mod_proxy.c b/modules/proxy/mod_proxy.c
index 9664ba1a35..6fb930f4d9 100644
--- a/modules/proxy/mod_proxy.c
+++ b/modules/proxy/mod_proxy.c
@@ -465,9 +465,11 @@ static int proxy_handler(request_rec *r)
                  "Trying to run scheme_handler");
     access_status = proxy_run_scheme_handler(r, conf, url, NULL, 0);
     if (DECLINED == access_status) {
-        ap_log_error(APLOG_MARK, APLOG_DEBUG | APLOG_NOERRNO, 0, r->server,
-                     "Neither CONNECT, HTTP or FTP for %s",
-                     r->uri);
+        ap_log_error(APLOG_MARK, APLOG_WARNING | APLOG_NOERRNO, 0, r->server,
+                    "proxy: No protocol handler was valid for the URL %s. "
+                    "If you are using a DSO version of mod_proxy, make sure "
+                    "the proxy submodules are included in the configuration "
+                    "using LoadModule.", r->uri);
         return HTTP_FORBIDDEN;
     }
     return access_status;