You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@trafficserver.apache.org by zw...@apache.org on 2011/03/17 18:40:05 UTC

svn commit: r1082602 - /trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc

Author: zwoop
Date: Thu Mar 17 17:40:05 2011
New Revision: 1082602

URL: http://svn.apache.org/viewvc?rev=1082602&view=rev
Log:
TS-588 disable "homepageredirect" effectively, but this fixes stats pages.

Modified:
    trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc

Modified: trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc?rev=1082602&r1=1082601&r2=1082602&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc (original)
+++ trafficserver/traffic/trunk/proxy/http/remap/RemapPlugins.cc Thu Mar 17 17:40:05 2011
@@ -234,7 +234,9 @@ RemapPlugins::run_single_remap()
 
     _request_url->path_set(newPath, newPathLen);
 
-    if (map->homePageRedirect && fromPathLen == requestPathLen) {
+    // TODO: This is horribly wrong and broken, when can this trigger??? Check
+    // above, we already return on _s->remap_redirect ... 
+    if (map->homePageRedirect && fromPathLen == requestPathLen && _s->remap_redirect) {
       URL redirect_url;
 
       redirect_url.create(NULL);