You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jo...@apache.org on 2012/07/24 17:06:25 UTC

svn commit: r1365109 - in /httpd/test/framework/trunk/t: conf/extra.conf.in conf/proxy.conf.in modules/proxy.t

Author: jorton
Date: Tue Jul 24 15:06:25 2012
New Revision: 1365109

URL: http://svn.apache.org/viewvc?rev=1365109&view=rev
Log:
- add test case for PR 45434, seems to be a 2.2->2.4 regression?

Modified:
    httpd/test/framework/trunk/t/conf/extra.conf.in
    httpd/test/framework/trunk/t/conf/proxy.conf.in
    httpd/test/framework/trunk/t/modules/proxy.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=1365109&r1=1365108&r2=1365109&view=diff
==============================================================================
--- httpd/test/framework/trunk/t/conf/extra.conf.in (original)
+++ httpd/test/framework/trunk/t/conf/extra.conf.in Tue Jul 24 15:06:25 2012
@@ -322,6 +322,8 @@
     RedirectTemp /temp2 http://@SERVERNAME@:@PORT@/alias/index.html
     RedirectPermanent /perm2 http://@SERVERNAME@:@PORT@/alias/index.html
 
+    Redirect permanent /modules/alias/redirect-me http://@SERVERNAME@:@PORT@/modules/alias/5.html
+
     ScriptAlias /cgi @SERVERROOT@/htdocs/modules/alias
     ScriptAliasMatch /aliascgi-(.*) @SERVERROOT@/htdocs/modules/alias/$1
 </IfModule>

Modified: httpd/test/framework/trunk/t/conf/proxy.conf.in
URL: http://svn.apache.org/viewvc/httpd/test/framework/trunk/t/conf/proxy.conf.in?rev=1365109&r1=1365108&r2=1365109&view=diff
==============================================================================
--- httpd/test/framework/trunk/t/conf/proxy.conf.in (original)
+++ httpd/test/framework/trunk/t/conf/proxy.conf.in Tue Jul 24 15:06:25 2012
@@ -30,7 +30,16 @@
        ProxyPass balancer://foo/
      </Location>
 
+     ## PR 45434 tests
+     <Proxy balancer://pr45434>
+       BalancerMember http://@SERVERNAME@:@PORT@/modules
+     </Proxy> 
+     
+     ProxyPass /pr45434 balancer://pr45434/alias
+     ProxyPassReverse /pr45434 balancer://pr45434/alias
+
    </VirtualHost>
+
 </IfModule>
 
 <IfModule mod_rewrite.c>

Modified: httpd/test/framework/trunk/t/modules/proxy.t
URL: http://svn.apache.org/viewvc/httpd/test/framework/trunk/t/modules/proxy.t?rev=1365109&r1=1365108&r2=1365109&view=diff
==============================================================================
--- httpd/test/framework/trunk/t/modules/proxy.t (original)
+++ httpd/test/framework/trunk/t/modules/proxy.t Tue Jul 24 15:06:25 2012
@@ -6,7 +6,7 @@ use Apache::TestRequest;
 use Apache::TestUtil;
 use Apache::TestConfig ();
 
-plan tests => 15, need_module 'proxy';
+plan tests => 17, need_module 'proxy';
 
 Apache::TestRequest::module("proxy_http_reverse");
 Apache::TestRequest::user_agent(requests_redirectable => 0);
@@ -65,7 +65,16 @@ if (have_module('alias')) {
     $r = GET("/reverse/perm");
     ok t_cmp($r->code, 301, "reverse proxy of redirect");
     ok t_cmp($r->header("Location"), qr{http://[^/]*/reverse/alias}, "reverse proxy rewrote redirect");
+
+    # More complex reverse mapping case with the balancer, PR 45434
+    Apache::TestRequest::module("proxy_http_balancer");
+    my $hostport = Apache::TestRequest::hostport();
+    $r = GET("/pr45434/redirect-me");
+    ok t_cmp($r->code, 301, "reverse proxy of redirect via balancer");
+    ok t_cmp($r->header("Location"), "http://$hostport/pr45434/5.html", "reverse proxy via balancer rewrote redirect");
+    Apache::TestRequest::module("proxy_http_reverse"); # flip back 
+
 } else {
-    skip "skipping tests without mod_alias" foreach (1..2);
+    skip "skipping tests without mod_alias" foreach (1..4);
 }