You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by fi...@apache.org on 2005/10/18 23:34:43 UTC

svn commit: r326256 - in /httpd/httpd/branches/1.3.x/src: CHANGES modules/standard/mod_cgi.c

Author: fielding
Date: Tue Oct 18 14:34:41 2005
New Revision: 326256

URL: http://svn.apache.org/viewcvs?rev=326256&view=rev
Log:
Remove CGI block on OPTIONS method so that scripts can
respond to OPTIONS directly rather than via server default.

PR: 15242
Reviewed-by: Paul Querna, Andre Malo, William A. Rowe, Jr.


Modified:
    httpd/httpd/branches/1.3.x/src/CHANGES
    httpd/httpd/branches/1.3.x/src/modules/standard/mod_cgi.c

Modified: httpd/httpd/branches/1.3.x/src/CHANGES
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/1.3.x/src/CHANGES?rev=326256&r1=326255&r2=326256&view=diff
==============================================================================
--- httpd/httpd/branches/1.3.x/src/CHANGES (original)
+++ httpd/httpd/branches/1.3.x/src/CHANGES Tue Oct 18 14:34:41 2005
@@ -1,5 +1,9 @@
 Changes with Apache 1.3.35
 
+  *) mod_cgi: Remove block on OPTIONS method so that scripts can
+     respond to OPTIONS directly rather than via server default.
+     [Roy Fielding] PR 15242
+
 Changes with Apache 1.3.34
 
   *) hsregex: fix potential core dumping on 64 bit machines, such as

Modified: httpd/httpd/branches/1.3.x/src/modules/standard/mod_cgi.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/1.3.x/src/modules/standard/mod_cgi.c?rev=326256&r1=326255&r2=326256&view=diff
==============================================================================
--- httpd/httpd/branches/1.3.x/src/modules/standard/mod_cgi.c (original)
+++ httpd/httpd/branches/1.3.x/src/modules/standard/mod_cgi.c Tue Oct 18 14:34:41 2005
@@ -337,13 +337,6 @@
 
     struct cgi_child_stuff cld;
 
-    if (r->method_number == M_OPTIONS) {
-	/* 99 out of 100 CGI scripts, this is all they support */
-	r->allowed |= (1 << M_GET);
-	r->allowed |= (1 << M_POST);
-	return DECLINED;
-    }
-
     if ((argv0 = strrchr(r->filename, '/')) != NULL)
 	argv0++;
     else