httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r1697339 - in /httpd/httpd/trunk: docs/manual/mod/core.xml modules/http2/README.h2 server/core.c
Date Mon, 24 Aug 2015 09:26:18 GMT
Author: icing
Date: Mon Aug 24 09:26:17 2015
New Revision: 1697339

changed Protocols to let vhosts override servers, removed old H2Engine example from readme


Modified: httpd/httpd/trunk/docs/manual/mod/core.xml
--- httpd/httpd/trunk/docs/manual/mod/core.xml (original)
+++ httpd/httpd/trunk/docs/manual/mod/core.xml Mon Aug 24 09:26:17 2015
@@ -3740,11 +3740,11 @@ Protocol https
         <p>It is safe to specify protocols that are unavailable/disabled. Such
         protocol names will simply be ignored.</p>
-        <p>Protocols specified in base servers and virtual hosts are concatenated
-            by appending the base ones, if there are configured protocols,
-            to the virtual host ones. Since protocols such as HTTP/2 allow
-            connection reuse under certain conditions, restricting protocols for
-            individual virtual hosts might not work as you expect it to.</p>
+        <p>Protocols specified in base servers are inherited for virtual hosts 
+            only if the virtual host has no own Protocols directive. Or, the other
+            way around, Protocols directives in virtual hosts replace any
+            such directive in the base server.
+        </p>
     <seealso><directive module="core">ProtocolsHonorOrder</directive></seealso>

Modified: httpd/httpd/trunk/modules/http2/README.h2
--- httpd/httpd/trunk/modules/http2/README.h2 (original)
+++ httpd/httpd/trunk/modules/http2/README.h2 Mon Aug 24 09:26:17 2015
@@ -34,13 +34,26 @@ shared cousins.
-The most important configuration options is
+If mod_h2 is enabled for a site or not depends on the new "Protocols"
+directive. This directive list all protocols enabled for a server or
+virtual host.
+If you do not specify "Protocols" all available protocols are enabled. For
+sites using TLS, the protocol supported by mod_h2 is "h2". For cleartext
+http:, the offered protocol is "h2c".
+The following is an example of a server that only supports http/1.1 in
+general and offers h2 for a specific virtual host.
+    ...
+    Protocols http/1.1
+    <virtualhost *:443>
+        Protocols h2 http/1.1
+        ...
+    </virtualhost>
-    H2Engine On|Off
-which can be set on the base server or a virtual host. By default, the
-engine is 'Off'. Please see the documentation of mod_h2 for a complete
-list and explanation of other options.
+Please see the documentation of mod_h2 for a complete list and explanation 
+of all options.

Modified: httpd/httpd/trunk/server/core.c
--- httpd/httpd/trunk/server/core.c (original)
+++ httpd/httpd/trunk/server/core.c Mon Aug 24 09:26:17 2015
@@ -554,7 +554,8 @@ static void *merge_core_server_configs(a
                            ? virt->merge_trailers
                            : base->merge_trailers;
-    conf->protocols = apr_array_append(p, virt->protocols, base->protocols);
+    conf->protocols = ((virt->protocols->nelts > 0)? 
+                       virt->protocols : base->protocols);
     conf->protocols_honor_order = ((virt->protocols_honor_order < 0)?
                                        base->protocols_honor_order :

View raw message