This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push: new abb5e58 Fixing 404 on redirects abb5e58 is described below commit abb5e581b97e00364611e4509bac5a95536f1747 Author: Zizhong Zhang <zizh...@linkedin.com> AuthorDate: Tue Mar 13 17:18:20 2018 -0700 Fixing 404 on redirects This is a bug introduced with PR #2335. The `if` is to avoid unnecessary works when buffering the post body, but since redirects also need to execute the logic flow and the request info may be changed during the process of redirects. We need to extract the request info every time redirects happen. --- proxy/http/HttpTransact.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc index 9af7900..6085b1b 100644 --- a/proxy/http/HttpTransact.cc +++ b/proxy/http/HttpTransact.cc @@ -1123,7 +1123,7 @@ HttpTransact::HandleRequest(State *s) { TxnDebug("http_trans", "START HttpTransact::HandleRequest"); - if (!s->request_data.hdr) { + if (!s->state_machine->is_waiting_for_full_body) { ink_assert(!s->hdr_info.server_request.valid()); HTTP_INCREMENT_DYN_STAT(http_incoming_requests_stat); -- To stop receiving notification emails like this one, please contact zw...@apache.org.