Mailing List Archive

Re: svn commit: r1908300 - in /httpd/test/framework/trunk/t: conf/extra.conf.in modules/rewrite.t
On 3/11/23 10:31 PM, covener@apache.org wrote:
> Author: covener
> Date: Sat Mar 11 21:31:14 2023
> New Revision: 1908300
>
> URL: http://svn.apache.org/viewvc?rev=1908300&view=rev
> Log:
> test [P] flag
>
> Modified:
> httpd/test/framework/trunk/t/conf/extra.conf.in
> httpd/test/framework/trunk/t/modules/rewrite.t
>
> Modified: httpd/test/framework/trunk/t/conf/extra.conf.in
> URL: http://svn.apache.org/viewvc/httpd/test/framework/trunk/t/conf/extra.conf.in?rev=1908300&r1=1908299&r2=1908300&view=diff
> ==============================================================================
> --- httpd/test/framework/trunk/t/conf/extra.conf.in (original)
> +++ httpd/test/framework/trunk/t/conf/extra.conf.in Sat Mar 11 21:31:14 2023
> @@ -272,10 +272,14 @@
> RewriteRule ^/modules/rewrite/escaping/local/(.*) /?$1
> RewriteRule ^/modules/rewrite/escaping/redir/(.*) http://@SERVERNAME@:@PORT@/?$1 [R]
> RewriteRule ^/modules/rewrite/escaping/redir_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [R,NE]
> + RewriteRule ^/modules/rewrite/escaping/proxy/(.*) http://@SERVERNAME@:@PORT@/?$1 [P]
> + RewriteRule ^/modules/rewrite/escaping/proxy_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [P]

Shouldn't this be [P,NE] instead?

> <LocationMatch ^/modules/rewrite/escaping/fixups/>
> RewriteRule local/(.*) /?$1
> RewriteRule redir/(.*) http://@SERVERNAME@:@PORT@/?$1 [R]
> RewriteRule redir_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [R,NE]
> + RewriteRule proxy/(.*) http://@SERVERNAME@:@PORT@/?$1 [P]
> + RewriteRule proxy_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [P,NE]
> </LocationMatch>
>
> <VirtualHost cve_2011_3368_rewrite>
>

Regards

RĂ¼diger
Re: svn commit: r1908300 - in /httpd/test/framework/trunk/t: conf/extra.conf.in modules/rewrite.t [ In reply to ]
On Mon, Mar 13, 2023 at 3:26?AM Ruediger Pluem <rpluem@apache.org> wrote:
>
>
>
> On 3/11/23 10:31 PM, covener@apache.org wrote:
> > Author: covener
> > Date: Sat Mar 11 21:31:14 2023
> > New Revision: 1908300
> >
> > URL: http://svn.apache.org/viewvc?rev=1908300&view=rev
> > Log:
> > test [P] flag
> >
> > Modified:
> > httpd/test/framework/trunk/t/conf/extra.conf.in
> > httpd/test/framework/trunk/t/modules/rewrite.t
> >
> > Modified: httpd/test/framework/trunk/t/conf/extra.conf.in
> > URL: http://svn.apache.org/viewvc/httpd/test/framework/trunk/t/conf/extra.conf.in?rev=1908300&r1=1908299&r2=1908300&view=diff
> > ==============================================================================
> > --- httpd/test/framework/trunk/t/conf/extra.conf.in (original)
> > +++ httpd/test/framework/trunk/t/conf/extra.conf.in Sat Mar 11 21:31:14 2023
> > @@ -272,10 +272,14 @@
> > RewriteRule ^/modules/rewrite/escaping/local/(.*) /?$1
> > RewriteRule ^/modules/rewrite/escaping/redir/(.*) http://@SERVERNAME@:@PORT@/?$1 [R]
> > RewriteRule ^/modules/rewrite/escaping/redir_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [R,NE]
> > + RewriteRule ^/modules/rewrite/escaping/proxy/(.*) http://@SERVERNAME@:@PORT@/?$1 [P]
> > + RewriteRule ^/modules/rewrite/escaping/proxy_ne/(.*) http://@SERVERNAME@:@PORT@/?$1 [P]
>
> Shouldn't this be [P,NE] instead?

Thank you, fixed.