You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2022/08/10 17:00:00 UTC

[trafficserver] branch 9.2.x updated: Ignore POST request case from a check for background fill (#9013)

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

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


The following commit(s) were added to refs/heads/9.2.x by this push:
     new dad42297a Ignore POST request case from a check for background fill (#9013)
dad42297a is described below

commit dad42297ad70ece5dbdf4ae82245e599f359cc1c
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Tue Aug 9 10:30:20 2022 +0900

    Ignore POST request case from a check for background fill (#9013)
    
    (cherry picked from commit 1f3e1111931e26b5ad483a8bc80b3ae7edc0b568)
---
 proxy/http/HttpTunnel.h | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/proxy/http/HttpTunnel.h b/proxy/http/HttpTunnel.h
index ff31ef5f4..b4a876ef8 100644
--- a/proxy/http/HttpTunnel.h
+++ b/proxy/http/HttpTunnel.h
@@ -546,12 +546,14 @@ HttpTunnel::has_consumer_besides_client() const
       continue;
     }
 
-    if (consumer.vc_type == HT_HTTP_CLIENT) {
-      res = false;
+    switch (consumer.vc_type) {
+    case HT_HTTP_CLIENT:
       continue;
-    } else {
-      res = true;
-      break;
+    case HT_HTTP_SERVER:
+      // ignore uploading data to servers
+      continue;
+    default:
+      return true;
     }
   }