You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2016/05/20 16:49:44 UTC

[trafficserver] 12/15: TS-3485: Remove Warning message about ip-allow enforcement not being present for HTTP2.

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

sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

commit c5cfb5bbc670c3c877d941932b737a78753a11fb
Author: shinrich <sh...@ieee.org>
AuthorDate: Tue May 3 18:02:00 2016 -0500

    TS-3485: Remove Warning message about ip-allow enforcement not being present for HTTP2.
    
    (cherry picked from commit 2dd1c4f02414f7639e6cddc580d26f1bc5f1502c)
---
 lib/records/RecHttp.cc            | 2 --
 proxy/http2/Http2SessionAccept.cc | 4 ----
 2 files changed, 6 deletions(-)

diff --git a/lib/records/RecHttp.cc b/lib/records/RecHttp.cc
index dd91105..8620044 100644
--- a/lib/records/RecHttp.cc
+++ b/lib/records/RecHttp.cc
@@ -621,8 +621,6 @@ ts_session_protocol_well_known_name_indices_init()
   REC_ReadConfigInteger(http2_enabled, "proxy.config.http2.enabled");
   if (!http2_enabled) {
     DEFAULT_TLS_SESSION_PROTOCOL_SET.markOut(HTTP2_PROTOCOL_SET);
-  } else {
-    Warning("Be aware that access control checks for HTTP/2 connections are not active!");
   }
 
   DEFAULT_NON_TLS_SESSION_PROTOCOL_SET = HTTP_PROTOCOL_SET;
diff --git a/proxy/http2/Http2SessionAccept.cc b/proxy/http2/Http2SessionAccept.cc
index c32c9d9..7aeefc7 100644
--- a/proxy/http2/Http2SessionAccept.cc
+++ b/proxy/http2/Http2SessionAccept.cc
@@ -38,10 +38,6 @@ Http2SessionAccept::~Http2SessionAccept()
 void
 Http2SessionAccept::accept(NetVConnection *netvc, MIOBuffer *iobuf, IOBufferReader *reader)
 {
-  // XXX we need to refactor the ACL checks from HttpSessionAccept so that we can invoke them here, and also in
-  // the SPDY protocol layer ...
-  // Warning("skipping access control checks for HTTP/2 connection");
-
   netvc->attributes = this->options.transport_type;
 
   const sockaddr *client_ip = netvc->get_remote_addr();

-- 
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.