You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bc...@apache.org on 2022/06/14 15:07:57 UTC

[trafficserver] branch 9.1.x updated: Do not modify Transfer-Encoding header on retry (#8899)

This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch 9.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.1.x by this push:
     new 7a2eed0d1 Do not modify Transfer-Encoding header on retry (#8899)
7a2eed0d1 is described below

commit 7a2eed0d13647ceb4ebbc167384e8607e1571b9f
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Tue Jun 14 13:34:53 2022 +0900

    Do not modify Transfer-Encoding header on retry (#8899)
    
    (cherry picked from commit f45d490b7c3a3cb91cbc6a815b9939b19101e4d2)
---
 proxy/http/HttpSM.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 0416edccc..e67ceb5c3 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -6193,7 +6193,8 @@ HttpSM::attach_server_session(PoolableSession *s)
   if (ua_txn->has_request_body(t_state.hdr_info.request_content_length,
                                t_state.client_info.transfer_encoding == HttpTransact::CHUNKED_ENCODING)) {
     // See if we need to insert a chunked header
-    if (!t_state.hdr_info.server_request.presence(MIME_PRESENCE_CONTENT_LENGTH)) {
+    if (!t_state.hdr_info.server_request.presence(MIME_PRESENCE_CONTENT_LENGTH) &&
+        !t_state.hdr_info.server_request.presence(MIME_PRESENCE_TRANSFER_ENCODING)) {
       // Stuff in a TE setting so we treat this as chunked, sort of.
       t_state.server_info.transfer_encoding = HttpTransact::CHUNKED_ENCODING;
       t_state.hdr_info.server_request.value_append(MIME_FIELD_TRANSFER_ENCODING, MIME_LEN_TRANSFER_ENCODING, HTTP_VALUE_CHUNKED,