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 2021/11/01 18:18:38 UTC

[trafficserver] branch 9.1.x updated (21f17b3 -> 442195e)

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

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


    from 21f17b3  Updated ChangeLog
     new 0287aa7  Revert "Add some checking to validate the scheme matches the wire protocol. (#8465)"
     new 442195e  Updated ChangeLog

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG-9.1.1      |  1 -
 proxy/http/HttpSM.cc | 12 ------------
 2 files changed, 13 deletions(-)

[trafficserver] 01/02: Revert "Add some checking to validate the scheme matches the wire protocol. (#8465)"

Posted by bc...@apache.org.
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

commit 0287aa7eac364d2510bc45c4bac2d104b118fc1d
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon Nov 1 10:49:30 2021 -0700

    Revert "Add some checking to validate the scheme matches the wire protocol. (#8465)"
    
    This reverts commit d6462761f284634e482a95974002ed271b0efa1b.
---
 proxy/http/HttpSM.cc | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 2cf3e6a..0416edc 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -863,18 +863,6 @@ HttpSM::state_read_client_request_header(int event, void *data)
       break;
     }
 
-    if (!is_internal) {
-      auto scheme = t_state.hdr_info.client_request.url_get()->scheme_get_wksidx();
-      if ((client_connection_is_ssl && (scheme == URL_WKSIDX_HTTP || scheme == URL_WKSIDX_WS)) ||
-          (!client_connection_is_ssl && (scheme == URL_WKSIDX_HTTPS || scheme == URL_WKSIDX_WSS))) {
-        SMDebug("http", "scheme [%s] vs. protocol [%s] mismatch", hdrtoken_index_to_wks(scheme),
-                client_connection_is_ssl ? "tls" : "plaintext");
-        t_state.http_return_code = HTTP_STATUS_BAD_REQUEST;
-        call_transact_and_set_next_state(HttpTransact::BadRequest);
-        break;
-      }
-    }
-
     if (_from_early_data) {
       // Only allow early data for safe methods defined in RFC7231 Section 4.2.1.
       // https://tools.ietf.org/html/rfc7231#section-4.2.1

[trafficserver] 02/02: Updated ChangeLog

Posted by bc...@apache.org.
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

commit 442195ec092dd3d1709058208a34b2bdfe536827
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon Nov 1 11:09:44 2021 -0700

    Updated ChangeLog
---
 CHANGELOG-9.1.1 | 1 -
 1 file changed, 1 deletion(-)

diff --git a/CHANGELOG-9.1.1 b/CHANGELOG-9.1.1
index c031a5b..607d3d5 100644
--- a/CHANGELOG-9.1.1
+++ b/CHANGELOG-9.1.1
@@ -31,6 +31,5 @@ Changes with Apache Traffic Server 9.1.1
   #8452 - Detect and handle chunk header size truncation
   #8453 - Ignore ECONNABORTED on blocking accept
   #8455 - Fix output '\n' HTTP field line endings
-  #8465 - Add some checking to validate the scheme matches the wire protocol.
   #8475 - Check length before search accept-encoding header
   #8476 - Fix case of brotli