From beb29a300b500ff575641cbfc4839f794fbe93be Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vladim=C3=ADr=20Chlup?= <vchlup@redhat.com>
Date: Wed, 11 Dec 2024 09:57:20 +0100
Subject: [PATCH 1/2] Require the directive from the documentation

---
 native/mod_manager/mod_manager.c | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/native/mod_manager/mod_manager.c b/native/mod_manager/mod_manager.c
index 4ebb3fa3..fa90b554 100644
--- a/native/mod_manager/mod_manager.c
+++ b/native/mod_manager/mod_manager.c
@@ -3405,11 +3405,24 @@ static const char*cmd_manager_maxmesssize(cmd_parms *cmd, void *mconfig, const c
 static const char*cmd_manager_enable_mcmp_receive(cmd_parms *cmd, void *dummy)
 {
     mod_manager_config *mconf = ap_get_module_config(cmd->server->module_config, &manager_module);
-    if (!cmd->server->is_virtual)
-        return "EnableMCPMReceive must be in a VirtualHost";
-    mconf->enable_mcmp_receive = -1;
+    ap_directive_t *directive = cmd->directive->parent ? cmd->directive->parent->first_child : cmd->directive;
+    (void)dummy;
+
+    if (!cmd->server->is_virtual) {
+        return "EnableMCMPReceive must be in a VirtualHost";
+    }
+
+    while (directive) {
+        if (strcmp(directive->directive, "<Directory") == 0) {
+            return "Directory cannot be used with EnableMCMPReceive, use Location instead";
+        }
+        directive = directive->next;
+    }
+
+    mconf->enable_mcmp_receive = 1;
     return NULL;
 }
+
 static const char*cmd_manager_enable_mcmp_receive_deprecated(cmd_parms *cmd, void *dummy)
 {
     ap_log_error(APLOG_MARK, APLOG_WARNING, 0, NULL,

From 755088da204dedf795144664db803fcabcad7780 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Vladim=C3=ADr=20Chlup?= <vchlup@redhat.com>
Date: Thu, 12 Dec 2024 13:33:36 +0100
Subject: [PATCH 2/2] Use the same directive as the documentation

---
 docs/userguide/src/main/docbook/modules/faq.xml        | 10 +++++-----
 .../userguide/src/main/docbook/modules/quick-start.xml |  8 ++++----
 test/java/README.txt                                   |  4 ++--
 test/java/installhttpd.sh                              |  8 ++++----
 4 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/docs/userguide/src/main/docbook/modules/faq.xml b/docs/userguide/src/main/docbook/modules/faq.xml
index 11fbe8a9..7bfd27ef 100644
--- a/docs/userguide/src/main/docbook/modules/faq.xml
+++ b/docs/userguide/src/main/docbook/modules/faq.xml
@@ -166,18 +166,18 @@ GET /
         <listitem>
           <para>"client denied by server configuration":</para>
           <para>
-            The directory in the VirtualHost is not allowed for the client.
+            The location in the VirtualHost is not allowed for the client.
             If you have something like:
           </para>
           <programlisting>Mon Jun 28 18:08:47 2010] [error] [client 10.33.144.3] client denied by server configuration: /</programlisting>
           <para>
             You need to have something like:
           </para>
-          <programlisting><![CDATA[<Directory />
+          <programlisting><![CDATA[<Location />
   Order deny,allow
   Deny from all
   Allow from 10.33.144.3
-</Directory>]]></programlisting>
+</Location>]]></programlisting>
         </listitem>
       </orderedlist>
     </section>
@@ -191,11 +191,11 @@ GET /
     </para>
     <programlisting><![CDATA[
 <VirtualHost localhost:6666>
-    <Directory />
+    <Location />
      Order deny,allow
      Deny from all
      Allow from 127.0.0.1
-    </Directory>
+    </Location>
     ServerAdvertise on http://localhost:6666
     EnableMCPMReceive
 </VirtualHost>
diff --git a/docs/userguide/src/main/docbook/modules/quick-start.xml b/docs/userguide/src/main/docbook/modules/quick-start.xml
index 67e43205..d1bf55a3 100644
--- a/docs/userguide/src/main/docbook/modules/quick-start.xml
+++ b/docs/userguide/src/main/docbook/modules/quick-start.xml
@@ -122,11 +122,11 @@ LoadModule advertise_module /opt/jboss/httpd/lib/httpd/modules/mod_advertise.so
 Listen 10.33.144.3:6666
 <VirtualHost 10.33.144.3:6666>
  
-  <Directory />
+  <Location />
     Order deny,allow
     Deny from all
     Allow from 10.33.144.
-  </Directory>
+  </Location>
  
   KeepAliveTimeout 60
   MaxKeepAliveRequests 0
@@ -149,11 +149,11 @@ LoadModule advertise_module modules/mod_advertise.so
 Listen 10.33.144.3:6666
 <VirtualHost 10.33.144.3:6666>
  
-  <Directory />
+  <Location />
     Order deny,allow
     Deny from all
     Allow from 10.33.144.
-  </Directory>
+  </Location>
  
   KeepAliveTimeout 60
   MaxKeepAliveRequests 0
diff --git a/test/java/README.txt b/test/java/README.txt
index 1cbd25e8..627dd4f3 100644
--- a/test/java/README.txt
+++ b/test/java/README.txt
@@ -17,11 +17,11 @@ NOTE the httpd should have something like the following in httpd.conf
 <IfModule manager_module>
    Listen jfcpc:6666
    <VirtualHost jfcpc:6666>
-    <Directory />
+    <Location />
       Order deny,allow
       Deny from all
       Allow from 10.33.144
-    </Directory>
+    </Location>
 
    KeepAliveTimeout 300
    MaxKeepAliveRequests 0
diff --git a/test/java/installhttpd.sh b/test/java/installhttpd.sh
index ed2e01b5..86c79216 100644
--- a/test/java/installhttpd.sh
+++ b/test/java/installhttpd.sh
@@ -314,11 +314,11 @@ then
   Listen @IP@:6666
   ManagerBalancerName mycluster
   <VirtualHost @IP@:6666>
-    <Directory />
+    <Location />
      Order deny,allow
      Deny from all
      Allow from @SUBIP@
-    </Directory>
+    </Location>
 
     KeepAliveTimeout 300
     MaxKeepAliveRequests 0
@@ -343,9 +343,9 @@ EOF
   Listen @IP@:6666
   ManagerBalancerName mycluster
   <VirtualHost @IP@:6666>
-    <Directory />
+    <Location />
      Require ip @SUBIP@
-    </Directory>
+    </Location>
 
     KeepAliveTimeout 300
     MaxKeepAliveRequests 0