On 08/02/11 11:46, Erik Trauschke wrote:
On Tue, 2011-08-02 at 11:28 -0700, Shawn Walker wrote:
On 08/02/11 11:23, Erik Trauschke wrote:
On Tue, 2011-08-02 at 11:15 -0700, Shawn Walker wrote:
On 08/02/11 10:59, Erik Trauschke wrote:
Bug 18769 - apache proxy conf needs another update

This is just an update of the example configuration.

I'd add a comment to the file noting that the sample configuration
requires Apache 2.2.18 or newer.

Ok, I can do that, but since we are shipping a recent enough apache
version it works out of the box. We would have to check for a minimum
version of apache for every option we use when we want to make sure that
it works for everybody's custom environment.

Yes, but it's still helpful, especially if they're on Solaris 10 or some
other os and they're just using this is an example config.

Alright, convinced.
--- a/src/util/apache2/proxy-example.conf       Fri Jul 29 13:15:39 2011 -0700
+++ b/src/util/apache2/proxy-example.conf       Tue Aug 02 11:45:21 2011 -0700
@@ -70,9 +70,10 @@
  #
  # Packages may contain URL encoded forward slashes. To make sure these
are
  # not interpreted as directory delimiters Apache can be instructed to
just
-# leave them as they are.
+# leave them as they are. For the "NoDecode" option to work Apache
2.2.18
+# or later is required.
  #
-AllowEncodedSlashes On
+AllowEncodedSlashes NoDecode

  #
  # Crank up MaxKeepAliveRequests so that clients can make a large number

LGTM.

-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to