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);