You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2015/06/17 04:12:30 UTC
trafficserver git commit: TS-3666: Handle chunking properly on error
Repository: trafficserver
Updated Branches:
refs/heads/master 1ce4abcac -> 8a2b6f62e
TS-3666: Handle chunking properly on error
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/8a2b6f62
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/8a2b6f62
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/8a2b6f62
Branch: refs/heads/master
Commit: 8a2b6f62ea6440be371f76b55b4f3566216ec646
Parents: 1ce4abc
Author: Michael Graham <mg...@bloxx.com>
Authored: Tue Jun 16 20:06:02 2015 -0600
Committer: Phil Sorber <so...@apache.org>
Committed: Tue Jun 16 20:07:24 2015 -0600
----------------------------------------------------------------------
proxy/http/HttpTransact.cc | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/8a2b6f62/proxy/http/HttpTransact.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index ca6338d..f925775 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -1466,12 +1466,13 @@ HttpTransact::HandleApiErrorJump(State *s)
// case the txn was reenabled in error by a plugin from hook SEND_RESPONSE_HDR.
// build_response doesn't clean the header. So clean it up before.
// Do fields_clear() instead of clear() to prevent memory leak
- // and set the source to internal so chunking is handled correctly
if (s->hdr_info.client_response.valid()) {
s->hdr_info.client_response.fields_clear();
- s->source = SOURCE_INTERNAL;
}
+ // Set the source to internal so chunking is handled correctly
+ s->source = SOURCE_INTERNAL;
+
/**
The API indicated an error. Lets use a >=400 error from the state (if one's set) or fallback to a
generic HTTP/1.X 500 INKApi Error