Repository: trafficserver Updated Branches: refs/heads/5.0.x 9a9d9e20b -> 04d883ed5
TS-2881: Redirection handling broken Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/04d883ed Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/04d883ed Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/04d883ed Branch: refs/heads/5.0.x Commit: 04d883ed5d92aa3e19ee8f704e0cef1ab36935ad Parents: 9a9d9e2 Author: Sudheer Vinukonda <[email protected]> Authored: Mon Jun 9 10:43:40 2014 -0700 Committer: Bryan Call <[email protected]> Committed: Mon Jun 9 10:56:31 2014 -0700 ---------------------------------------------------------------------- CHANGES | 2 ++ proxy/http/HttpSM.cc | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/trafficserver/blob/04d883ed/CHANGES ---------------------------------------------------------------------- diff --git a/CHANGES b/CHANGES index 98805b3..ab2d742 100644 --- a/CHANGES +++ b/CHANGES @@ -1,6 +1,8 @@ -*- coding: utf-8 -*- Changes with Apache Traffic Server 5.0.0 + *) [TS-2881] Redirection handling broken + *) [TS-2875] Change hwloc library env variables from hwloc_ to HWLOC_. *) [TS-2874] Make some of the SPDY metrics persistent, and also fix how we http://git-wip-us.apache.org/repos/asf/trafficserver/blob/04d883ed/proxy/http/HttpSM.cc ---------------------------------------------------------------------- diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc index 858499f..1d455cd 100644 --- a/proxy/http/HttpSM.cc +++ b/proxy/http/HttpSM.cc @@ -7588,6 +7588,8 @@ HttpSM::is_redirect_required() { bool redirect_required = (enable_redirection && (redirection_tries <= HttpConfig::m_master.number_of_redirections)); + DebugSM("http_redirect", "is_redirect_required %u", redirect_required); + if (redirect_required == true) { HTTPStatus status = t_state.hdr_info.client_response.status_get(); // check to see if the response from the orgin was a 301, 302, or 303 @@ -7609,8 +7611,6 @@ HttpSM::is_redirect_required() // if redirect_url is set by an user's plugin, ats will redirect to this url anyway. if (redirect_url != NULL) { redirect_required = true; - } else { - redirect_required = false; } } return redirect_required;
