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 2010/08/27 19:07:51 UTC

svn commit: r990210 - /trafficserver/traffic/trunk/proxy/http2/HttpSM.cc

Author: zwoop
Date: Fri Aug 27 17:07:51 2010
New Revision: 990210

URL: http://svn.apache.org/viewvc?rev=990210&view=rev
Log:
TS-414 INKHttpTxnPristineUrlGet() returns NULL in reverse proxy mode

Author: Wendy Huang
Review: leif
Tested: FC13 / 64-bit

Modified:
    trafficserver/traffic/trunk/proxy/http2/HttpSM.cc

Modified: trafficserver/traffic/trunk/proxy/http2/HttpSM.cc
URL: http://svn.apache.org/viewvc/trafficserver/traffic/trunk/proxy/http2/HttpSM.cc?rev=990210&r1=990209&r2=990210&view=diff
==============================================================================
--- trafficserver/traffic/trunk/proxy/http2/HttpSM.cc (original)
+++ trafficserver/traffic/trunk/proxy/http2/HttpSM.cc Fri Aug 27 17:07:51 2010
@@ -843,11 +843,6 @@ HttpSM::state_read_client_request_header
     if (HttpConfig::m_master.number_of_redirections)
       enable_redirection = HttpConfig::m_master.redirection_enabled;
 
-    // Preserve pristine url before remap
-    t_state.pristine_url.create(t_state.hdr_info.client_request.url_get()->m_heap);
-    t_state.pristine_url.copy(t_state.hdr_info.client_request.url_get());
-
-
     call_transact_and_set_next_state(HttpTransact::ModifyRequest);
 
     break;
@@ -3838,6 +3833,10 @@ HttpSM::do_remap_request(bool run_inline
     return;
   }
 
+  // Preserve pristine url before remap
+  t_state.pristine_url.create(t_state.hdr_info.client_request.url_get()->m_heap);
+  t_state.pristine_url.copy(t_state.hdr_info.client_request.url_get());
+
   Debug("url_rewrite", "Found a remap map entry for [%lld], attempting to remap request and call any plugins", sm_id);
   Action *remap_action_handle = remapProcessor.perform_remap(this, &t_state);