You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by mi...@apache.org on 2011/12/18 18:21:37 UTC

svn commit: r1220460 - /httpd/httpd/trunk/modules/debugging/mod_firehose.c

Author: minfrin
Date: Sun Dec 18 17:21:36 2011
New Revision: 1220460

URL: http://svn.apache.org/viewvc?rev=1220460&view=rev
Log:
mod_firehose: Ensure directives are defined global only.

Modified:
    httpd/httpd/trunk/modules/debugging/mod_firehose.c

Modified: httpd/httpd/trunk/modules/debugging/mod_firehose.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/debugging/mod_firehose.c?rev=1220460&r1=1220459&r2=1220460&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/debugging/mod_firehose.c (original)
+++ httpd/httpd/trunk/modules/debugging/mod_firehose.c Sun Dec 18 17:21:36 2011
@@ -387,7 +387,7 @@ static int firehose_create_request(reque
     return OK;
 }
 
-/* TODO: Make sure the connection directives are enforced global only.
+/* Ideas for extension:
  *
  * TODO: An idea for configuration. Let the filename directives be per-directory,
  * with a global hashtable of filename to filehandle mappings. As each directive
@@ -588,7 +588,7 @@ static const char *firehose_enable_conne
         void *dummy, const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;
@@ -603,7 +603,7 @@ static const char *firehose_enable_conne
         void *dummy, const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;
@@ -618,7 +618,7 @@ static const char *firehose_enable_reque
         const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;
@@ -633,7 +633,7 @@ static const char *firehose_enable_reque
         const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;
@@ -648,7 +648,7 @@ static const char *firehose_enable_proxy
         void *dummy, const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;
@@ -663,7 +663,7 @@ static const char *firehose_enable_proxy
         void *dummy, const char *arg1, const char *arg2)
 {
 
-    const char *err = ap_check_cmd_context(cmd, NOT_IN_DIR_LOC_FILE
+    const char *err = ap_check_cmd_context(cmd, GLOBAL_ONLY
             | NOT_IN_LIMIT);
     if (err != NULL) {
         return err;