[trafficserver] branch master updated: Fixing 404 on redirects

[zw...@apache.org]

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 <zi...@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
zwoop@apache.org.