You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jf...@apache.org on 2011/09/08 14:41:31 UTC

svn commit: r1166657 - /httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c

Author: jfclere
Date: Thu Sep  8 12:41:30 2011
New Revision: 1166657

URL: http://svn.apache.org/viewvc?rev=1166657&view=rev
Log:
Use HTTP_NOT_IMPLEMENTED instead of HTTP_BAD_REQUEST.

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

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c?rev=1166657&r1=1166656&r2=1166657&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c Thu Sep  8 12:41:30 2011
@@ -214,9 +214,11 @@ static int ap_proxy_ajp_request(apr_pool
                      "proxy: AJP: request failed to %pI (%s)",
                      conn->worker->cp->addr,
                      conn->worker->s->hostname);
-        if (status == AJP_EOVERFLOW || status == AJP_EBAD_METHOD)
+        if (status == AJP_EOVERFLOW)
             return HTTP_BAD_REQUEST;
-        else {
+        else if  (status == AJP_EBAD_METHOD) {
+            return HTTP_NOT_IMPLEMENTED;
+        } else {
             /*
              * This is only non fatal when the method is idempotent. In this
              * case we can dare to retry it with a different worker if we are