[trafficserver] branch 9.1.x updated: Updated ChangeLog

2021-10-27 Thread bcall
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 db23ffd  Updated ChangeLog
db23ffd is described below

commit db23ffde281dc739d9f3b90bc8eb26aa7d363b6a
Author: Bryan Call 
AuthorDate: Wed Oct 27 13:56:09 2021 -0700

Updated ChangeLog
---
 CHANGELOG-9.1.1 | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/CHANGELOG-9.1.1 b/CHANGELOG-9.1.1
index 9bcdd5a..a016c2a 100644
--- a/CHANGELOG-9.1.1
+++ b/CHANGELOG-9.1.1
@@ -24,3 +24,10 @@ Changes with Apache Traffic Server 9.1.1
   #8315 - doc: Fixes curl syntax for PUSH example
   #8349 - Fix yamlcpp include folder by using the YAMLCPP_INCLUDE variable 
(#8319)
   #8352 - change MemArena::make test to remove memory leak
+  #8370 - Traffic Dump: update json-schema for new tuple requirements
+  #8419 - Fix a potential H2 stall issue (#8381)
+  #8451 - Reject Transfer-Encoding in pre-HTTP/1.1 requests
+  #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.


[trafficserver] 01/04: Detect and handle chunk header size truncation (#8452)

2021-10-27 Thread bcall
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 094125d492b0d99209f792084578d445440e7944
Author: Brian Neradt 
AuthorDate: Wed Oct 27 11:30:54 2021 -0500

Detect and handle chunk header size truncation (#8452)

This detects if a chunk header size is too large and, if so, closes the
connection.

(cherry picked from commit 83c89f3d217d473ecb000b68c910c0f183c3a355)
---
 include/tscore/ink_memory.h|  19 +++
 proxy/http/HttpTunnel.cc   |  11 +-
 src/tscore/Makefile.am |   1 +
 src/tscore/unit_tests/test_ink_memory.cc   | 141 +
 .../chunked_encoding/bad_chunked_encoding.test.py  |  86 -
 .../replays/malformed_chunked_header.replay.yaml   | 109 
 6 files changed, 364 insertions(+), 3 deletions(-)

diff --git a/include/tscore/ink_memory.h b/include/tscore/ink_memory.h
index fdaaa27..7fd8de1 100644
--- a/include/tscore/ink_memory.h
+++ b/include/tscore/ink_memory.h
@@ -26,6 +26,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 
@@ -205,6 +206,24 @@ ink_zero(T )
   memset(static_cast(), 0, sizeof(t));
 }
 
+/** Verify that we can safely shift value num_places places left.
+ *
+ * This checks that the shift will not cause the variable to overflow and that
+ * the value will not become negative.
+ *
+ * @param[in] value The value against which to check whether the shift is safe.
+ *
+ * @param[in] num_places The number of places to check that shifting left is 
safe.
+ *
+ */
+template 
+inline constexpr bool
+can_safely_shift_left(T value, int num_places)
+{
+  constexpr auto max_value = std::numeric_limits::max();
+  return value >= 0 && value <= (max_value >> num_places);
+}
+
 /** Scoped resources.
 
 An instance of this class is used to hold a contingent resource. When this 
object goes out of scope
diff --git a/proxy/http/HttpTunnel.cc b/proxy/http/HttpTunnel.cc
index 46aa8a8..6267316 100644
--- a/proxy/http/HttpTunnel.cc
+++ b/proxy/http/HttpTunnel.cc
@@ -36,6 +36,7 @@
 #include "HttpSM.h"
 #include "HttpDebugNames.h"
 #include "tscore/ParseRules.h"
+#include "tscore/ink_memory.h"
 
 static const int min_block_transfer_bytes = 256;
 static const char *const CHUNK_HEADER_FMT = "%" PRIx64 "\r\n";
@@ -134,8 +135,16 @@ ChunkedHandler::read_size()
   if (state == CHUNK_READ_SIZE) {
 // The http spec says the chunked size is always in hex
 if (ParseRules::is_hex(*tmp)) {
+  // Make sure we will not overflow running_sum with our shift.
+  if (!can_safely_shift_left(running_sum, 4)) {
+// We have no more space in our variable for the shift.
+state = CHUNK_READ_ERROR;
+done  = true;
+break;
+  }
   num_digits++;
-  running_sum *= 16;
+  // Shift over one hex value.
+  running_sum <<= 4;
 
   if (ParseRules::is_digit(*tmp)) {
 running_sum += *tmp - '0';
diff --git a/src/tscore/Makefile.am b/src/tscore/Makefile.am
index ed6788e..3a6cdfb 100644
--- a/src/tscore/Makefile.am
+++ b/src/tscore/Makefile.am
@@ -170,6 +170,7 @@ test_tscore_SOURCES = \
unit_tests/test_Extendible.cc \
unit_tests/test_History.cc \
unit_tests/test_ink_inet.cc \
+   unit_tests/test_ink_memory.cc \
unit_tests/test_IntrusiveHashMap.cc \
unit_tests/test_IntrusivePtr.cc \
unit_tests/test_IpMap.cc \
diff --git a/src/tscore/unit_tests/test_ink_memory.cc 
b/src/tscore/unit_tests/test_ink_memory.cc
new file mode 100644
index 000..fa6725b
--- /dev/null
+++ b/src/tscore/unit_tests/test_ink_memory.cc
@@ -0,0 +1,141 @@
+/** @file
+
+ink_memory unit tests.
+
+@section license License
+
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+#include 
+#include 
+#include "tscore/ink_memory.h"
+
+constexpr void
+test_can_safely_shift_int8_t()
+{
+  constexpr int8_t a = 0;
+  s

[trafficserver] 03/04: Fix output '\n' HTTP field line endings (#8455)

2021-10-27 Thread bcall
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 1a3ec881d5ed358fb52bc53e882c213c0dd4e3ac
Author: Brian Neradt 
AuthorDate: Wed Oct 27 13:35:41 2021 -0500

Fix output '\n' HTTP field line endings (#8455)

This is another attempt to fix what was initially addressed in #8096 but
got backed out via #8305. That more extensive patch was considered too
invasive and potentially risky.  This more targeted patch will fix
clients that only send the \n endings but it will force the \r\n line
ending on output.

This was mostly in place except for header lines that get
m_n_v_raw_printable set, which seems to be most header lines. The
addition checks to see if the header line ends in \r\n. If it does not
the m_n_v_raw_printable flag gets cleared and the logic that explicitly
adds the line endings while be invoked on output.

(cherry picked from commit 64f25678bfbbd1433cce703e3c43bcc49a53de56)
---
 proxy/hdrs/MIME.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/proxy/hdrs/MIME.cc b/proxy/hdrs/MIME.cc
index b88c5d5..3bf69fb 100644
--- a/proxy/hdrs/MIME.cc
+++ b/proxy/hdrs/MIME.cc
@@ -2580,6 +2580,8 @@ mime_parser_parse(MIMEParser *parser, HdrHeap *heap, 
MIMEHdrImpl *mh, const char
   }
   field_name.rtrim_if(::is_ws);
   raw_print_field = false;
+} else if (parsed.suffix(2) != "\r\n") {
+  raw_print_field = false;
 }
 
 // find value first


[trafficserver] 02/04: Reject Transfer-Encoding in pre-HTTP/1.1 requests (#8451)

2021-10-27 Thread bcall
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 420935b7d7c61ab94a8addaaa43b6affb1fa9ab7
Author: Brian Neradt 
AuthorDate: Wed Oct 27 13:34:46 2021 -0500

Reject Transfer-Encoding in pre-HTTP/1.1 requests (#8451)

Per spec, Transfer-Encoding is only supported in HTTP/1.1. For earlier
versions, we must reject Transfer-Encoding rather than interpret it
since downstream proxies may ignore the chunk header and rely upon the
Content-Length, or interpret the body some other way.  These differences
in interpretation may open up the door to compatibility issues. To
protect against this, we reply with a 4xx if the client uses
Transfer-Encoding with HTTP versions that do not support it.

(cherry picked from commit 6e5070118a20772a30c3fccee2cf1c44f0a21fc0)
---
 proxy/http/HttpTransact.cc | 11 +
 .../chunked_encoding/bad_chunked_encoding.test.py  | 53 --
 .../replays/chunked_in_http_1_0.replay.yaml| 48 
 3 files changed, 109 insertions(+), 3 deletions(-)

diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 813dfda..380184d 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -5358,6 +5358,17 @@ HttpTransact::check_request_validity(State *s, HTTPHdr 
*incoming_hdr)
   return BAD_CONNECT_PORT;
 }
 
+if (s->client_info.transfer_encoding == CHUNKED_ENCODING && 
incoming_hdr->version_get() < HTTP_1_1) {
+  // Per spec, Transfer-Encoding is only supported in HTTP/1.1. For earlier
+  // versions, we must reject Transfer-Encoding rather than interpret it
+  // since downstream proxies may ignore the chunk header and rely upon the
+  // Content-Length, or interpret the body some other way. These
+  // differences in interpretation may open up the door to compatibility
+  // issues. To protect against this, we reply with a 4xx if the client
+  // uses Transfer-Encoding with HTTP versions that do not support it.
+  return UNACCEPTABLE_TE_REQUIRED;
+}
+
 // Require Content-Length/Transfer-Encoding for POST/PUSH/PUT
 if ((scheme == URL_WKSIDX_HTTP || scheme == URL_WKSIDX_HTTPS) &&
 (method == HTTP_WKSIDX_POST || method == HTTP_WKSIDX_PUSH || method == 
HTTP_WKSIDX_PUT) &&
diff --git a/tests/gold_tests/chunked_encoding/bad_chunked_encoding.test.py 
b/tests/gold_tests/chunked_encoding/bad_chunked_encoding.test.py
index 38e4206..d64daa5 100644
--- a/tests/gold_tests/chunked_encoding/bad_chunked_encoding.test.py
+++ b/tests/gold_tests/chunked_encoding/bad_chunked_encoding.test.py
@@ -71,6 +71,53 @@ tr.StillRunningAfter = server
 tr.StillRunningAfter = ts
 
 
+class HTTP10Test:
+chunkedReplayFile = "replays/chunked_in_http_1_0.replay.yaml"
+
+def __init__(self):
+self.setupOriginServer()
+self.setupTS()
+
+def setupOriginServer(self):
+self.server = Test.MakeVerifierServerProcess("verifier-server1", 
self.chunkedReplayFile)
+
+def setupTS(self):
+self.ts = Test.MakeATSProcess("ts2", enable_tls=True, 
enable_cache=False)
+self.ts.addDefaultSSLFiles()
+self.ts.Disk.records_config.update({
+"proxy.config.diags.debug.enabled": 1,
+"proxy.config.diags.debug.tags": "http",
+"proxy.config.ssl.server.cert.path": f'{self.ts.Variables.SSLDir}',
+"proxy.config.ssl.server.private_key.path": 
f'{self.ts.Variables.SSLDir}',
+"proxy.config.ssl.client.verify.server.policy": 'PERMISSIVE',
+})
+self.ts.Disk.ssl_multicert_config.AddLine(
+'dest_ip=* ssl_cert_name=server.pem ssl_key_name=server.key'
+)
+self.ts.Disk.remap_config.AddLine(
+f"map / http://127.0.0.1:{self.server.Variables.http_port}/;,
+)
+
+def runChunkedTraffic(self):
+tr = Test.AddTestRun()
+tr.AddVerifierClientProcess(
+"client1",
+self.chunkedReplayFile,
+http_ports=[self.ts.Variables.port],
+https_ports=[self.ts.Variables.ssl_port],
+other_args='--thread-limit 1')
+tr.Processes.Default.StartBefore(self.server)
+tr.Processes.Default.StartBefore(self.ts)
+tr.StillRunningAfter = self.server
+tr.StillRunningAfter = self.ts
+
+def run(self):
+self.runChunkedTraffic()
+
+
+HTTP10Test().run()
+
+
 class MalformedChunkHeaderTest:
 chunkedReplayFile = "replays/malformed_chunked_header.replay.yaml"
 
@@ -79,7 +126,7 @@ class MalformedChunkHeaderTest:
 self.setupTS()
 
 def setupOriginServer(self):
-self.server = Test.MakeVerifierServerP

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

2021-10-27 Thread bcall
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 d6462761f284634e482a95974002ed271b0efa1b
Author: Alan M. Carroll 
AuthorDate: Wed Oct 27 13:41:34 2021 -0500

Add some checking to validate the scheme matches the wire protocol. (#8465)

(cherry picked from commit 92849ce8e99155c914aea4b82ed63e10e428bee1)

 Conflicts:
proxy/http/HttpSM.cc
---
 proxy/http/HttpSM.cc | 12 
 1 file changed, 12 insertions(+)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 0416edc..2cf3e6a 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -863,6 +863,18 @@ 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] branch 9.1.x updated (8657eb0 -> d646276)

2021-10-27 Thread bcall
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 8657eb0  Fix a potential H2 stall issue (#8381) (#8419)
 new 094125d  Detect and handle chunk header size truncation (#8452)
 new 420935b  Reject Transfer-Encoding in pre-HTTP/1.1 requests (#8451)
 new 1a3ec88  Fix output '\n' HTTP field line endings (#8455)
 new d646276  Add some checking to validate the scheme matches the wire 
protocol. (#8465)

The 4 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:
 include/tscore/ink_memory.h|  19 +++
 proxy/hdrs/MIME.cc |   2 +
 proxy/http/HttpSM.cc   |  12 ++
 proxy/http/HttpTransact.cc |  11 ++
 proxy/http/HttpTunnel.cc   |  11 +-
 src/tscore/Makefile.am |   1 +
 src/tscore/unit_tests/test_ink_memory.cc   | 141 +
 .../chunked_encoding/bad_chunked_encoding.test.py  | 133 ++-
 .../replays/chunked_in_http_1_0.replay.yaml}   |  29 ++---
 .../replays/malformed_chunked_header.replay.yaml   | 109 
 10 files changed, 450 insertions(+), 18 deletions(-)
 create mode 100644 src/tscore/unit_tests/test_ink_memory.cc
 copy tests/gold_tests/{dns/replay/single_transaction.replay.yaml => 
chunked_encoding/replays/chunked_in_http_1_0.replay.yaml} (65%)
 create mode 100644 
tests/gold_tests/chunked_encoding/replays/malformed_chunked_header.replay.yaml


[trafficserver] branch 8.1.x updated: Add some checking to validate the scheme matches the wire protocol. (#8464)

2021-10-27 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/8.1.x by this push:
 new feefc5e  Add some checking to validate the scheme matches the wire 
protocol. (#8464)
feefc5e is described below

commit feefc5e4abc5011dfad5dcfef3f22998faf6e2d4
Author: Alan M. Carroll 
AuthorDate: Wed Oct 27 13:41:47 2021 -0500

Add some checking to validate the scheme matches the wire protocol. (#8464)
---
 proxy/http/HttpSM.cc | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index f222714..791b625 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -732,6 +732,17 @@ HttpSM::state_read_client_request_header(int event, void 
*data)
   case PARSE_RESULT_DONE:
 SMDebug("http", "[%" PRId64 "] done parsing client request header", sm_id);
 
+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;
+  }
+}
 ua_txn->set_session_active();
 
 if (t_state.hdr_info.client_request.version_get() == HTTPVersion(1, 1) &&


[trafficserver] branch master updated: Add some checking to validate the scheme matches the wire protocol. (#8465)

2021-10-27 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 92849ce  Add some checking to validate the scheme matches the wire 
protocol. (#8465)
92849ce is described below

commit 92849ce8e99155c914aea4b82ed63e10e428bee1
Author: Alan M. Carroll 
AuthorDate: Wed Oct 27 13:41:34 2021 -0500

Add some checking to validate the scheme matches the wire protocol. (#8465)
---
 proxy/http/HttpSM.cc | 16 ++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index aef9fc4..e488f84 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -878,6 +878,18 @@ 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
@@ -1910,8 +1922,8 @@ HttpSM::state_http_server_open(int event, void *data)
 this->create_server_txn(new_session);
 
 // Since the UnixNetVConnection::action_ or SocksEntry::action_ may be 
returned from netProcessor.connect_re, and the
-// SocksEntry::action_ will be copied into UnixNetVConnection::action_ 
before call back NET_EVENT_OPEN from SocksEntry::free(),
-// so we just compare the Continuation between pending_action and VC's 
action_.
+// SocksEntry::action_ will be copied into UnixNetVConnection::action_ 
before call back NET_EVENT_OPEN from
+// SocksEntry::free(), so we just compare the Continuation between 
pending_action and VC's action_.
 ink_release_assert(pending_action.empty() || 
pending_action.get_continuation() == vc->get_action()->continuation);
 pending_action = nullptr;
 


[trafficserver] branch 8.1.x updated: Ignore ECONNABORTED on blocking accept (#8456)

2021-10-27 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/8.1.x by this push:
 new 2b07874  Ignore ECONNABORTED on blocking accept (#8456)
2b07874 is described below

commit 2b078741ecf14cbc7f5773b3e14ef0c1d3cf4cfb
Author: Masaori Koshiba 
AuthorDate: Thu Oct 28 01:53:25 2021 +0900

Ignore ECONNABORTED on blocking accept (#8456)

(cherry picked from commit 5b727f177fa7d1bea4b0571551dbb8f680b4ef62)

Conflicts:
iocore/net/UnixNetAccept.cc
---
 iocore/net/P_UnixNet.h  |  4 +++-
 iocore/net/UnixNetAccept.cc | 30 --
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/iocore/net/P_UnixNet.h b/iocore/net/P_UnixNet.h
index 0440b72..8e4d18b 100644
--- a/iocore/net/P_UnixNet.h
+++ b/iocore/net/P_UnixNet.h
@@ -447,6 +447,7 @@ change_net_connections_throttle(const char *token, RecDataT 
data_type, RecData v
   return 0;
 }
 
+// 2  - ignore
 // 1  - transient
 // 0  - report as warning
 // -1 - fatal
@@ -454,8 +455,9 @@ TS_INLINE int
 accept_error_seriousness(int res)
 {
   switch (res) {
-  case -EAGAIN:
   case -ECONNABORTED:
+return 2;
+  case -EAGAIN:
   case -ECONNRESET: // for Linux
   case -EPIPE:  // also for Linux
 return 1;
diff --git a/iocore/net/UnixNetAccept.cc b/iocore/net/UnixNetAccept.cc
index 5d5288d..c9915c5 100644
--- a/iocore/net/UnixNetAccept.cc
+++ b/iocore/net/UnixNetAccept.cc
@@ -261,19 +261,29 @@ NetAccept::do_blocking_accept(EThread *t)
   do {
 if ((res = server.accept()) < 0) {
   int seriousness = accept_error_seriousness(res);
-  if (seriousness >= 0) { // not so bad
-if (!seriousness) {   // bad enough to warn about
-  check_transient_accept_error(res);
-}
+  switch (seriousness) {
+  case 0:
+// bad enough to warn about
+check_transient_accept_error(res);
 safe_delay(net_throttle_delay);
 return 0;
+  case 1:
+// not so bad but needs delay
+safe_delay(net_throttle_delay);
+return 0;
+  case 2:
+// ignore
+return 0;
+  case -1:
+[[fallthrough]];
+  default:
+if (!action_->cancelled) {
+  SCOPED_MUTEX_LOCK(lock, action_->mutex ? action_->mutex : t->mutex, 
t);
+  action_->continuation->handleEvent(EVENT_ERROR, (void 
*)static_cast(res));
+  Warning("accept thread received fatal error: errno = %d", errno);
+}
+return -1;
   }
-  if (!action_->cancelled) {
-SCOPED_MUTEX_LOCK(lock, action_->mutex, t);
-action_->continuation->handleEvent(EVENT_ERROR, (void *)(intptr_t)res);
-Warning("accept thread received fatal error: errno = %d", errno);
-  }
-  return -1;
 }
 // check for throttle
 if (!opt.backdoor && check_net_throttle(ACCEPT)) {


[trafficserver] branch master updated: Ignore ECONNABORTED on blocking accept (#8453)

2021-10-27 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 268b540  Ignore ECONNABORTED on blocking accept (#8453)
268b540 is described below

commit 268b540edae0b3e51d033795a4dd7404a5756a93
Author: Masaori Koshiba 
AuthorDate: Thu Oct 28 01:53:14 2021 +0900

Ignore ECONNABORTED on blocking accept (#8453)
---
 iocore/net/P_UnixNet.h  |  4 +++-
 iocore/net/UnixNetAccept.cc | 30 --
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/iocore/net/P_UnixNet.h b/iocore/net/P_UnixNet.h
index 0043e1d..cff9439 100644
--- a/iocore/net/P_UnixNet.h
+++ b/iocore/net/P_UnixNet.h
@@ -480,6 +480,7 @@ change_net_connections_throttle(const char *token, RecDataT 
data_type, RecData v
   return 0;
 }
 
+// 2  - ignore
 // 1  - transient
 // 0  - report as warning
 // -1 - fatal
@@ -487,8 +488,9 @@ TS_INLINE int
 accept_error_seriousness(int res)
 {
   switch (res) {
-  case -EAGAIN:
   case -ECONNABORTED:
+return 2;
+  case -EAGAIN:
   case -ECONNRESET: // for Linux
   case -EPIPE:  // also for Linux
 return 1;
diff --git a/iocore/net/UnixNetAccept.cc b/iocore/net/UnixNetAccept.cc
index d1ecc22..1ce5923 100644
--- a/iocore/net/UnixNetAccept.cc
+++ b/iocore/net/UnixNetAccept.cc
@@ -295,19 +295,29 @@ NetAccept::do_blocking_accept(EThread *t)
   do {
 if ((res = server.accept()) < 0) {
   int seriousness = accept_error_seriousness(res);
-  if (seriousness >= 0) { // not so bad
-if (!seriousness) {   // bad enough to warn about
-  check_transient_accept_error(res);
-}
+  switch (seriousness) {
+  case 0:
+// bad enough to warn about
+check_transient_accept_error(res);
 safe_delay(net_throttle_delay);
 return 0;
+  case 1:
+// not so bad but needs delay
+safe_delay(net_throttle_delay);
+return 0;
+  case 2:
+// ignore
+return 0;
+  case -1:
+[[fallthrough]];
+  default:
+if (!action_->cancelled) {
+  SCOPED_MUTEX_LOCK(lock, action_->mutex ? action_->mutex : t->mutex, 
t);
+  action_->continuation->handleEvent(EVENT_ERROR, (void 
*)static_cast(res));
+  Warning("accept thread received fatal error: errno = %d", errno);
+}
+return -1;
   }
-  if (!action_->cancelled) {
-SCOPED_MUTEX_LOCK(lock, action_->mutex ? action_->mutex : t->mutex, t);
-action_->continuation->handleEvent(EVENT_ERROR, (void 
*)static_cast(res));
-Warning("accept thread received fatal error: errno = %d", errno);
-  }
-  return -1;
 }
 // check for throttle
 if (!opt.backdoor && check_net_throttle(ACCEPT)) {


[trafficserver] branch master updated: Report an error if configure can't find zlib (#8446)

2021-10-24 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 a100761  Report an error if configure can't find zlib (#8446)
a100761 is described below

commit a100761fd30e4b978ef56d9b565d1a09df9e43cd
Author: Bryan Call 
AuthorDate: Sun Oct 24 07:59:01 2021 -0700

Report an error if configure can't find zlib (#8446)
---
 configure.ac | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/configure.ac b/configure.ac
index d32c4cf..27aa527 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1373,6 +1373,9 @@ AM_CONDITIONAL([BUILD_JA3_PLUGIN], [test 
"x${enable_ja3_plugin}" = "xyes"])
 #
 # Check for zlib presence and usability
 TS_CHECK_ZLIB
+if test "x${enable_zlib}" != "xyes"; then
+  AC_MSG_ERROR([Cannot find zlib library. Configure --with-zlib=DIR])
+fi
 
 #
 # Check for lzma presence and usability


[trafficserver] branch master updated (a316345 -> de96b04)

2021-10-14 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from a316345  Doc: fix typos in Strategy documentation (#8408)
 add de96b04  Increased the max number of operations for marking PR and 
issues stale to 100 (#8394)

No new revisions were added by this update.

Summary of changes:
 .github/workflows/stale.yml | 1 +
 1 file changed, 1 insertion(+)


[trafficserver] branch master updated (6c3e9d2 -> 9467454)

2021-10-13 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 6c3e9d2  AuTest: Execute Test Python Scripts with sys.executable 
(#8412)
 add 9467454  Added required checks before merging (#8413)

No new revisions were added by this update.

Summary of changes:
 .asf.yaml | 14 ++
 1 file changed, 14 insertions(+)


[trafficserver] branch master updated: Revert "Fixed issue with macOS Catalina and pcre 8.43 enabling pcre-jit (#6189)" (#8341)

2021-09-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 3b3942c  Revert "Fixed issue with macOS Catalina and pcre 8.43 
enabling pcre-jit (#6189)" (#8341)
3b3942c is described below

commit 3b3942c05116ce8457f4794a8079c085084f166f
Author: Bryan Call 
AuthorDate: Tue Sep 21 10:28:16 2021 -0700

Revert "Fixed issue with macOS Catalina and pcre 8.43 enabling pcre-jit 
(#6189)" (#8341)

This reverts commit 093317c808b01304abf0a6b4aaf9c34791e3e08e.
---
 src/tscore/Regex.cc | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/tscore/Regex.cc b/src/tscore/Regex.cc
index 81379cd..d9947dc 100644
--- a/src/tscore/Regex.cc
+++ b/src/tscore/Regex.cc
@@ -28,7 +28,7 @@
 #include "tscore/ink_memory.h"
 #include "tscore/Regex.h"
 
-#if defined(PCRE_CONFIG_JIT) && !defined(darwin) // issue with macOS Catalina 
and pcre 8.43
+#ifdef PCRE_CONFIG_JIT
 struct RegexThreadKey {
   RegexThreadKey() { ink_thread_key_create(>key, reinterpret_cast(_jit_stack_free)); }
   ink_thread_key key;
@@ -82,13 +82,13 @@ Regex::compile(const char *pattern, const unsigned flags)
 return false;
   }
 
-#if defined(PCRE_CONFIG_JIT) && !defined(darwin) // issue with macOS Catalina 
and pcre 8.43
+#ifdef PCRE_CONFIG_JIT
   study_opts |= PCRE_STUDY_JIT_COMPILE;
 #endif
 
   regex_extra = pcre_study(regex, study_opts, );
 
-#if defined(PCRE_CONFIG_JIT) && !defined(darwin) // issue with macOS Catalina 
and pcre 8.43
+#ifdef PCRE_CONFIG_JIT
   if (regex_extra) {
 pcre_assign_jit_stack(regex_extra, _jit_stack, nullptr);
   }
@@ -127,7 +127,7 @@ Regex::exec(std::string_view const , int *ovector, int 
ovecsize) const
 Regex::~Regex()
 {
   if (regex_extra) {
-#if defined(PCRE_CONFIG_JIT) && !defined(darwin) // issue with macOS Catalina 
and pcre 8.43
+#ifdef PCRE_CONFIG_JIT
 pcre_free_study(regex_extra);
 #else
 pcre_free(regex_extra);


[trafficserver] branch master updated (82bcd99 -> c30f5ed)

2021-08-30 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 82bcd99  Fixes an issue with next hop self detection, issue #8254 
(#8276)
 add c30f5ed  Fixes a typo in the Rate Limit plugin (#8293)

No new revisions were added by this update.

Summary of changes:
 doc/admin-guide/plugins/rate_limit.en.rst | 1 +
 1 file changed, 1 insertion(+)


[trafficserver] branch master updated (b9668b7 -> f7430ca)

2021-08-30 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from b9668b7  log port # when port is recycled (#8284)
 add f7430ca  Fix the skipping logic for autest and docs (#8227)

No new revisions were added by this update.

Summary of changes:
 ci/jenkins/bin/autest.sh | 4 ++--
 ci/jenkins/bin/docs.sh   | 4 +++-
 2 files changed, 5 insertions(+), 3 deletions(-)


[trafficserver-site] branch asf-site updated: Fixed typo

2021-08-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/asf-site by this push:
 new 0260ce1  Fixed typo
0260ce1 is described below

commit 0260ce1a5dbc46dd0fe9cc80a77c6da368d325c8
Author: Bryan Call 
AuthorDate: Tue Aug 17 15:48:31 2021 -0700

Fixed typo
---
 source/markdown/index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/source/markdown/index.html b/source/markdown/index.html
index 06ccffb..1c2cc9a 100644
--- a/source/markdown/index.html
+++ b/source/markdown/index.html
@@ -287,8 +287,8 @@
 
   
 
-  August 17, 2021:We are releasing both v9.1.0, which is 
our next current release.b>
-  June 24, 2021:We are releasing both v9.0.2 and v8.1.1 
which include security fixes. We recommend everyone to upgrade to one of these 
versions of ATS.
+  August 17, 2021:We are releasing both v9.1.0, which is 
our next current release.
+  June 24, 2021:We are releasing both v9.0.2 and v8.1.1 
which include security fixes. We recommend everyone to upgrade to one of these 
versions of ATS.
   April 16, 2021:We are pleased to announce v9.0.1, which 
is bug-fix release on the v9.x LTS train.
   December 14, 2020:We are pleased to announce the first 
release of ATS v9.0.0, which is our new LTS release cycle. It's available for 
immediate downloads from the regular Downloads page.
   December 2, 2020:We are releasing both v8.1.1 and v7.1.12 
which include bug fixes. We recommend everyone to upgrade to one of these 
versions of ATS.


[trafficserver-site] branch asf-site updated: Fixed a link

2021-08-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/asf-site by this push:
 new b27bbd7  Fixed a link
b27bbd7 is described below

commit b27bbd7bdd8de6c042eaf1966e90098e8f78fb0e
Author: Bryan Call 
AuthorDate: Tue Aug 17 15:46:10 2021 -0700

Fixed a link
---
 source/markdown/downloads.mdtext | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/source/markdown/downloads.mdtext b/source/markdown/downloads.mdtext
index 1394288..f50bea4 100644
--- a/source/markdown/downloads.mdtext
+++ b/source/markdown/downloads.mdtext
@@ -42,10 +42,10 @@ and hash signatures.
  class="download_ts">Traffic Server 9.1.0
 
 v9.1.0 is our latest stable release. Additional details for this release are 
in the
-[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.1.0)
+[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.1.x/CHANGELOG-9.1.0)
 and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/32?closed=1).
 
-For details on the v9.0.x release, please see
+For details on the v9.x release, please see
 [9.0.x 
News](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/whats-new.en.html).
 There are also
 details about [upgrading to 
9.x](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/upgrading.en.html).
 


[trafficserver-site] 01/02: Updated the correct milestones on the download page

2021-08-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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

commit b7470bbd9029fc6d2357663838811fade1692380
Author: Bryan Call 
AuthorDate: Thu Jun 24 15:08:18 2021 -0700

Updated the correct milestones on the download page
---
 source/markdown/downloads.mdtext | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/source/markdown/downloads.mdtext b/source/markdown/downloads.mdtext
index 8cbf9ac..d48dd51 100644
--- a/source/markdown/downloads.mdtext
+++ b/source/markdown/downloads.mdtext
@@ -43,7 +43,7 @@ and hash signatures.
 
 v9.0.2 is our latest stable release. Additional details for this release are 
in the
 
[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.0.2)
-and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/31?closed=1).
+and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/48?closed=1).
 
 For details on the v9.0.x release, please see
 [9.0.x 
News](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/whats-new.en.html).
 There are also
@@ -59,7 +59,7 @@ details about [upgrading to 
9.x](https://docs.trafficserver.apache.org/en/9.0.x/
 
 v8.1.2 is our current stable release in the previous release train. Additional 
details for this release are in the
 
[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/8.1.x/CHANGELOG-8.1.2)
-and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/44?closed=1).
+and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/47?closed=1).
 
 For details on the v8.0.x release, please see
 [8.0.x 
News](https://cwiki.apache.org/confluence/display/TS/What's+New+in+v8.0.x). 
There are also


[trafficserver-site] branch asf-site updated (91b8fd1 -> ded8524)

2021-08-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 91b8fd1  Updated the .asf.yaml file
 new b7470bb  Updated the correct milestones on the download page
 new ded8524  Added 9.1.0

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:
 source/markdown/downloads.mdtext | 22 +++---
 source/markdown/index.html   |  3 ++-
 2 files changed, 13 insertions(+), 12 deletions(-)


[trafficserver-site] 02/02: Added 9.1.0

2021-08-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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

commit ded852419b4a3840eab4df85edff763ec495ee0b
Author: Bryan Call 
AuthorDate: Tue Aug 17 15:43:28 2021 -0700

Added 9.1.0
---
 source/markdown/downloads.mdtext | 20 ++--
 source/markdown/index.html   |  3 ++-
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/source/markdown/downloads.mdtext b/source/markdown/downloads.mdtext
index d48dd51..1394288 100644
--- a/source/markdown/downloads.mdtext
+++ b/source/markdown/downloads.mdtext
@@ -19,7 +19,7 @@ RSS:   /rss/releases.rss
 
 
 
-The latest stable release of Apache Traffic Server is v9.0.2, released on 
2021-06-24.
+The latest stable release of Apache Traffic Server is v9.1.0, released on 
2021-08-17.
 In addition, we continue to support the v8.1.x LTS release train, currently 
v8.1.2,
 which was released on 2021-06-24. We follow the [Semantic 
Versioning](http://semver.org)
 scheme. The goal is to release patch releases frequently, and minor releases 
as needed.
@@ -32,18 +32,18 @@ will be needed.  You can also
 [browse through all releases](https://archive.apache.org/dist/trafficserver/)
 and hash signatures.
 
-# Current v9.x Release -- 9.0.2 # {#9.0.2}
+# Current v9.x Release -- 9.1.0 # {#9.1.0}
 
- Apache Traffic Server v9.0.2 was released on June 24th, 2021.
- 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.2.tar.bz2.asc)]
- 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.2.tar.bz2.sha512)]
+ Apache Traffic Server v9.1.0 was released on August 17th, 2021.
+ 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-9.1.0.tar.bz2.asc)]
+ 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-9.1.0.tar.bz2.sha512)]
 
- https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-9.0.2.tar.bz2;
- class="download_ts">Traffic Server 9.0.2
+ https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-9.1.0.tar.bz2;
+ class="download_ts">Traffic Server 9.1.0
 
-v9.0.2 is our latest stable release. Additional details for this release are 
in the
-[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.0.2)
-and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/48?closed=1).
+v9.1.0 is our latest stable release. Additional details for this release are 
in the
+[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.1.0)
+and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/32?closed=1).
 
 For details on the v9.0.x release, please see
 [9.0.x 
News](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/whats-new.en.html).
 There are also
diff --git a/source/markdown/index.html b/source/markdown/index.html
index 3aa236c..06ccffb 100644
--- a/source/markdown/index.html
+++ b/source/markdown/index.html
@@ -287,9 +287,10 @@
 
   
 
+  August 17, 2021:We are releasing both v9.1.0, which is 
our next current release.b>
   June 24, 2021:We are releasing both v9.0.2 and v8.1.1 
which include security fixes. We recommend everyone to upgrade to one of these 
versions of ATS.
   April 16, 2021:We are pleased to announce v9.0.1, which 
is bug-fix release on the v9.x LTS train.
-  December 14, 2020:We are pleased to announce the first 
release of ATS v9.0.0, which is our new LTS release cycle. It's available for 
immediate downloads from the regular Downloads page..
+  December 14, 2020:We are pleased to announce the first 
release of ATS v9.0.0, which is our new LTS release cycle. It's available for 
immediate downloads from the regular Downloads page.
   December 2, 2020:We are releasing both v8.1.1 and v7.1.12 
which include bug fixes. We recommend everyone to upgrade to one of these 
versions of ATS.
   August 29, 2020:We are pleased to announce the 
availability of our latest stable release, v8.1.0! This is a significant bug 
and performance improvement over the last v8.0.x release, and this is also our 
current LTS train. No new release for 8.0.x is planned.
   June 23, 2020:We are releasing both v8.0.8 and v7.1.11 
which include bug fixes.  We recommend everyone to upgrade to one of these 
versions of ATS.


[trafficserver] branch master updated (3ff09c9 -> 11f987e)

2021-08-05 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 3ff09c9  Modified slice to leverage APIs to specify buffer size and 
watermark. (#8089)
 add 11f987e  Remove some const casting (#8207)

No new revisions were added by this update.

Summary of changes:
 include/tscore/Regex.h   |  4 ++--
 include/tscore/Regression.h  |  2 +-
 src/tscore/Regex.cc  |  9 +++--
 src/tscore/Regression.cc |  9 -
 src/tscore/ink_file.cc   |  4 ++--
 src/tscore/ink_inet.cc   |  4 ++--
 src/tscore/unit_tests/test_BufferWriterFormat.cc | 10 +-
 7 files changed, 19 insertions(+), 23 deletions(-)


[trafficserver] branch 9.1.x updated: Ran clang-tidy over the 9.1.x branch (#8186)

2021-08-03 Thread bcall
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 76dcc80  Ran clang-tidy over the 9.1.x branch (#8186)
76dcc80 is described below

commit 76dcc800964b80200774ade84fe3c9212b518008
Author: Bryan Call 
AuthorDate: Tue Aug 3 15:21:05 2021 -0700

Ran clang-tidy over the 9.1.x branch (#8186)
---
 .../plugins/c-api/ssl_preaccept/ssl_preaccept.cc   |  2 +-
 iocore/cache/test/main.cc  |  4 +--
 plugins/background_fetch/background_fetch.cc   |  2 +-
 .../cache_range_requests/cache_range_requests.cc   |  4 +--
 plugins/escalate/escalate.cc   |  2 +-
 plugins/esi/esi.cc |  2 +-
 plugins/esi/test/parser_test.cc|  2 +-
 plugins/esi/test/utils_test.cc |  6 ++--
 plugins/experimental/access_control/utils.cc   |  2 +-
 .../experimental/cache_fill/background_fetch.cc|  2 +-
 plugins/experimental/cache_fill/cache_fill.cc  |  5 ++--
 .../collapsed_forwarding/collapsed_forwarding.cc   |  4 +--
 plugins/experimental/cookie_remap/cookie_remap.cc  |  4 +--
 plugins/experimental/magick/magick.cc  |  2 +-
 plugins/experimental/maxmind_acl/mmdb.cc   | 32 +++---
 plugins/experimental/memcache/tsmemcache.cc|  2 +-
 plugins/experimental/mysql_remap/mysql_remap.cc|  4 +--
 plugins/experimental/rate_limit/rate_limit.cc  |  2 +-
 plugins/experimental/statichit/statichit.cc| 12 
 .../experimental/stream_editor/stream_editor.cc|  4 +--
 plugins/header_rewrite/conditions_geo_maxmind.cc   |  4 +--
 plugins/header_rewrite/header_rewrite.cc   | 10 +++
 plugins/multiplexer/ats-multiplexer.cc |  2 +-
 proxy/CacheControl.cc  |  2 +-
 proxy/ParentSelection.cc   |  2 +-
 proxy/hdrs/HuffmanCodec.cc |  4 +--
 proxy/http/HttpTransact.cc |  2 +-
 proxy/http2/HPACK.cc   |  4 +--
 proxy/logging/LogField.cc  |  2 +-
 src/traffic_crashlog/traffic_crashlog.cc   |  4 +--
 src/traffic_ctl/config.cc  |  4 +--
 src/traffic_logstats/logstats.cc   | 14 +-
 src/traffic_server/Crash.cc|  2 +-
 src/traffic_server/InkAPI.cc   |  2 +-
 src/traffic_server/InkAPITest.cc   |  2 +-
 src/traffic_server/SocksProxy.cc   | 10 +++
 src/tscore/ink_file.cc |  4 +--
 src/tscore/ink_queue.cc|  4 +--
 src/tscore/unit_tests/test_layout.cc   |  3 +-
 39 files changed, 91 insertions(+), 89 deletions(-)

diff --git a/example/plugins/c-api/ssl_preaccept/ssl_preaccept.cc 
b/example/plugins/c-api/ssl_preaccept/ssl_preaccept.cc
index d580584..be662ad 100644
--- a/example/plugins/c-api/ssl_preaccept/ssl_preaccept.cc
+++ b/example/plugins/c-api/ssl_preaccept/ssl_preaccept.cc
@@ -39,7 +39,7 @@
 
 namespace
 {
-typedef std::pair IpRange;
+using IpRange  = std::pair;
 using IpRangeQueue = std::deque;
 IpRangeQueue ClientBlindTunnelIp;
 
diff --git a/iocore/cache/test/main.cc b/iocore/cache/test/main.cc
index 7c57a0c..3585ab6 100644
--- a/iocore/cache/test/main.cc
+++ b/iocore/cache/test/main.cc
@@ -210,7 +210,7 @@ CacheWriteTest::start_test(int event, void *e)
   }
 
   SET_HANDLER(::write_event);
-  cacheProcessor.open_write(this, 0, , (CacheHTTPHdr 
*)this->info.request_get(), old_info);
+  cacheProcessor.open_write(this, 0, , static_cast(this->info.request_get()), old_info);
   return 0;
 }
 
@@ -271,7 +271,7 @@ CacheReadTest::start_test(int event, void *e)
   key = generate_key(this->info);
 
   SET_HANDLER(::read_event);
-  cacheProcessor.open_read(this, , (CacheHTTPHdr 
*)this->info.request_get(), >params);
+  cacheProcessor.open_read(this, , static_cast(this->info.request_get()), >params);
   return 0;
 }
 
diff --git a/plugins/background_fetch/background_fetch.cc 
b/plugins/background_fetch/background_fetch.cc
index cc5c7fd..ba68372 100644
--- a/plugins/background_fetch/background_fetch.cc
+++ b/plugins/background_fetch/background_fetch.cc
@@ -55,7 +55,7 @@ static const std::array 
FILTER_HEADERS{
 // Hold the global background fetch state. This is currently shared across all
 // configurations, as a singleton. ToDo: Would it ever make sense to do this
 // per remap rule? Maybe for per-remap logging ??
-typedef std::unordered_map OutstandingRequests;
+using OutstandingRequests = std::unordered_map;
 
 class BgFetchState
 {
diff --git a/plugins/cache_range_requests/cache_range_requests.cc 
b/plugins/cache_range_requests/cache_range_requests.cc
index 5b43d45..cb294e8

[trafficserver] branch 9.1.x updated (e987d0a -> 152d6ea)

2021-08-03 Thread bcall
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 e987d0a  Revert "Apply log throttling to HTTP/2 session error rate 
messages (#7772)"
 new 80aaf66  Enforce HTTP parsing restrictions on HTTP versions supported 
(#7875)
 new 152d6ea  Minor updates to HTTP version validation (#8189)

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:
 proxy/hdrs/HTTP.cc | 34 ++
 proxy/hdrs/HTTP.h  |  4 +++-
 proxy/http/HttpSM.cc   |  4 
 proxy/http/HttpTransact.cc |  4 
 4 files changed, 37 insertions(+), 9 deletions(-)


[trafficserver] 01/02: Enforce HTTP parsing restrictions on HTTP versions supported (#7875)

2021-08-03 Thread bcall
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 80aaf6669799093bbc4623d298abd96fcdb73e90
Author: Sudheer Vinukonda 
AuthorDate: Mon May 24 21:14:19 2021 -0700

Enforce HTTP parsing restrictions on HTTP versions supported (#7875)

This change restricts allowed HTTP versions to 1.0, 1.1 on the
HTTP request line to prevent potential mishandling, request
smugging or other vulns due to random/arbitrary version tags

Note that HTTP/2.0 and HTTP/3.0 are negotiated via ALPN on TLS
and not via the HTTP request line.

(cherry picked from commit f36cf6a6e5b1372916170541bec681a80b34c46f)
---
 proxy/hdrs/HTTP.cc | 37 +
 proxy/hdrs/HTTP.h  |  4 +++-
 proxy/http/HttpSM.cc   |  4 
 proxy/http/HttpTransact.cc |  4 
 4 files changed, 40 insertions(+), 9 deletions(-)

diff --git a/proxy/hdrs/HTTP.cc b/proxy/hdrs/HTTP.cc
index 3cf0e18..793f6e8 100644
--- a/proxy/hdrs/HTTP.cc
+++ b/proxy/hdrs/HTTP.cc
@@ -623,12 +623,36 @@ http_hdr_type_set(HTTPHdrImpl *hh, HTTPType type)
 }
 
 /*-
+  RFC2616 specifies that HTTP version is of the format .
+  in the request line.  However, the features supported and in use are
+  for versions 1.0, 1.1 and 2.0 (with HTTP/3.0 being developed). HTTP/2.0
+  and HTTP/3.0 are both negotiated using ALPN over TLS and not via the HTTP
+  request line thus leaving the versions supported on the request line to be
+  HTTP/1.0 and HTTP/1.1 alone. This utility checks if the HTTP Version
+  received in the request line is one of these and returns false otherwise
   -*/
 
-void
+bool
+is_version_supported(const uint8_t major, const uint8_t minor)
+{
+  if (major == 1) {
+return minor == 1 || minor == 0;
+  }
+
+  return false;
+}
+
+bool
+is_http_hdr_version_supported(const HTTPVersion _version)
+{
+  return is_version_supported(http_version.get_major(), 
http_version.get_minor());
+}
+
+bool
 http_hdr_version_set(HTTPHdrImpl *hh, const HTTPVersion )
 {
   hh->m_version = ver;
+  return is_version_supported(ver.get_major(), ver.get_minor());
 }
 
 /*-
@@ -939,13 +963,12 @@ http_parser_parse_req(HTTPParser *parser, HdrHeap *heap, 
HTTPHdrImpl *hh, const
   if (err < 0) {
 return err;
   }
-  http_hdr_version_set(hh, version);
+  if (!http_hdr_version_set(hh, version)) {
+return PARSE_RESULT_ERROR;
+  }
 
   end= real_end;
   parser->m_parsing_http = false;
-  if (version == HTTP_0_9) {
-return PARSE_RESULT_ERROR;
-  }
 
   ParseResult ret = mime_parser_parse(>m_mime_parser, heap, 
hh->m_fields_impl, start, end, must_copy_strings, eof,
   false, max_hdr_field_size);
@@ -1094,12 +1117,10 @@ http_parser_parse_req(HTTPParser *parser, HdrHeap 
*heap, HTTPHdrImpl *hh, const
   return PARSE_RESULT_ERROR;
 }
 
-if (version == HTTP_0_9) {
+if (!http_hdr_version_set(hh, version)) {
   return PARSE_RESULT_ERROR;
 }
 
-http_hdr_version_set(hh, version);
-
 end= real_end;
 parser->m_parsing_http = false;
   }
diff --git a/proxy/hdrs/HTTP.h b/proxy/hdrs/HTTP.h
index 3214c98..dabde58 100644
--- a/proxy/hdrs/HTTP.h
+++ b/proxy/hdrs/HTTP.h
@@ -422,7 +422,7 @@ inkcoreapi int http_hdr_print(HdrHeap *heap, HTTPHdrImpl 
*hh, char *buf, int buf
 
 void http_hdr_describe(HdrHeapObjImpl *obj, bool recurse = true);
 
-inkcoreapi void http_hdr_version_set(HTTPHdrImpl *hh, const HTTPVersion );
+inkcoreapi bool http_hdr_version_set(HTTPHdrImpl *hh, const HTTPVersion );
 
 const char *http_hdr_method_get(HTTPHdrImpl *hh, int *length);
 inkcoreapi void http_hdr_method_set(HdrHeap *heap, HTTPHdrImpl *hh, const char 
*method, int16_t method_wks_idx, int method_length,
@@ -460,6 +460,8 @@ HTTPValRange*  http_parse_range (const char *buf, 
Arena *arena);
 */
 HTTPValTE *http_parse_te(const char *buf, int len, Arena *arena);
 
+inkcoreapi bool is_http_hdr_version_supported(const HTTPVersion _version);
+
 class IOBufferReader;
 
 class HTTPHdr : public MIMEHdr
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 856c110..68a6343 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -823,6 +823,10 @@ HttpSM::state_read_client_request_header(int event, void 
*data)
   t_state.http_return_code = HTTP_STATUS_REQUEST_URI_TOO_LONG :
   t_state.http_return_code = HTTP_STATUS_NONE;
 
+if 
(!is_http_hdr_version_supported(t_state.hdr_info.client_request.version_get())) 
{
+  t_state.http_return_code = HTTP_STATUS_HTTPVER_NOT_SUPPORTED;
+}
+
 call_transact_

[trafficserver] 02/02: Minor updates to HTTP version validation (#8189)

2021-08-03 Thread bcall
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 152d6eafdf53deebb8c403928d66550cf0a0c9d6
Author: Bryan Call 
AuthorDate: Tue Aug 3 14:03:04 2021 -0700

Minor updates to HTTP version validation (#8189)

Renamed the functions to be more explicit about only supporting HTTP/1.x
Changed the version check to be only a logic statement

(cherry picked from commit c5105cd0ec77b71a15cf01b61b1ddbb07a8d44b8)

 Conflicts:
proxy/hdrs/HTTP.h
---
 proxy/hdrs/HTTP.cc   | 15 ++-
 proxy/hdrs/HTTP.h|  2 +-
 proxy/http/HttpSM.cc |  2 +-
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/proxy/hdrs/HTTP.cc b/proxy/hdrs/HTTP.cc
index 793f6e8..c0b16cf 100644
--- a/proxy/hdrs/HTTP.cc
+++ b/proxy/hdrs/HTTP.cc
@@ -633,26 +633,23 @@ http_hdr_type_set(HTTPHdrImpl *hh, HTTPType type)
   -*/
 
 bool
-is_version_supported(const uint8_t major, const uint8_t minor)
+is_http1_version(const uint8_t major, const uint8_t minor)
 {
-  if (major == 1) {
-return minor == 1 || minor == 0;
-  }
-
-  return false;
+  // Return true if 1.1 or 1.0
+  return (major == 1) && (minor == 1 || minor == 0);
 }
 
 bool
-is_http_hdr_version_supported(const HTTPVersion _version)
+is_http1_hdr_version_supported(const HTTPVersion _version)
 {
-  return is_version_supported(http_version.get_major(), 
http_version.get_minor());
+  return is_http1_version(http_version.get_major(), http_version.get_minor());
 }
 
 bool
 http_hdr_version_set(HTTPHdrImpl *hh, const HTTPVersion )
 {
   hh->m_version = ver;
-  return is_version_supported(ver.get_major(), ver.get_minor());
+  return is_http1_version(ver.get_major(), ver.get_minor());
 }
 
 /*-
diff --git a/proxy/hdrs/HTTP.h b/proxy/hdrs/HTTP.h
index dabde58..52602af 100644
--- a/proxy/hdrs/HTTP.h
+++ b/proxy/hdrs/HTTP.h
@@ -460,7 +460,7 @@ HTTPValRange*  http_parse_range (const char *buf, 
Arena *arena);
 */
 HTTPValTE *http_parse_te(const char *buf, int len, Arena *arena);
 
-inkcoreapi bool is_http_hdr_version_supported(const HTTPVersion _version);
+bool is_http1_hdr_version_supported(const HTTPVersion _version);
 
 class IOBufferReader;
 
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 68a6343..f461287 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -823,7 +823,7 @@ HttpSM::state_read_client_request_header(int event, void 
*data)
   t_state.http_return_code = HTTP_STATUS_REQUEST_URI_TOO_LONG :
   t_state.http_return_code = HTTP_STATUS_NONE;
 
-if 
(!is_http_hdr_version_supported(t_state.hdr_info.client_request.version_get())) 
{
+if 
(!is_http1_hdr_version_supported(t_state.hdr_info.client_request.version_get()))
 {
   t_state.http_return_code = HTTP_STATUS_HTTPVER_NOT_SUPPORTED;
 }
 


[trafficserver] branch master updated (532e30c -> c5105cd)

2021-08-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 532e30c  Fix error connection logging crash (#8191)
 add c5105cd  Minor updates to HTTP version validation (#8189)

No new revisions were added by this update.

Summary of changes:
 proxy/hdrs/HTTP.cc   | 15 ++-
 proxy/hdrs/HTTP.h|  2 +-
 proxy/http/HttpSM.cc |  2 +-
 3 files changed, 8 insertions(+), 11 deletions(-)


[trafficserver-site] branch asf-site updated: Updated the .asf.yaml file

2021-08-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/asf-site by this push:
 new 91b8fd1  Updated the .asf.yaml file
91b8fd1 is described below

commit 91b8fd144ed933108b01f6328477dc3e50d27f73
Author: Bryan Call 
AuthorDate: Tue Aug 3 10:29:06 2021 -0700

Updated the .asf.yaml file
---
 .asf.yaml | 29 +
 1 file changed, 29 insertions(+)

diff --git a/.asf.yaml b/.asf.yaml
index a6585d7..88f4ebd 100644
--- a/.asf.yaml
+++ b/.asf.yaml
@@ -1,3 +1,32 @@
+# Documentation 
https://cwiki.apache.org/confluence/display/INFRA/git+-+.asf.yaml+features
+notifications:
+  commits: commits@trafficserver.apache.org
+  issues: iss...@trafficserver.apache.org
+  pullrequests: git...@trafficserver.apache.org
+
+github:
+  description: "Apache Traffic Serverâ„¢ is a fast, scalable and extensible 
HTTP/1.1 and HTTP/2 compliant caching proxy server."
+  homepage: https://trafficserver.apache.org/
+  labels:
+- proxy
+- cdn
+- cache
+- apache
+  features:
+# Enable wiki for documentation
+wiki: false
+# Enable issue management
+issues: true
+# Enable projects for project management boards
+projects: false
+  enabled_merge_buttons:
+# Enable squash button:
+squash: true
+# Disable rebase button:
+rebase: false
+# Disable merge button:
+merge: false
+
 publish:
   whoami: asf-site
 


[trafficserver] branch master updated (1fb81df -> 56868c1)

2021-08-02 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 1fb81df  Fixing TS_HTTP_REQUEST_BUFFER_READ_COMPLETE_HOOK enum value. 
(#8066)
 add 56868c1  Ran clang-tidy over the master branch (#8187)

No new revisions were added by this update.

Summary of changes:
 include/tscore/LogMessage.h |  2 +-
 iocore/net/P_SSLUtils.h |  2 +-
 iocore/net/SSLConfig.cc |  8 
 iocore/net/SSLUtils.cc  |  2 +-
 mgmt/YamlCfg.cc |  2 +-
 mgmt/YamlCfg.h  |  2 +-
 plugins/experimental/cache_fill/background_fetch.cc |  2 +-
 plugins/experimental/cache_fill/cache_fill.cc   |  5 +++--
 plugins/experimental/parent_select/strategy.cc  |  2 +-
 plugins/experimental/rate_limit/sni_limiter.cc  |  2 +-
 plugins/experimental/rate_limit/sni_selector.cc |  2 +-
 plugins/experimental/rate_limit/txn_limiter.cc  |  2 +-
 plugins/header_rewrite/conditions_geo_maxmind.cc|  4 ++--
 plugins/header_rewrite/header_rewrite.cc| 10 +-
 proxy/http/HttpTransact.cc  |  2 +-
 proxy/http/remap/UrlMapping.cc  |  2 +-
 src/tscore/LogMessage.cc|  4 ++--
 src/tscore/unit_tests/freelist_benchmark.cc |  4 ++--
 src/tscore/unit_tests/test_MMH.cc   | 10 ++
 19 files changed, 36 insertions(+), 33 deletions(-)


[trafficserver] branch 9.1.x updated: Revert "Apply log throttling to HTTP/2 session error rate messages (#7772)"

2021-07-29 Thread bcall
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 e987d0a  Revert "Apply log throttling to HTTP/2 session error rate 
messages (#7772)"
e987d0a is described below

commit e987d0a4237152476f49f18c3c245ba2a36363b5
Author: Bryan Call 
AuthorDate: Thu Jul 29 08:48:43 2021 -0700

Revert "Apply log throttling to HTTP/2 session error rate messages (#7772)"

This reverts commit 78a7df6eeed1ad2dbebf8e711129ab8d469c8641.
---
 proxy/http2/Http2ClientSession.cc | 14 ++
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/proxy/http2/Http2ClientSession.cc 
b/proxy/http2/Http2ClientSession.cc
index f55abf6..de7034f 100644
--- a/proxy/http2/Http2ClientSession.cc
+++ b/proxy/http2/Http2ClientSession.cc
@@ -360,10 +360,9 @@ Http2ClientSession::main_event_handler(int event, void 
*edata)
Http2::stream_error_rate_threshold) { // For a case many stream 
errors happened
   ip_port_text_buffer ipb;
   const char *client_ip = ats_ip_ntop(get_remote_addr(), ipb, sizeof(ipb));
-  SiteThrottledWarning("HTTP/2 session error client_ip=%s session_id=%" 
PRId64
-   " closing a connection, because its stream error 
rate (%f) exceeded the threshold (%f)",
-   client_ip, connection_id(), 
this->connection_state.get_stream_error_rate(),
-   Http2::stream_error_rate_threshold);
+  Warning("HTTP/2 session error client_ip=%s session_id=%" PRId64
+  " closing a connection, because its stream error rate (%f) 
exceeded the threshold (%f)",
+  client_ip, connection_id(), 
this->connection_state.get_stream_error_rate(), 
Http2::stream_error_rate_threshold);
   Http2SsnDebug("Preparing for graceful shutdown because of a high stream 
error rate");
   cause_of_death = Http2SessionCod::HIGH_ERROR_RATE;
   this->connection_state.set_shutdown_state(HTTP2_SHUTDOWN_NOT_INITIATED, 
Http2ErrorCode::HTTP2_ERROR_ENHANCE_YOUR_CALM);
@@ -558,10 +557,9 @@ Http2ClientSession::state_process_frame_read(int event, 
VIO *vio, bool inside_fr
 if (this->connection_state.get_stream_error_rate() > std::min(1.0, 
Http2::stream_error_rate_threshold * 2.0)) {
   ip_port_text_buffer ipb;
   const char *client_ip = ats_ip_ntop(get_remote_addr(), ipb, sizeof(ipb));
-  SiteThrottledWarning("HTTP/2 session error client_ip=%s session_id=%" 
PRId64
-   " closing a connection, because its stream error 
rate (%f) exceeded the threshold (%f)",
-   client_ip, connection_id(), 
this->connection_state.get_stream_error_rate(),
-   Http2::stream_error_rate_threshold);
+  Warning("HTTP/2 session error client_ip=%s session_id=%" PRId64
+  " closing a connection, because its stream error rate (%f) 
exceeded the threshold (%f)",
+  client_ip, connection_id(), 
this->connection_state.get_stream_error_rate(), 
Http2::stream_error_rate_threshold);
   err = Http2ErrorCode::HTTP2_ERROR_ENHANCE_YOUR_CALM;
 }
 


[trafficserver-site] branch asf-site updated: adding .asf.yaml file

2021-07-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/asf-site by this push:
 new ad88da9  adding .asf.yaml file
ad88da9 is described below

commit ad88da9595da98394e97f4e1191c136f09b8e5ac
Author: Bryan Call 
AuthorDate: Wed Jul 28 10:34:42 2021 -0700

adding .asf.yaml file
---
 .asf.yaml | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/.asf.yaml b/.asf.yaml
new file mode 100644
index 000..a6585d7
--- /dev/null
+++ b/.asf.yaml
@@ -0,0 +1,6 @@
+publish:
+  whoami: asf-site
+
+staging:
+  whoami: asf-site
+


[trafficserver] branch master updated (ceb2b38 -> 84cf02e)

2021-07-26 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from ceb2b38  Add plugin parent_select reloading (#8075)
 add 84cf02e  Added missing milestones and updated slow log report script 
(#8168)

No new revisions were added by this update.

Summary of changes:
 proxy/http/HttpSM.cc | 11 +--
 tools/slow_log_report.pl | 12 ++--
 2 files changed, 15 insertions(+), 8 deletions(-)


[trafficserver] branch 9.1.x updated: Updates to webp_transform to convert webp to jpeg (#8005)

2021-07-19 Thread bcall
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 c98b986  Updates to webp_transform to convert webp to jpeg (#8005)
c98b986 is described below

commit c98b9869b737b62084557f9bf6b37eeaf741d8c1
Author: Bryan Call 
AuthorDate: Thu Jul 15 09:19:47 2021 -0700

Updates to webp_transform to convert webp to jpeg (#8005)

Co-authored-by: Siva Renganathan 
(cherry picked from commit b2004604e00bc7216856ea5e79082b4e53118a76)
---
 doc/admin-guide/plugins/webp_transform.en.rst  |   7 +-
 .../experimental/webp_transform/ImageTransform.cc  | 148 ++---
 plugins/experimental/webp_transform/Makefile.inc   |  12 +-
 3 files changed, 138 insertions(+), 29 deletions(-)

diff --git a/doc/admin-guide/plugins/webp_transform.en.rst 
b/doc/admin-guide/plugins/webp_transform.en.rst
index 1fa2262..f729a59 100644
--- a/doc/admin-guide/plugins/webp_transform.en.rst
+++ b/doc/admin-guide/plugins/webp_transform.en.rst
@@ -22,16 +22,17 @@
 Webp Transform Plugin
 *
 
-This plugin converts jpeg and png images and transforms them into webp format.
+This plugin converts jpeg and png images and transforms them into webp format 
for browsers that support webp.
+Also, the plugin converts webp images and transforms them to jpeg for browsers 
that don't support webp
 All response with content-type 'image/jpeg' or 'image/png' will go through the 
transform.
-Content-type is changed to 'image/webp' on successful transformation.
+Content-type is changed to 'image/webp' or 'image/jpeg' on successful 
transformation.
 
 Installation
 
 
 Add the following line to :file:`plugin.config`::
 
-webp_transform.so
+webp_transform.so [convert_to_jpeg,convert_to_webp]
 
 
 Note
diff --git a/plugins/experimental/webp_transform/ImageTransform.cc 
b/plugins/experimental/webp_transform/ImageTransform.cc
index 88d19cd..0ade23e 100644
--- a/plugins/experimental/webp_transform/ImageTransform.cc
+++ b/plugins/experimental/webp_transform/ImageTransform.cc
@@ -23,10 +23,10 @@
 #include "tscpp/api/GlobalPlugin.h"
 #include "tscpp/api/TransformationPlugin.h"
 #include "tscpp/api/Logger.h"
+#include "tscpp/api/Stat.h"
 
 #include 
 
-using std::string;
 using namespace Magick;
 using namespace atscppapi;
 
@@ -35,12 +35,23 @@ using namespace atscppapi;
 namespace
 {
 GlobalPlugin *plugin;
-}
+
+enum class ImageEncoding { webp, jpeg, png, unknown };
+
+bool config_convert_to_webp = false;
+bool config_convert_to_jpeg = false;
+
+Stat stat_convert_to_webp;
+Stat stat_convert_to_jpeg;
+} // namespace
 
 class ImageTransform : public TransformationPlugin
 {
 public:
-  ImageTransform(Transaction ) : TransformationPlugin(transaction, 
TransformationPlugin::RESPONSE_TRANSFORMATION)
+  ImageTransform(Transaction , ImageEncoding input_image_type, 
ImageEncoding transform_image_type)
+: TransformationPlugin(transaction, 
TransformationPlugin::RESPONSE_TRANSFORMATION),
+  _input_image_type(input_image_type),
+  _transform_image_type(transform_image_type)
   {
 TransformationPlugin::registerHook(HOOK_READ_RESPONSE_HEADERS);
   }
@@ -48,8 +59,22 @@ public:
   void
   handleReadResponseHeaders(Transaction ) override
   {
-transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/webp";
-transaction.getServerResponse().getHeaders()["Vary"] = 
"Content-Type"; // to have a separate cache entry.
+switch (_transform_image_type) {
+case ImageEncoding::webp:
+  transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/webp";
+  break;
+case ImageEncoding::jpeg:
+  transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/jpeg";
+  break;
+case ImageEncoding::png:
+  transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/png";
+  break;
+case ImageEncoding::unknown:
+  // do nothing
+  break;
+}
+
+transaction.getServerResponse().getHeaders()["Vary"] = "Accept"; // to 
have a separate cache entry
 
 TS_DEBUG(TAG, "url %s", 
transaction.getServerRequest().getUrl().getUrlString().c_str());
 transaction.resume();
@@ -64,15 +89,35 @@ public:
   void
   handleInputComplete() override
   {
-string input_data = _img.str();
+std::string input_data = _img.str();
 Blob input_blob(input_data.data(), input_data.length());
 Image image;
-image.read(input_blob);
 
-Blob output_blob;
-image.magick("WEBP");
-image.write(_blob);
-produce(std::string_view(reinterpret_cast(output_blob.data()), output_blob.length()));
+try {
+  image.read(input_blob);
+
+  

[trafficserver] branch master updated: Fixed spelling mistakes in code and other files (#8061)

2021-07-15 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 9de93aa  Fixed spelling mistakes in code and other files (#8061)
9de93aa is described below

commit 9de93aa991f0e76cd30a921f436182fe8386fef8
Author: Bryan Call 
AuthorDate: Thu Jul 15 10:54:34 2021 -0700

Fixed spelling mistakes in code and other files (#8061)
---
 LICENSE|  6 ++---
 README-EC2 |  2 +-
 build/ax_lib_curl.m4   |  2 +-
 ci/jenkins/bin/gh-mirror.sh|  2 +-
 ci/jenkins/git-jenkins-setup.sh|  2 +-
 configs/strategies.schema.json |  2 +-
 configs/strategies.yaml.default|  4 +--
 configure.ac   | 12 -
 contrib/install_trafficserver.sh   |  2 +-
 .../test/SynTest/Tests/Psi/psi_files/tc6_file.txt  |  2 +-
 .../lua-api/modsecurity/ats-luajit-modsecurity.lua |  2 +-
 example/plugins/lua-api/modsecurity/owasp.conf |  2 +-
 include/ts/InkAPIPrivateIOCore.h   |  2 +-
 include/ts/apidefs.h.in|  2 +-
 include/tscore/BufferWriter.h  |  2 +-
 include/tscore/Errata.h| 10 
 include/tscore/Extendible.h|  8 +++---
 include/tscore/I_Layout.h  |  2 +-
 include/tscore/NumericType.h   |  2 +-
 include/tscore/ink_inet.h  |  2 +-
 include/tscpp/api/Url.h|  2 +-
 include/tscpp/util/MemSpan.h   |  2 +-
 include/tscpp/util/TextView.h  |  2 +-
 iocore/cache/Cache.cc  |  4 +--
 iocore/cache/I_Store.h |  2 +-
 iocore/eventsystem/I_IOBuffer.h|  2 +-
 iocore/eventsystem/I_ProxyAllocator.h  |  4 +--
 iocore/eventsystem/P_UnixEThread.h |  2 +-
 iocore/eventsystem/UnixEThread.cc  |  2 +-
 iocore/net/ALPNSupport.cc  |  2 +-
 iocore/net/I_NetProcessor.h|  2 +-
 iocore/net/P_Connection.h  |  2 +-
 iocore/net/P_SNIActionPerformer.h  |  2 +-
 iocore/net/P_UnixNet.h |  2 +-
 iocore/net/P_UnixNetProcessor.h|  2 +-
 iocore/net/QUICMultiCertConfigLoader.cc|  2 +-
 iocore/net/QUICNetVConnection.cc   |  2 +-
 iocore/net/SSLConfig.cc|  2 +-
 iocore/net/SSLSNIConfig.cc |  2 +-
 iocore/net/SSLSessionTicket.cc |  4 +--
 iocore/net/SSLUtils.cc |  6 ++---
 iocore/net/TLSBasicSupport.cc  |  2 +-
 iocore/net/TLSSNISupport.cc|  2 +-
 iocore/net/TLSSessionResumptionSupport.cc  |  2 +-
 iocore/net/quic/QUICContext.h  |  2 +-
 lib/perl/lib/Apache/TS/Config/Records.pm   |  2 +-
 lib/records/RecUtils.cc|  6 ++---
 lib/yamlcpp/src/scanner.cpp|  2 +-
 mgmt/RecordsConfig.cc  |  2 +-
 mgmt/WebMgmtUtils.cc   | 14 +-
 mgmt/YamlCfg.h |  2 +-
 mgmt/api/TSControlMain.cc  |  2 +-
 plugins/cache_promote/cache_promote.cc |  2 +-
 plugins/cache_promote/lru_policy.cc|  4 +--
 plugins/cachekey/cachekey.cc   |  4 +--
 plugins/certifier/certifier.cc |  2 +-
 plugins/compress/compress.cc   |  2 +-
 .../access_control/unit_tests/test_utils.cc|  2 +-
 plugins/experimental/cookie_remap/cookie_remap.cc  |  4 +--
 plugins/experimental/cookie_remap/strip.h  |  2 +-
 .../experimental/fastcgi/src/ats_fcgi_client.cc|  8 +++---
 plugins/experimental/fastcgi/src/configuru.hpp | 14 +-
 .../experimental/fastcgi/src/server_connection.cc  |  2 +-
 plugins/experimental/icap/icap_plugin.cc   |  2 +-
 plugins/experimental/magick/README |  2 +-
 plugins/experimental/maxmind_acl/mmdb.cc   |  2 +-
 plugins/experimental/mysql_remap/lib/dictionary.c  |  4 +--
 plugins/experimental/mysql_remap/lib/dictionary.h  |  4 +--
 .../experimental/parent_select/consistenthash.cc   |  4 +--
 .../parent_select/consistenthash_config.cc |  2 +-
 plugins/experimental/slice/Config.h|  2 +-
 plugins/experimental/slice/README.md   |  2 +-
 .../experimental/slice/unit-tests/slice_test.cc|  2 +-
 .../experimental/ssl_session_reuse

[trafficserver] branch master updated (ca22b29 -> b200460)

2021-07-15 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from ca22b29  Fix bug in atscppapi::TranformationPluginState constructor. 
(#8063)
 add b200460  Updates to webp_transform to convert webp to jpeg (#8005)

No new revisions were added by this update.

Summary of changes:
 doc/admin-guide/plugins/webp_transform.en.rst  |   7 +-
 .../experimental/webp_transform/ImageTransform.cc  | 148 ++---
 plugins/experimental/webp_transform/Makefile.inc   |  12 +-
 3 files changed, 138 insertions(+), 29 deletions(-)


[trafficserver] branch master updated: Remove check_system_constants(), which is unused (#8064)

2021-07-14 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 73d9b35  Remove check_system_constants(), which is unused (#8064)
73d9b35 is described below

commit 73d9b355fc28096cfd753cee0ab1473a2e1630b3
Author: Masato Gosui <82705154+nekomachi-to...@users.noreply.github.com>
AuthorDate: Thu Jul 15 03:21:00 2021 +0900

Remove check_system_constants(), which is unused (#8064)
---
 src/traffic_server/traffic_server.cc | 8 
 1 file changed, 8 deletions(-)

diff --git a/src/traffic_server/traffic_server.cc 
b/src/traffic_server/traffic_server.cc
index 9cf0ff7..c8490b0 100644
--- a/src/traffic_server/traffic_server.cc
+++ b/src/traffic_server/traffic_server.cc
@@ -1522,11 +1522,6 @@ syslog_log_configure()
 }
 
 static void
-check_system_constants()
-{
-}
-
-static void
 init_http_header()
 {
   url_init();
@@ -1744,9 +1739,6 @@ main(int /* argc ATS_UNUSED */, const char **argv)
   pcre_malloc = ats_malloc;
   pcre_free   = ats_free;
 
-  // Verify system dependent 'constants'
-  check_system_constants();
-
   // Define the version info
   appVersionInfo.setup(PACKAGE_NAME, "traffic_server", PACKAGE_VERSION, 
__DATE__, __TIME__, BUILD_MACHINE, BUILD_PERSON, "");
 


[trafficserver] branch webp_to_jpeg updated (5824310 -> 91ec775)

2021-07-14 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 5824310  Ran clang-format over the code
 add 91ec775  Change separate if statements to a switch

No new revisions were added by this update.

Summary of changes:
 plugins/experimental/webp_transform/ImageTransform.cc | 16 +++-
 1 file changed, 11 insertions(+), 5 deletions(-)


[trafficserver] branch master updated: Fixed spelling mistakes in docs (#8060)

2021-07-13 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 1c20be0  Fixed spelling mistakes in docs (#8060)
1c20be0 is described below

commit 1c20be0767020e493f04131a60ebd8b738f739ed
Author: Bryan Call 
AuthorDate: Tue Jul 13 14:21:42 2021 -0700

Fixed spelling mistakes in docs (#8060)
---
 doc/admin-guide/plugins/regex_revalidate.en.rst  |  2 +-
 doc/dot/SimpleStateDiag.dot  |  2 +-
 doc/dot/SimpleStateDiagAPI.dot   |  2 +-
 .../LC_MESSAGES/admin-guide/files/records.config.en.po   | 16 
 .../ja/LC_MESSAGES/admin-guide/files/remap.config.en.po  |  6 +++---
 .../LC_MESSAGES/admin-guide/files/storage.config.en.po   |  2 +-
 .../ja/LC_MESSAGES/admin-guide/installation/index.en.po  |  4 ++--
 .../admin-guide/monitoring/logging/log-formats.en.po |  2 +-
 .../ja/LC_MESSAGES/admin-guide/performance/index.en.po   |  4 ++--
 .../ja/LC_MESSAGES/admin-guide/plugins/cachekey.en.po|  2 +-
 doc/locale/ja/LC_MESSAGES/admin-guide/plugins/esi.en.po  |  2 +-
 .../ja/LC_MESSAGES/admin-guide/plugins/generator.en.po   |  2 +-
 .../ja/LC_MESSAGES/admin-guide/plugins/geoip_acl.en.po   |  2 +-
 .../ja/LC_MESSAGES/admin-guide/plugins/sslheaders.en.po  |  4 ++--
 .../ja/LC_MESSAGES/admin-guide/plugins/ts_lua.en.po  | 10 +-
 .../ja/LC_MESSAGES/admin-guide/plugins/url_sig.en.po |  2 +-
 .../ja/LC_MESSAGES/admin-guide/security/index.en.po  |  4 ++--
 doc/locale/ja/LC_MESSAGES/appendices/glossary.en.po  |  2 +-
 .../developer-guide/api/functions/TSCacheRemove.en.po|  2 +-
 .../api/functions/TSHttpConnectWithPluginId.en.po|  2 +-
 .../api/functions/TSHttpOverridableConfig.en.po  |  2 +-
 .../api/functions/TSHttpTxnErrorBodySet.en.po|  2 +-
 .../api/functions/TSLifecycleHookAdd.en.po   |  2 +-
 .../api/functions/TSSslContextFindBy.en.po   |  2 +-
 .../developer-guide/api/functions/TSVConnReenable.en.po  |  2 +-
 .../developer-guide/architecture/architecture.en.po  |  6 +++---
 .../developer-guide/architecture/data-structures.en.po   |  2 +-
 .../developer-guide/architecture/tiered-storage.en.po|  2 +-
 .../ja/LC_MESSAGES/developer-guide/config-vars.en.po |  4 ++--
 .../developer-guide/documentation/building.en.po |  2 +-
 .../developer-guide/documentation/conventions.en.po  |  4 ++--
 .../developer-guide/documentation/ts-markup.en.po|  2 +-
 .../developer-guide/host-resolution-proposal.en.po   |  2 +-
 .../developer-guide/plugins/adding-statistics.en.po  |  2 +-
 .../hooks-and-transactions/trafficserver-timers.en.po|  2 +-
 .../plugins/http-headers/mime-headers.en.po  |  2 +-
 .../sample-null-transformation-plugin.en.po  |  2 +-
 .../ja/LC_MESSAGES/developer-guide/plugins/mutexes.en.po |  4 ++--
 .../developer-guide/release-process/index.en.po  |  4 ++--
 .../developer-guide/testing-with-vagrant/index.en.po |  2 +-
 doc/locale/ja/LC_MESSAGES/getting-started.en.po  |  2 +-
 doc/locale/ja/LC_MESSAGES/sdk/trafficserver-timers.en.po |  2 +-
 42 files changed, 65 insertions(+), 65 deletions(-)

diff --git a/doc/admin-guide/plugins/regex_revalidate.en.rst 
b/doc/admin-guide/plugins/regex_revalidate.en.rst
index d122134..3c2fb66 100644
--- a/doc/admin-guide/plugins/regex_revalidate.en.rst
+++ b/doc/admin-guide/plugins/regex_revalidate.en.rst
@@ -76,7 +76,7 @@ The configuration parameter `--state-file` or `-f` may be 
used to configure
 the plugin to maintain a state file with the last loaded configuration.
 Normally when ATS restarts the epoch times of all rules are reset to
 the first config file load time which will cause all matching assets to
-issue new IMS requests to their parents for mathing rules.
+issue new IMS requests to their parents for matching rules.
 
 This option allows the revalidate rule "epoch" times to be retained between ATS
 restarts.  This state file by default is placed in var/trafficserver/
diff --git a/doc/dot/SimpleStateDiag.dot b/doc/dot/SimpleStateDiag.dot
index 0a49b69..570a0fc 100644
--- a/doc/dot/SimpleStateDiag.dot
+++ b/doc/dot/SimpleStateDiag.dot
@@ -56,7 +56,7 @@ orientation = "portrait";
 "VALID" -> "PICK_ADDR" [ label = "no" ];
 "VALID" -> "SETUP_S_READ" [ label = "yes" ];
 "SETUP_S_READ" -> "SETUP_TRANS" [ label = "Uncachable" ];
-"SETUP_S_READ" -> "SETUP_CACHE_WRITE" [ label = "Cachable" ];
+"SETUP_S_READ" -> "SETUP_CACHE_WRITE" [ label = "Cacheable" ];
 "SETUP_CACHE_WRITE" -> "SETUP_TRANS";
 "SETUP_TRANS" -> "TUNNEL";
 &

[trafficserver] branch webp_to_jpeg updated (c7aea76 -> 5824310)

2021-06-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from c7aea76  Revert back to original encoding when conversion fails (#7997)
 add 5824310  Ran clang-format over the code

No new revisions were added by this update.

Summary of changes:
 plugins/experimental/webp_transform/ImageTransform.cc | 15 ---
 1 file changed, 8 insertions(+), 7 deletions(-)


[trafficserver] branch webp_to_jpeg updated (792d267 -> c7aea76)

2021-06-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


 discard 792d267  Revert back to original encoding when conversion fails (#7997)
omit 48b495b  Display more information on exception
omit 5c0558f  Updated to handle ImageMagick++ exceptions
omit 9a0f785  Updates to webp_transform to convert webp to jpeg
 add fef47d7  Increment ssl_error_syscall only if not EOF (#7225)
 add 2950a71  New option to dead server to not retry during dead period 
(#7142)
 add 73dd46c  Revert "Create an explicit runroot.yaml for AuTests (#7177)" 
(#7235)
 add e97bc75  clean up body factory tests (#7236)
 add 788224f   Do not cache Transfer-Encoding header (#7234)
 add ab37864  Stop crash on disk failure (#7218)
 add 10aabb1  Ensure that ca override does not get lost (#7219)
 add cdc0646  Make double Au test more reliable. (#7216)
 add 75b0fb0  RolledLogDeleter: do not sort on each candidate 
consideration. (#7243)
 add 3796e88  Fix for plugins ASAN suppression file (#7249)
 add 556d18f  Add support for server protocol stack API (#7239)
 add eea8c55  Incorporates the latest CI build changes (#7251)
 add cb4ff10  [multiplexer] option to skip post/put requests (#7233)
 add ef29798  Fix test_error_page_selection memory leaks and logic errors 
(#7248)
 add e6e6ca2  Remove useless if for port set assertion. (#7250)
 add c83f7e1  Remove some usless defines, which just obsfucates code (#7252)
 add 718bef4  Treat objects with negative max-age CC directives as stale. 
(#7260)
 add fb0bf03  Bugfix: set a default inactivity timeout only if a read or 
write I/O operation was set (#7226)
 add 439e317  HostDB: Fix cache data version checking to use full version, 
not major version. (#7263)
 add bea630e  Let Dedicated EThreads use `EThread::schedule` (#7228)
 add 73d39b1  Document external log rotation support via SIGUSR2 (#7265)
 add aa16a29  Allow initial // in request targets. (#7266)
 add 36edf3c  Removes commented out code from esi plugin (#7273)
 add 4ffd34a  gracefully handle TSReleaseAsserts in statichit and generator 
plugins (#7269)
 add bb847ee  Respecting default rolling_enabled in plugins. (#7275)
 add a2ee1af  url_sig add 'ignore_expiry = true' option for log replay 
testing (#7231)
 add 2522933  Fix truncated reponse on HTTP/2 graceful shutdown (#7267)
 add 9499ca9  Add AuTest for HTTP/2 Graceful Shutdown (#7271)
 add 233f991  Fix proxy.process.http.current_client_transactions (#7258)
 add f033304  Fix example in default sni.yaml configuration. (#7277)
 add 82966c6  Add support for a new (TSMgmtDataTypeGet) mgmt API function 
to retrieve the record data type (#7221)
 add afefa73  HostDB: remove unused field in HostDBApplicationInfo, and 
update remaining types in http_data to fix broken padding. (#7264)
 add a2d1515  7096: Synchronize Server Session Management and Network I/O 
(#7278)
 add d285211  Sticky server does not work with H2 client (#7261)
 add 786463b  Fix bad HTTP/2 post client causing stuck HttpSM (#7237)
 add bf45adf  Enable all h2spec test (#7289)
 add e65cf23  Allow disabling SO_MARK and IP_TOS usage (#7292)
 add dfa7cf1  Remove unfinished h2c support (#7286)
 add 0c88b24  Adds a shell script to help build the H3 toolchains (#7299)
 add 7bef8ca  Reduce the number of write operation on H2 (#7282)
 add 89dccb5  Follow the comments in I_Thread.h, add an independent 
ink_thread_key for EThread. (#6288)
 add a52bd12  s3_auth: demote noisy errors around configuration that 
doesn't affect plugin usability (#7306)
 add 1765c9f  Get appropriate locks on SSN_START hook delays (#7295)
 add e359ac3  Add option for hybrid global and thread session pools (#6978)
 add ef18164  fixup in HttpSM to only set [TS_MILESTONE_SERVER_CLOSE if 
TS_MILESTONE_SERVER_CONNECT has been set (#7259)
 add b71249c  Updates the Dockerfile with more packages (#7323)
 add b21ecc8  Remove unnecessary cast from ReverseProxy. (#7329)
 add 31d2284  Remove the last remnants of the enable_url_expandomatic 
(#7276)
 add 97354bf  TLS Session Reuse: Downgrade noisy log to debug (#7344)
 add 01bd0f8  TLS Session Reuse: Downgrade add_session messages to debug 
(#7345)
 add 526952f  Fix vc close migration race condition (#7337)
 add 3f27d75  AuTest for incoming PROXY Protocol v1 (#7326)
 add 2c9d4c7  Add note to background fetch about include/exclude (#7343)
 add 4e2ac3b  Fix lookup split dns rule with fast path (#7320)
 add 3f11f15  Set thread mutex to the DNSHandler mutex of SplitDNS (#7321)
 add fee5ba1  Allow for regex_remap of pristine URL. (#7347)
 add 3113501  ESI: Ensure gzip header is always initialized (#7360)
 add 8eb6826  Add negative caching tests and fixes. (#7361

[trafficserver] branch webp_to_jpeg updated: Revert back to original encoding when conversion fails (#7997)

2021-06-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/webp_to_jpeg by this push:
 new 792d267  Revert back to original encoding when conversion fails (#7997)
792d267 is described below

commit 792d2675d2c18dd6cae54537c56f903b4bf745c5
Author: constreference <75616549+constrefere...@users.noreply.github.com>
AuthorDate: Mon Jun 28 21:44:47 2021 +0530

Revert back to original encoding when conversion fails (#7997)
---
 .../experimental/webp_transform/ImageTransform.cc  | 63 ++
 1 file changed, 42 insertions(+), 21 deletions(-)

diff --git a/plugins/experimental/webp_transform/ImageTransform.cc 
b/plugins/experimental/webp_transform/ImageTransform.cc
index a14b9bc..1c37045 100644
--- a/plugins/experimental/webp_transform/ImageTransform.cc
+++ b/plugins/experimental/webp_transform/ImageTransform.cc
@@ -36,7 +36,7 @@ namespace
 {
 GlobalPlugin *plugin;
 
-enum class TransformImageType { webp, jpeg };
+enum class ImageEncoding { webp, jpeg, png, unknown };
 
 bool config_convert_to_webp = false;
 bool config_convert_to_jpeg = false;
@@ -48,8 +48,8 @@ Stat stat_convert_to_jpeg;
 class ImageTransform : public TransformationPlugin
 {
 public:
-  ImageTransform(Transaction , TransformImageType image_type)
-: TransformationPlugin(transaction, 
TransformationPlugin::RESPONSE_TRANSFORMATION), _image_type(image_type)
+  ImageTransform(Transaction , ImageEncoding input_image_type, 
ImageEncoding transform_image_type)
+: TransformationPlugin(transaction, 
TransformationPlugin::RESPONSE_TRANSFORMATION), 
_input_image_type(input_image_type), _transform_image_type(transform_image_type)
   {
 TransformationPlugin::registerHook(HOOK_READ_RESPONSE_HEADERS);
   }
@@ -57,12 +57,16 @@ public:
   void
   handleReadResponseHeaders(Transaction ) override
   {
-if (_image_type == TransformImageType::webp) {
+if (_transform_image_type == ImageEncoding::webp) {
   transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/webp";
-} else {
+} 
+if (_transform_image_type == ImageEncoding::jpeg) {
   transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/jpeg";
 }
-transaction.getServerResponse().getHeaders()["Vary"] = "Accpet"; // to 
have a separate cache entry
+if (_transform_image_type == ImageEncoding::png) {
+  transaction.getServerResponse().getHeaders()["Content-Type"] = 
"image/png";
+}
+transaction.getServerResponse().getHeaders()["Vary"] = "Accept"; // to 
have a separate cache entry
 
 TS_DEBUG(TAG, "url %s", 
transaction.getServerRequest().getUrl().getUrlString().c_str());
 transaction.resume();
@@ -85,13 +89,13 @@ public:
   image.read(input_blob);
 
   Blob output_blob;
-  if (_image_type == TransformImageType::webp) {
+  if (_transform_image_type == ImageEncoding::webp) {
 stat_convert_to_webp.increment(1);
 TSDebug(TAG, "Transforming jpeg or png to webp");
 image.magick("WEBP");
   } else {
 stat_convert_to_jpeg.increment(1);
-TSDebug(TAG, "Transforming wepb to jpeg");
+TSDebug(TAG, "Transforming webp to jpeg");
 image.magick("JPEG");
   }
   image.write(_blob);
@@ -99,10 +103,12 @@ public:
 } catch (Magick::Warning ) {
   TSError("ImageMagick++ warning: %s", warning.what());
   produce(std::string_view(reinterpret_cast(input_blob.data()), input_blob.length()));
+  _transform_image_type = _input_image_type; // Revert to original 
encoding on error
 } catch (Magick::Error ) {
-  TSError("ImageMagick++ error: %s _image_type: %d input_data.length(): 
%zd", error.what(), (int)_image_type,
+  TSError("ImageMagick++ error: %s _image_type: %d input_data.length(): 
%zd", error.what(), (int)_transform_image_type,
   input_data.length());
   produce(std::string_view(reinterpret_cast(input_blob.data()), input_blob.length()));
+  _transform_image_type = _input_image_type; // Revert to original 
encoding on error
 }
 
 setOutputComplete();
@@ -112,7 +118,8 @@ public:
 
 private:
   std::stringstream _img;
-  TransformImageType _image_type;
+  ImageEncoding _input_image_type;
+  ImageEncoding _transform_image_type; 
 };
 
 class GlobalHookPlugin : public GlobalPlugin
@@ -122,37 +129,51 @@ public:
   void
   handleReadResponseHeaders(Transaction ) override
   {
+
+// This variable stores the incoming image type
+ImageEncoding input_image_type = ImageEncoding::unknown;
+
 // This method tries to optimize the amount of string searching at the 
expense of double checking some of the booleans
 

[trafficserver] branch master updated (d7a0b1d -> e44ca80)

2021-06-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from d7a0b1d  Enforce case for well known methods (#7886)
 add e44ca80  Update TSHttpTxnAborted API to distinguish client/server 
aborts (#7901)

No new revisions were added by this update.

Summary of changes:
 doc/developer-guide/api/functions/TSHttpTxnAborted.en.rst | 6 +-
 include/ts/ts.h   | 3 ++-
 plugins/esi/esi.cc| 3 ++-
 plugins/lua/ts_lua_http.c | 4 ++--
 src/traffic_server/InkAPI.cc  | 6 --
 5 files changed, 15 insertions(+), 7 deletions(-)


[trafficserver] branch master updated (ecebfba -> 51516e9)

2021-06-25 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from ecebfba  Allow to TLS handshake to error out on TSVConnReenable (#7994)
 add 51516e9  limit m_current_range to max value in RangeTransform (#4843)

No new revisions were added by this update.

Summary of changes:
 proxy/Transform.cc | 4 
 1 file changed, 4 insertions(+)


[trafficserver] branch master updated (7dbb6cb -> 66ebf92)

2021-06-25 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 7dbb6cb  Add hook for loading certificate and key data from plugin  
(#6609)
 add 66ebf92  Cleanup: Get rid of HTTP2_SESSION_EVENT_INIT (#7878)

No new revisions were added by this update.

Summary of changes:
 proxy/http2/Http2ClientSession.cc   |  4 +--
 proxy/http2/Http2ConnectionState.cc | 56 ++---
 proxy/http2/Http2ConnectionState.h  |  3 +-
 3 files changed, 32 insertions(+), 31 deletions(-)


[trafficserver] branch master updated (bd93f2a -> d455509)

2021-06-25 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from bd93f2a  Don't rely on SSLNetVC when HttpSM gathers info about SSL 
(#7961)
 add d455509  Fix typo in configure.ac (#7993)

No new revisions were added by this update.

Summary of changes:
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


[trafficserver-site] branch asf-site updated: 9.0.2 and 8.1.2 release information

2021-06-24 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/asf-site by this push:
 new c6fabc0  9.0.2 and 8.1.2 release information
c6fabc0 is described below

commit c6fabc0d4b4019e95512cfa30f578cbc2f1301b5
Author: Bryan Call 
AuthorDate: Thu Jun 24 14:58:23 2021 -0700

9.0.2 and 8.1.2 release information
---
 source/markdown/downloads.mdtext | 36 +-
 source/markdown/index.html   | 56 +---
 2 files changed, 24 insertions(+), 68 deletions(-)

diff --git a/source/markdown/downloads.mdtext b/source/markdown/downloads.mdtext
index 242680a..8cbf9ac 100644
--- a/source/markdown/downloads.mdtext
+++ b/source/markdown/downloads.mdtext
@@ -19,9 +19,9 @@ RSS:   /rss/releases.rss
 
 
 
-The latest stable release of Apache Traffic Server is v9.0.1, released on 
2021-04-16.
-In addition, we continue to support the v8.1.x LTS release train, currently 
v8.1.1,
-which was released on 2020-12-02. We follow the [Semantic 
Versioning](http://semver.org)
+The latest stable release of Apache Traffic Server is v9.0.2, released on 
2021-06-24.
+In addition, we continue to support the v8.1.x LTS release train, currently 
v8.1.2,
+which was released on 2021-06-24. We follow the [Semantic 
Versioning](http://semver.org)
 scheme. The goal is to release patch releases frequently, and minor releases 
as needed.
 Within the major versions, all such patch and minor releases are all 
compatible.
 
@@ -32,33 +32,33 @@ will be needed.  You can also
 [browse through all releases](https://archive.apache.org/dist/trafficserver/)
 and hash signatures.
 
-# Current v9.x Release -- 9.0.1 # {#9.0.1}
+# Current v9.x Release -- 9.0.2 # {#9.0.2}
 
- Apache Traffic Server v9.0.1 was released on April 16th, 2021.
- 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.1.tar.bz2.asc)]
- 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.1.tar.bz2.sha512)]
+ Apache Traffic Server v9.0.2 was released on June 24th, 2021.
+ 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.2.tar.bz2.asc)]
+ 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-9.0.2.tar.bz2.sha512)]
 
- https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-9.0.1.tar.bz2;
- class="download_ts">Traffic Server 9.0.1
+ https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-9.0.2.tar.bz2;
+ class="download_ts">Traffic Server 9.0.2
 
-v9.0.1 is our latest stable release. Additional details for this release are 
in the
-[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.0.1)
+v9.0.2 is our latest stable release. Additional details for this release are 
in the
+[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/9.0.x/CHANGELOG-9.0.2)
 and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/31?closed=1).
 
 For details on the v9.0.x release, please see
 [9.0.x 
News](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/whats-new.en.html).
 There are also
 details about [upgrading to 
9.x](https://docs.trafficserver.apache.org/en/9.0.x/release-notes/upgrading.en.html).
 
-# Current v8.x Release -- 8.1.1 # {#8.1.1}
+# Current v8.x Release -- 8.1.2 # {#8.1.2}
 
- Apache Traffic Server v8.1.1 was released on December 2nd, 2020.
- 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-8.1.1.tar.bz2.asc)]
- 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-8.1.1.tar.bz2.sha512)]
+ Apache Traffic Server v8.1.2 was released on June 24th, 2020.
+ 
[[`PGP`](https://www.apache.org/dist/trafficserver/trafficserver-8.1.2.tar.bz2.asc)]
+ 
[[`SHA512`](https://www.apache.org/dist/trafficserver/trafficserver-8.1.2.tar.bz2.sha512)]
 
- https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-8.1.1.tar.bz2;
 class="download_ts">Traffic Server 8.1.1
+ https://www.apache.org/dyn/closer.cgi/trafficserver/trafficserver-8.1.2.tar.bz2;
 class="download_ts">Traffic Server 8.1.2
 
-v8.1.1 is our current stable release in the previous release train. Additional 
details for this release are in the
-[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/8.1.x/CHANGELOG-8.1.1)
+v8.1.2 is our current stable release in the previous release train. Additional 
details for this release are in the
+[CHANGELOG](https://raw.githubusercontent.com/apache/trafficserver/8.1.x/CHANGELOG-8.1.2)
 and the the related [Github Issues and 
PRs](https://github.com/apache/trafficserver/milestone/44?closed=1).
 
 For details on the v8.0.x release, please see
diff --git a/source/markdown/index.html b/source/markdown/index.html
index 9301cb8..3aa236c 100644
--- a/source/markdown/index.html

[trafficserver] branch master updated (6c1780c -> 9e7cece)

2021-06-22 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 6c1780c  ESI plugin documentation updates. (#7970)
 add 9e7cece  Thread safe Mersenne Twister 64 using c++11 (#7859)

No new revisions were added by this update.

Summary of changes:
 include/tscore/{defalloc.h => Random.h}| 34 ++
 src/tscore/Makefile.am |  4 +-
 .../inliner/util.h => src/tscore/Random.cc | 12 +++--
 src/tscore/unit_tests/test_Random.cc   | 52 ++
 4 files changed, 85 insertions(+), 17 deletions(-)
 copy include/tscore/{defalloc.h => Random.h} (65%)
 copy plugins/experimental/inliner/util.h => src/tscore/Random.cc (83%)
 create mode 100644 src/tscore/unit_tests/test_Random.cc


[trafficserver] branch master updated (dbc4f8e -> a2ee5ab)

2021-06-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from dbc4f8e  Note YAML parser library bug, and work-around, in 
documentation. (#7963)
 add a2ee5ab  Fixed compile error with Linux AIO unit test (#7958)

No new revisions were added by this update.

Summary of changes:
 iocore/aio/test_AIO.cc | 9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)


[trafficserver] branch master updated (2b13eb3 -> 668d0f8)

2021-06-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 2b13eb3  String the url fragment for outgoing requests (#7966)
 add 668d0f8  Ensure that the content-length value is only digits (#7964)

No new revisions were added by this update.

Summary of changes:
 proxy/hdrs/HTTP.cc | 11 +++
 1 file changed, 11 insertions(+)


[trafficserver] branch master updated (034965e -> 2b13eb3)

2021-06-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 034965e  Fix for HTTP/2 frames (#7965)
 add 2b13eb3  String the url fragment for outgoing requests (#7966)

No new revisions were added by this update.

Summary of changes:
 proxy/http/HttpTransact.cc | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)


[trafficserver] branch master updated (cdb17a7 -> 034965e)

2021-06-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from cdb17a7  Improve parsing error messages for strategies.yaml. (#7948)
 add 034965e  Fix for HTTP/2 frames (#7965)

No new revisions were added by this update.

Summary of changes:
 proxy/http2/Http2ClientSession.cc | 14 +++---
 1 file changed, 7 insertions(+), 7 deletions(-)


[trafficserver] branch master updated: Fixed memory leak in the QUIC stream manager (#7951)

2021-06-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 bf3fa57  Fixed memory leak in the QUIC stream manager (#7951)
bf3fa57 is described below

commit bf3fa57a4ff5c291aff06ef688c909e80f45b9b5
Author: Bryan Call 
AuthorDate: Fri Jun 18 16:04:04 2021 -0700

Fixed memory leak in the QUIC stream manager (#7951)

* Fixed memory leak in the QUIC stream manager.
Removed the proxy allocator for the streams.  They can be added
back in if there is a performance issue.
---
 iocore/eventsystem/I_Thread.h|  3 ---
 iocore/net/quic/QUICStreamFactory.cc | 36 
 iocore/net/quic/QUICStreamManager.cc |  7 +++
 iocore/net/quic/QUICStreamManager.h  |  1 +
 4 files changed, 12 insertions(+), 35 deletions(-)

diff --git a/iocore/eventsystem/I_Thread.h b/iocore/eventsystem/I_Thread.h
index f2f1b1c..7ec7c3e 100644
--- a/iocore/eventsystem/I_Thread.h
+++ b/iocore/eventsystem/I_Thread.h
@@ -123,9 +123,6 @@ public:
   ProxyAllocator http2ClientSessionAllocator;
   ProxyAllocator http2StreamAllocator;
   ProxyAllocator quicClientSessionAllocator;
-  ProxyAllocator quicBidiStreamAllocator;
-  ProxyAllocator quicSendStreamAllocator;
-  ProxyAllocator quicReceiveStreamAllocator;
   ProxyAllocator httpServerSessionAllocator;
   ProxyAllocator hdrHeapAllocator;
   ProxyAllocator strHeapAllocator;
diff --git a/iocore/net/quic/QUICStreamFactory.cc 
b/iocore/net/quic/QUICStreamFactory.cc
index 9762895..548be10 100644
--- a/iocore/net/quic/QUICStreamFactory.cc
+++ b/iocore/net/quic/QUICStreamFactory.cc
@@ -26,30 +26,20 @@
 #include "QUICUnidirectionalStream.h"
 #include "QUICStreamFactory.h"
 
-ClassAllocator 
quicBidiStreamAllocator("quicBidiStreamAllocator");
-ClassAllocator 
quicSendStreamAllocator("quicSendStreamAllocator");
-ClassAllocator 
quicReceiveStreamAllocator("quicReceiveStreamAllocator");
-
 QUICStreamVConnection *
 QUICStreamFactory::create(QUICStreamId sid, uint64_t local_max_stream_data, 
uint64_t remote_max_stream_data)
 {
   QUICStreamVConnection *stream = nullptr;
   switch (QUICTypeUtil::detect_stream_direction(sid, 
this->_info->direction())) {
   case QUICStreamDirection::BIDIRECTIONAL:
-// TODO Free the stream somewhere
-stream = THREAD_ALLOC(quicBidiStreamAllocator, this_ethread());
-new (stream) QUICBidirectionalStream(this->_rtt_provider, this->_info, 
sid, local_max_stream_data, remote_max_stream_data);
+stream = new QUICBidirectionalStream(this->_rtt_provider, this->_info, 
sid, local_max_stream_data, remote_max_stream_data);
 break;
   case QUICStreamDirection::SEND:
-// TODO Free the stream somewhere
-stream = THREAD_ALLOC(quicSendStreamAllocator, this_ethread());
-new (stream) QUICSendStream(this->_info, sid, remote_max_stream_data);
+stream = new QUICSendStream(this->_info, sid, remote_max_stream_data);
 break;
   case QUICStreamDirection::RECEIVE:
 // server side
-// TODO Free the stream somewhere
-stream = THREAD_ALLOC(quicReceiveStreamAllocator, this_ethread());
-new (stream) QUICReceiveStream(this->_rtt_provider, this->_info, sid, 
local_max_stream_data);
+stream = new QUICReceiveStream(this->_rtt_provider, this->_info, sid, 
local_max_stream_data);
 break;
   default:
 ink_assert(false);
@@ -62,23 +52,5 @@ QUICStreamFactory::create(QUICStreamId sid, uint64_t 
local_max_stream_data, uint
 void
 QUICStreamFactory::delete_stream(QUICStreamVConnection *stream)
 {
-  if (!stream) {
-return;
-  }
-
-  stream->~QUICStreamVConnection();
-  switch (stream->direction()) {
-  case QUICStreamDirection::BIDIRECTIONAL:
-THREAD_FREE(static_cast(stream), 
quicBidiStreamAllocator, this_thread());
-break;
-  case QUICStreamDirection::SEND:
-THREAD_FREE(static_cast(stream), 
quicSendStreamAllocator, this_thread());
-break;
-  case QUICStreamDirection::RECEIVE:
-THREAD_FREE(static_cast(stream), 
quicReceiveStreamAllocator, this_thread());
-break;
-  default:
-ink_assert(false);
-break;
-  }
+  delete stream;
 }
diff --git a/iocore/net/quic/QUICStreamManager.cc 
b/iocore/net/quic/QUICStreamManager.cc
index da66d74..54a6d20 100644
--- a/iocore/net/quic/QUICStreamManager.cc
+++ b/iocore/net/quic/QUICStreamManager.cc
@@ -41,6 +41,13 @@ QUICStreamManager::QUICStreamManager(QUICContext *context, 
QUICApplicationMap *a
   }
 }
 
+QUICStreamManager::~QUICStreamManager()
+{
+  for (auto stream = stream_list.pop(); stream != nullptr; stream = 
stream_list.pop()) {
+_stream_factory.delete_stream(stream);
+  }
+}
+
 std::vector
 QUICStreamManager::interests()
 {
diff --git a/iocore/net/quic/QUICStreamManager.h 
b/iocore/net/quic/QUICStreamManager.h
index 05da5fc..f15803

[trafficserver] branch master updated (e492bd3 -> 0ff9631)

2021-06-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from e492bd3  Update GitHub stale action to auto close old PRs (#7952)
 add 0ff9631  Fixup TS_USE_LINUX_NATIVE_AIO AIO_MODE_NATIVE (#7832)

No new revisions were added by this update.

Summary of changes:
 iocore/aio/P_AIO.h|  1 +
 iocore/cache/Cache.cc | 11 +++
 2 files changed, 4 insertions(+), 8 deletions(-)


[trafficserver] branch master updated: Update GitHub stale action to auto close old PRs (#7952)

2021-06-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 e492bd3  Update GitHub stale action to auto close old PRs (#7952)
e492bd3 is described below

commit e492bd3fe8346b40aac9bfe605c3921898549322
Author: Bryan Call 
AuthorDate: Fri Jun 18 11:35:54 2021 -0700

Update GitHub stale action to auto close old PRs (#7952)
---
 .github/workflows/stale.yml | 11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml
index d7d06d8..6ba6a2d 100644
--- a/.github/workflows/stale.yml
+++ b/.github/workflows/stale.yml
@@ -1,5 +1,8 @@
 name: Mark stale issues and pull requests
 
+# Documentation
+# https://github.com/marketplace/actions/close-stale-issues
+
 on:
   push:
 paths:
@@ -9,9 +12,7 @@ on:
 
 jobs:
   stale:
-
 runs-on: ubuntu-latest
-
 steps:
 - uses: actions/stale@v3
   with:
@@ -20,8 +21,10 @@ jobs:
 stale-pr-message: 'This pull request has been automatically marked as 
stale because it has not had recent activity. Marking it stale to flag it for 
further consideration by the community.'
 stale-issue-label: 'Stale'
 stale-pr-label: 'Stale'
+exempt-issue-labels: 'In Progress'
+exempt-pr-labels: 'In Progress'
 days-before-pr-stale: 90
 days-before-issue-stale: 365
-days-before-pr-close: -1
+days-before-pr-close: 7
 days-before-issue-close: -1
-debug-only: true
+debug-only: false


[trafficserver] branch master updated (d0dda3f -> e01b5b3)

2021-06-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from d0dda3f  cache_promote: Don't promote on uncacheable requests (#7942)
 add e01b5b3  Fixed memory leak in QUIC ack frame unit test (#7947)

No new revisions were added by this update.

Summary of changes:
 iocore/net/quic/test/test_QUICAckFrameCreator.cc | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)


[trafficserver] branch master updated: Fixed spelling mistakes in the docs (#7896)

2021-05-28 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 35994e4  Fixed spelling mistakes in the docs (#7896)
35994e4 is described below

commit 35994e4b81f3d2e7b314753532a64ae5127bcadf
Author: Bryan Call 
AuthorDate: Fri May 28 13:58:25 2021 -0700

Fixed spelling mistakes in the docs (#7896)
---
 doc/admin-guide/files/records.config.en.rst   | 2 +-
 doc/admin-guide/files/remap.config.en.rst | 4 ++--
 doc/admin-guide/plugins/background_fetch.en.rst   | 2 +-
 doc/admin-guide/plugins/cachekey.en.rst   | 2 +-
 doc/admin-guide/plugins/geoip_acl.en.rst  | 2 +-
 doc/admin-guide/plugins/header_rewrite.en.rst | 2 +-
 doc/appendices/command-line/traffic_top.en.rst| 4 ++--
 doc/appendices/glossary.en.rst| 2 +-
 doc/developer-guide/api/functions/TSClientProtocolStack.en.rst| 2 +-
 doc/developer-guide/api/functions/TSHttpTxnReenable.en.rst| 4 ++--
 doc/developer-guide/api/functions/TSLifecycleHookAdd.en.rst   | 2 +-
 doc/developer-guide/api/functions/TSTypes.en.rst  | 2 +-
 doc/developer-guide/api/functions/TSVConnReenable.en.rst  | 6 +++---
 doc/developer-guide/cache-architecture/architecture.en.rst| 2 +-
 doc/developer-guide/core-architecture/heap.en.rst | 2 +-
 doc/developer-guide/documentation/building.en.rst | 2 +-
 doc/developer-guide/documentation/ts-markup.en.rst| 2 +-
 doc/developer-guide/internal-libraries/buffer-writer.en.rst   | 2 +-
 .../example-plugins/denylist/setting-up-a-transaction-hook.en.rst | 8 
 .../hooks-and-transactions/http-alternate-selection.en.rst| 2 +-
 .../plugins/hooks-and-transactions/http-sessions.en.rst   | 2 +-
 .../plugins/hooks-and-transactions/ssl-hooks.en.rst   | 6 +++---
 .../plugins/hooks-and-transactions/trafficserver-timers.en.rst| 2 +-
 doc/developer-guide/plugins/http-transformations/index.en.rst | 2 +-
 .../http-transformations/sample-null-transformation-plugin.en.rst | 6 +++---
 doc/developer-guide/plugins/mutexes.en.rst| 2 +-
 doc/developer-guide/release-process/index.en.rst  | 2 +-
 doc/developer-guide/threads-and-events.en.rst | 2 +-
 28 files changed, 40 insertions(+), 40 deletions(-)

diff --git a/doc/admin-guide/files/records.config.en.rst 
b/doc/admin-guide/files/records.config.en.rst
index 8dd961f..5bcb9a8 100644
--- a/doc/admin-guide/files/records.config.en.rst
+++ b/doc/admin-guide/files/records.config.en.rst
@@ -959,7 +959,7 @@ mptcp
  of the origin server matches.
``host``  Re-use server sessions, checking that the fully qualified
  domain name matches. In addition, if the session uses TLS, it 
also
- checks that the current transaction's host header value 
matchs the session's SNI.
+ checks that the current transaction's host header value 
matches the session's SNI.
``both``  Equivalent to ``host,ip``.
``hostonly``  Check that the fully qualified domain name matches.
``sni``   Check that the SNI of the session matches the SNI that would 
be used to
diff --git a/doc/admin-guide/files/remap.config.en.rst 
b/doc/admin-guide/files/remap.config.en.rst
index c1075d3..6df6863 100644
--- a/doc/admin-guide/files/remap.config.en.rst
+++ b/doc/admin-guide/files/remap.config.en.rst
@@ -363,7 +363,7 @@ be verified for validity.  If the "~" symbol was specified 
before referer
 regular expression, it means that the request with a matching referer header
 will be redirected to redirectURL. It can be used to create a so-called
 negative referer list.  If "*" was used as a referer regular expression -
-all referers are allowed.  Various combinations of "*" and "~" in a referer
+all referrers are allowed.  Various combinations of "*" and "~" in a referer
 list can be used to create different filtering rules.
 
 map_with_referer Examples
@@ -379,7 +379,7 @@ Explanation: Referer header must be in the request, only 
".*\.bar\.com" and "www
 
map_with_referer http://y.foo.bar.com/x/yy/  http://foo.bar.com/x/yy/ 
http://games.bar.com/new_games * ~.*\.evil\.com
 
-Explanation: Referer header must be in the request but all referers are 
allowed except ".*\.evil\.com".
+Explanation: Referer header must be in the request but all referrers are 
allowed except ".*\.evil\.com".
 
 ::
 
diff --git a/doc/admin-guide/plugins/background_fetch.en.rst 
b/doc/admin-guide/plugins/background_fetch.en.rst
index 8f28a7e..9f5

[trafficserver] branch 9.1.x updated: Makes sure the types are correct, avoiding compiler warnings (#7523)

2021-05-26 Thread bcall
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 6cc4493  Makes sure the types are correct, avoiding compiler warnings 
(#7523)
6cc4493 is described below

commit 6cc4493b33a90262c29faa0ce3c556eb86ea5a58
Author: Leif Hedstrom 
AuthorDate: Tue Feb 16 13:11:10 2021 -0700

Makes sure the types are correct, avoiding compiler warnings (#7523)

(cherry picked from commit dae3e763e49627ee24f5b6cac3863e1f0b74f013)
---
 proxy/hdrs/unit_tests/test_HdrUtils.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/proxy/hdrs/unit_tests/test_HdrUtils.cc 
b/proxy/hdrs/unit_tests/test_HdrUtils.cc
index 3a164c8..2910b4a 100644
--- a/proxy/hdrs/unit_tests/test_HdrUtils.cc
+++ b/proxy/hdrs/unit_tests/test_HdrUtils.cc
@@ -160,7 +160,7 @@ TEST_CASE("HdrUtils 2", "[proxy][hdrutils]")
   int skip   = 0;
   auto parse = mime_hdr_print(heap, mime.m_mime, buff, 
static_cast(sizeof(buff)), , );
   REQUIRE(parse != 0);
-  REQUIRE(idx == text.size());
+  REQUIRE(idx == static_cast(text.size()));
   REQUIRE(0 == memcmp(ts::TextView(buff, idx), text));
   heap->destroy();
 };
@@ -203,7 +203,7 @@ TEST_CASE("HdrUtils 3", "[proxy][hdrutils]")
   int skip   = 0;
   auto parse = mime_hdr_print(heap, mime.m_mime, buff, 
static_cast(sizeof(buff)), , );
   REQUIRE(parse != 0);
-  REQUIRE(idx == text.size());
+  REQUIRE(idx == static_cast(text.size()));
   REQUIRE(0 == memcmp(ts::TextView(buff, idx), text));
   heap->destroy();
 };


[trafficserver] branch master updated: Fixed some spelling mistakes in comments (#7869)

2021-05-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 c93e347  Fixed some spelling mistakes in comments (#7869)
c93e347 is described below

commit c93e34765d530c6c27b9adf362c58562fdacbc66
Author: Bryan Call 
AuthorDate: Fri May 21 16:03:22 2021 -0700

Fixed some spelling mistakes in comments (#7869)
---
 proxy/hdrs/HTTP.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/proxy/hdrs/HTTP.h b/proxy/hdrs/HTTP.h
index b344f77..88f2854 100644
--- a/proxy/hdrs/HTTP.h
+++ b/proxy/hdrs/HTTP.h
@@ -102,7 +102,7 @@ enum HTTPWarningCode {
   HTTP_WARNING_CODE_MISC_WARNING   = 199
 };
 
-/* squild log codes
+/* squid log codes
There is code (e.g. logstats) that depends on these errors coming at the 
end of this enum */
 enum SquidLogCode {
   SQUID_LOG_EMPTY = '0',
@@ -160,13 +160,13 @@ enum SquidLogCode {
   SQUID_LOG_ERR_UNKNOWN   = 'Z'
 };
 
-// squild log subcodes
+// squid log subcodes
 enum SquidSubcode {
   SQUID_SUBCODE_EMPTY = '0',
   SQUID_SUBCODE_NUM_REDIRECTIONS_EXCEEDED = '1',
 };
 
-/* squid hieratchy codes */
+/* squid hierarchy codes */
 enum SquidHierarchyCode {
   SQUID_HIER_EMPTY   = '0',
   SQUID_HIER_NONE= '1',


[trafficserver] branch master updated: Fixed ASAN issues with MMH test (#7868)

2021-05-21 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 c7f3918  Fixed ASAN issues with MMH test (#7868)
c7f3918 is described below

commit c7f391845d8910e4e3a7248641945b5f81e73d04
Author: Bryan Call 
AuthorDate: Fri May 21 16:01:56 2021 -0700

Fixed ASAN issues with MMH test (#7868)
---
 src/tscore/unit_tests/test_MMH.cc | 18 +-
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/tscore/unit_tests/test_MMH.cc 
b/src/tscore/unit_tests/test_MMH.cc
index cf53935..f8cf3aa 100644
--- a/src/tscore/unit_tests/test_MMH.cc
+++ b/src/tscore/unit_tests/test_MMH.cc
@@ -83,15 +83,18 @@ TEST_CASE("MMH", "[libts][MMH]")
   printf("** collision %d\n", xy);
   }
 
-  unsigned char *s  = (unsigned char *)MMH_x;
-  int l = sizeof(MMH_x);
-  unsigned char *s1 = (unsigned char *)ats_malloc(l + 3);
+  unsigned char *s   = (unsigned char *)MMH_x;
+  int l  = sizeof(MMH_x);
+  unsigned char *s1  = (unsigned char *)ats_malloc(l + sizeof(uint32_t));
+  unsigned char *free_s1 = s1;
   s1 += 1;
   memcpy(s1, s, l);
-  unsigned char *s2 = (unsigned char *)ats_malloc(l + 3);
+  unsigned char *s2  = (unsigned char *)ats_malloc(l + sizeof(uint32_t));
+  unsigned char *free_s2 = s2;
   s2 += 2;
   memcpy(s2, s, l);
-  unsigned char *s3 = (unsigned char *)ats_malloc(l + 3);
+  unsigned char *s3  = (unsigned char *)ats_malloc(l + sizeof(uint32_t));
+  unsigned char *free_s3 = s3;
   s3 += 3;
   memcpy(s3, s, l);
 
@@ -136,4 +139,9 @@ TEST_CASE("MMH", "[libts][MMH]")
 if (!(z % 7))
   printf("\n");
   }
+  printf("\n");
+
+  ats_free(free_s1);
+  ats_free(free_s2);
+  ats_free(free_s3);
 }


[trafficserver] branch master updated: Unifdef test code for MMH and moved it into its own test file (#7841)

2021-05-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 bed49e4  Unifdef test code for MMH and moved it into its own test file 
(#7841)
bed49e4 is described below

commit bed49e4dc5534257e7b00a153e7c75cd7713ff55
Author: Bryan Call 
AuthorDate: Tue May 18 16:12:08 2021 -0700

Unifdef test code for MMH and moved it into its own test file (#7841)

Fixed issues with the test and got it passing
---
 include/tscore/MMH.h  |   3 +
 src/tscore/MMH.cc | 129 +--
 src/tscore/Makefile.am|   3 +-
 src/tscore/unit_tests/test_MMH.cc | 139 ++
 4 files changed, 145 insertions(+), 129 deletions(-)

diff --git a/include/tscore/MMH.h b/include/tscore/MMH.h
index 6ab5bbd..877eb33 100644
--- a/include/tscore/MMH.h
+++ b/include/tscore/MMH.h
@@ -27,6 +27,9 @@
 #include "tscore/ink_defs.h"
 #include "tscore/CryptoHash.h"
 
+#define MMH_X_SIZE 512
+extern uint64_t MMH_x[MMH_X_SIZE + 8];
+
 struct MMH_CTX {
   uint64_t state[4];
   unsigned char buffer[32];
diff --git a/src/tscore/MMH.cc b/src/tscore/MMH.cc
index 64c40f6..b0ed146 100644
--- a/src/tscore/MMH.cc
+++ b/src/tscore/MMH.cc
@@ -27,10 +27,8 @@
 #include "tscore/ink_platform.h"
 #include "tscore/MMH.h"
 
-#define MMH_X_SIZE 512
-
 /* BUG: INKqa11504: need it be to 64 bits...otherwise it overflows */
-static uint64_t MMH_x[MMH_X_SIZE + 8] = {
+uint64_t MMH_x[MMH_X_SIZE + 8] = {
   0x3ee18b32, 0x746d0d6b, 0x591be6a3, 0x760bd17f, 0x363c765d, 0x4bf3d5c5, 
0x10f0510a, 0x39a84605, 0x2282b48f, 0x6903652e,
   0x1b491170, 0x1ab8407a, 0x776b8aa8, 0x5b126ffe, 0x5095db1a, 0x565fe90c, 
0x3ae1f068, 0x73fdf0cb, 0x72f39a81, 0x6a40a4a3,
   0x4ef557fe, 0x360c1a2c, 0x4579b0ea, 0x61dfd174, 0x269b242f, 0x752d6298, 
0x15f10fa3, 0x618b7ab3, 0x6699171f, 0x488f2c6c,
@@ -84,18 +82,6 @@ static uint64_t MMH_x[MMH_X_SIZE + 8] = {
   0x721a2a75, 0x13427ca9, 0x20e03cc9, 0x5f884596, 0x19dc210f, 0x066c954d, 
0x52f43f40, 0x5d9c256f, 0x7f0acaae, 0x1e186b81,
   0x55e9920f, 0x0e4f77b2, 0x6700ec53, 0x268837c0, 0x554ce08b, 0x4284e695, 
0x2127e806, 0x384cb53b, 0x51076b2f, 0x23f9eb15};
 
-// We don't need this generator in release.
-#ifdef TEST
-// generator for above
-static void
-ink_init_MMH()
-{
-  srand48(13); // must remain the same!
-  for (int i = 0; i < MMH_X_SIZE; i++)
-MMH_x[i] = lrand48();
-}
-#endif /* TEST */
-
 int
 ink_code_incr_MMH_init(MMH_CTX *ctx)
 {
@@ -380,116 +366,3 @@ MMHContext::finalize(CryptoHash )
 {
   return 0 == ink_code_incr_MMH_final(hash.u8, &_ctx);
 }
-
-#ifdef TEST
-
-#define TEST_COLLISIONS 1000
-
-static int
-xxcompar(uint32_t **x, uint32_t **y)
-{
-  for (int i = 0; i < 4; i++) {
-if (x[i] > y[i])
-  return 1;
-if (x[i] < y[i])
-  return -1;
-  }
-  return 0;
-}
-
-typedef uint32_t i4_t[4];
-i4_t *xxh;
-double *xf;
-
-main()
-{
-  union {
-unsigned char hash[16];
-uint32_t h[4];
-  } h;
-
-  xxh = (i4_t *)ats_malloc(4 * sizeof(uint32_t) * TEST_COLLISIONS);
-  xf  = (double *)ats_malloc(sizeof(double) * TEST_COLLISIONS);
-
-  printf("test collisions\n");
-  char *sc1 = "http://npdev:19080/1.666400/4000;;
-  char *sc2 = "http://npdev:19080/1.866600/4000;;
-  char *sc3 = "http://:@npdev/1.666400/4000;?;;
-  char *sc4 = "http://:@npdev/1.866600/4000;?;;
-  ink_code_MMH((unsigned char *)sc1, strlen(sc1), h.hash);
-  printf("%X %X %X %X\n", h.h[0], h.h[1], h.h[2], h.h[3]);
-  ink_code_MMH((unsigned char *)sc2, strlen(sc2), h.hash);
-  printf("%X %X %X %X\n", h.h[0], h.h[1], h.h[2], h.h[3]);
-  ink_code_MMH((unsigned char *)sc3, strlen(sc3), h.hash);
-  printf("%X %X %X %X\n", h.h[0], h.h[1], h.h[2], h.h[3]);
-  ink_code_MMH((unsigned char *)sc4, strlen(sc4), h.hash);
-  printf("%X %X %X %X\n", h.h[0], h.h[1], h.h[2], h.h[3]);
-
-  srand48(time(nullptr));
-  for (int xx = 0; xx < TEST_COLLISIONS; xx++) {
-char xs[256];
-xf[xx] = drand48();
-sprintf(xs, "http://@npdev/%16.14f/4000;?;, xf[xx]);
-ink_code_MMH((unsigned char *)xs, strlen(xs), (unsigned char *)[xx]);
-  }
-  qsort(xxh, TEST_COLLISIONS, 16, xxcompar);
-  for (int xy = 0; xy < TEST_COLLISIONS - 1; xy++) {
-if (xxh[xy][0] == xxh[xy + 1][0] && xxh[xy][1] == xxh[xy + 1][1] && 
xxh[xy][2] == xxh[xy + 1][2] &&
-xxh[xy][3] == xxh[xy + 1][3])
-  printf("** collision %d\n", xy);
-  }
-
-  unsigned char *s  = (unsigned char *)MMH_x;
-  int l = sizeof(MMH_x);
-  unsigned char *s1 = (unsigned char *)ats_malloc(l + 3);
-  s1 += 1;
-  memcpy(s1, s, l);
-  unsigned char *s2 = (unsigned char *)ats_malloc(l + 3);
-  s2 += 2;
-  memcpy(s2, s, l);
-  unsign

[trafficserver] branch master updated (e728d0a -> cc83e53)

2021-05-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from e728d0a  Extra braces for clang 5 / ubuntu 16.04 on array 
initialization (#7842)
 add cc83e53  Fixed double declaration types for log buffer tracking (#7847)

No new revisions were added by this update.

Summary of changes:
 proxy/logging/LogBuffer.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


[trafficserver] branch master updated: Extra braces for clang 5 / ubuntu 16.04 on array initialization (#7842)

2021-05-17 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 e728d0a  Extra braces for clang 5 / ubuntu 16.04 on array 
initialization (#7842)
e728d0a is described below

commit e728d0a2614b940821a1a401176b3a9612a5b5f1
Author: Bryan Call 
AuthorDate: Mon May 17 07:50:52 2021 -0700

Extra braces for clang 5 / ubuntu 16.04 on array initialization (#7842)
---
 src/tscore/Regex.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tscore/Regex.cc b/src/tscore/Regex.cc
index ea80116..1fe1e31 100644
--- a/src/tscore/Regex.cc
+++ b/src/tscore/Regex.cc
@@ -111,7 +111,7 @@ Regex::get_capture_count()
 bool
 Regex::exec(std::string_view const )
 {
-  std::array ovector = {0};
+  std::array ovector = {{0}};
   return this->exec(str, ovector.data(), ovector.size());
 }
 


[trafficserver] branch master updated: Fixed warning in gcc 11 about array not being initalized (#7840)

2021-05-14 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 db75f18  Fixed warning in gcc 11 about array not being initalized 
(#7840)
db75f18 is described below

commit db75f1869883614b49cd6276edcb138e44af7cad
Author: Bryan Call 
AuthorDate: Fri May 14 12:55:20 2021 -0700

Fixed warning in gcc 11 about array not being initalized (#7840)
---
 src/tscore/Regex.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tscore/Regex.cc b/src/tscore/Regex.cc
index 9d93c8d..ea80116 100644
--- a/src/tscore/Regex.cc
+++ b/src/tscore/Regex.cc
@@ -111,7 +111,7 @@ Regex::get_capture_count()
 bool
 Regex::exec(std::string_view const )
 {
-  std::array ovector;
+  std::array ovector = {0};
   return this->exec(str, ovector.data(), ovector.size());
 }
 


[trafficserver] branch master updated (eb765c0 -> 1d2be8e)

2021-05-14 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from eb765c0  Address assert on captive_action (#7807)
 add 1d2be8e  build_h3_tools: use OpenSSL_1_1_1k+quic (#7836)

No new revisions were added by this update.

Summary of changes:
 tools/build_h3_tools.sh | 31 ++-
 1 file changed, 22 insertions(+), 9 deletions(-)


[trafficserver] branch master updated (8585adf -> 1d3a201)

2021-05-13 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 8585adf  Remove unused member from HttpSM (#7835)
 add 1d3a201  Simplification dir_init_done (#7817)

No new revisions were added by this update.

Summary of changes:
 iocore/cache/Cache.cc | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)


[trafficserver] branch master updated (7e3d7fa -> ce9dd1a)

2021-04-26 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 7e3d7fa  if transaction status non-success, bypass intercept plugin 
(#7724)
 add ce9dd1a  AIO_NOT_IN_PROGRESS should not be 0 (#7734)

No new revisions were added by this update.

Summary of changes:
 iocore/aio/I_AIO.h| 2 +-
 iocore/aio/P_AIO.h| 8 
 iocore/cache/P_CacheVol.h | 4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)


[trafficserver] branch master updated: ink_utf8_to_latin1 is not defined, removing declaration (#7737)

2021-04-23 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 9436559  ink_utf8_to_latin1 is not defined, removing declaration 
(#7737)
9436559 is described below

commit 943655989da8e4364a641f79917a1969968116c7
Author: Bryan Call 
AuthorDate: Fri Apr 23 13:03:34 2021 -0700

ink_utf8_to_latin1 is not defined, removing declaration (#7737)
---
 include/tscore/ink_string.h | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/include/tscore/ink_string.h b/include/tscore/ink_string.h
index d7f9ae0..997e556 100644
--- a/include/tscore/ink_string.h
+++ b/include/tscore/ink_string.h
@@ -76,9 +76,6 @@ size_t ink_strlcpy(char *dst, const char *str, size_t siz);
 size_t ink_strlcat(char *dst, const char *str, size_t siz);
 #endif
 
-/* Convert from UTF-8 to latin-1/iso-8859-1.  This can be lossy. */
-void ink_utf8_to_latin1(const char *in, int inlen, char *out, int *outlen);
-
 /*===*
 
  Inline Functions


[trafficserver] branch master updated: Fix build on FreeBSD 13 (#7730)

2021-04-23 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 7612422  Fix build on FreeBSD 13 (#7730)
7612422 is described below

commit 76124222d179d55a2c2a2e74806377e54df744a9
Author: Brian Geffon 
AuthorDate: Fri Apr 23 07:35:56 2021 -0700

Fix build on FreeBSD 13 (#7730)

FreeBSD 13 added eventfd(2), which means that now having eventfd no
longer means that you'll also have epoll. This change updates
EventNotify to check for both eventfd(2) and epoll before trying to
use epoll.
---
 include/tscore/EventNotify.h |  2 +-
 src/tscore/EventNotify.cc| 18 +-
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/include/tscore/EventNotify.h b/include/tscore/EventNotify.h
index 864c809..09b6293 100644
--- a/include/tscore/EventNotify.h
+++ b/include/tscore/EventNotify.h
@@ -45,7 +45,7 @@ public:
   ~EventNotify();
 
 private:
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   int m_event_fd;
   int m_epoll_fd;
 #else
diff --git a/src/tscore/EventNotify.cc b/src/tscore/EventNotify.cc
index fb88fef..fae43c6 100644
--- a/src/tscore/EventNotify.cc
+++ b/src/tscore/EventNotify.cc
@@ -31,7 +31,7 @@
 #include "tscore/ink_hrtime.h"
 #include "tscore/ink_defs.h"
 
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
 #include 
 #include 
 #include 
@@ -39,7 +39,7 @@
 
 EventNotify::EventNotify()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   int ret;
   struct epoll_event ev;
 
@@ -63,7 +63,7 @@ EventNotify::EventNotify()
 void
 EventNotify::signal()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   uint64_t value = 1;
   //
   // If the addition would cause the counter's value of eventfd
@@ -79,7 +79,7 @@ EventNotify::signal()
 int
 EventNotify::wait()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   ssize_t nr, nr_fd;
   uint64_t value = 0;
   struct epoll_event ev;
@@ -107,7 +107,7 @@ EventNotify::wait()
 int
 EventNotify::timedwait(int timeout) // milliseconds
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   ssize_t nr, nr_fd = 0;
   uint64_t value = 0;
   struct epoll_event ev;
@@ -148,7 +148,7 @@ EventNotify::timedwait(int timeout) // milliseconds
 void
 EventNotify::lock()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
 // do nothing
 #else
   ink_mutex_acquire(_mutex);
@@ -158,7 +158,7 @@ EventNotify::lock()
 bool
 EventNotify::trylock()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   return true;
 #else
   return ink_mutex_try_acquire(_mutex);
@@ -168,7 +168,7 @@ EventNotify::trylock()
 void
 EventNotify::unlock()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
 // do nothing
 #else
   ink_mutex_release(_mutex);
@@ -177,7 +177,7 @@ EventNotify::unlock()
 
 EventNotify::~EventNotify()
 {
-#ifdef HAVE_EVENTFD
+#if defined(HAVE_EVENTFD) && TS_USE_EPOLL == 1
   close(m_event_fd);
   close(m_epoll_fd);
 #else


[trafficserver] branch master updated (790a7e3 -> 79698cc)

2021-04-20 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 790a7e3  Deny unknown transfer encoding values (#7694)
 add 79698cc  Ran clang-tidy over the code (#7708)

No new revisions were added by this update.

Summary of changes:
 .../plugins/c-api/ssl_preaccept/ssl_preaccept.cc   |  2 +-
 iocore/cache/test/main.cc  |  4 +--
 iocore/net/SSLSessionCache.cc  |  6 ++--
 iocore/net/SSLSessionCache.h   |  6 ++--
 plugins/background_fetch/background_fetch.cc   |  2 +-
 .../cache_range_requests/cache_range_requests.cc   |  4 +--
 plugins/escalate/escalate.cc   |  2 +-
 plugins/esi/esi.cc |  2 +-
 plugins/esi/test/parser_test.cc|  2 +-
 plugins/esi/test/utils_test.cc |  6 ++--
 plugins/experimental/access_control/utils.cc   |  2 +-
 .../collapsed_forwarding/collapsed_forwarding.cc   |  4 +--
 plugins/experimental/cookie_remap/cookie_remap.cc  |  4 +--
 plugins/experimental/magick/magick.cc  |  2 +-
 plugins/experimental/maxmind_acl/mmdb.cc   | 32 +++---
 plugins/experimental/memcache/tsmemcache.cc|  6 ++--
 plugins/experimental/mysql_remap/mysql_remap.cc|  4 +--
 .../experimental/parent_select/consistenthash.cc   |  2 +-
 .../parent_select/consistenthash_config.cc | 17 ++--
 plugins/experimental/parent_select/healthstatus.cc |  3 +-
 .../experimental/parent_select/parent_select.cc| 17 ++--
 plugins/experimental/parent_select/strategy.cc | 14 +-
 plugins/experimental/rate_limit/rate_limit.cc  |  2 +-
 plugins/experimental/statichit/statichit.cc| 12 
 .../experimental/stream_editor/stream_editor.cc|  4 +--
 plugins/multiplexer/ats-multiplexer.cc |  2 +-
 proxy/CacheControl.cc  |  2 +-
 proxy/ParentSelection.cc   |  2 +-
 proxy/hdrs/HuffmanCodec.cc |  4 +--
 proxy/http2/HPACK.cc   |  4 +--
 proxy/logging/LogField.cc  |  2 +-
 src/traffic_crashlog/traffic_crashlog.cc   |  4 +--
 src/traffic_ctl/config.cc  |  4 +--
 src/traffic_logstats/logstats.cc   | 14 +-
 src/traffic_server/Crash.cc|  2 +-
 src/traffic_server/InkAPI.cc   |  2 +-
 src/traffic_server/InkAPITest.cc   |  2 +-
 src/traffic_server/SocksProxy.cc   | 10 +++
 src/tscore/ink_file.cc |  4 +--
 src/tscore/ink_queue.cc|  4 +--
 src/tscore/unit_tests/test_layout.cc   |  3 +-
 41 files changed, 112 insertions(+), 114 deletions(-)


[trafficserver] branch master updated (a8e98d6 -> cc9c446)

2021-04-10 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from a8e98d6  New rate_limit plugin for simple resource limitations (#7623)
 add cc9c446  Automatically marks PRs and issues stale (#7675)

No new revisions were added by this update.

Summary of changes:
 .github/workflows/stale.yml | 27 +++
 1 file changed, 27 insertions(+)
 create mode 100644 .github/workflows/stale.yml


[trafficserver] branch bryancall-patch-2 updated (4cfec76 -> 49b94e8)

2021-04-09 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-2
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from 4cfec76  Update to run if the stale.yml file has been updated
 add 49b94e8  Update stale.yml

No new revisions were added by this update.

Summary of changes:
 .github/workflows/stale.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


[trafficserver] branch bryancall-patch-2 updated (d4e9ca1 -> 4cfec76)

2021-04-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-2
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from d4e9ca1  Automatically marks PRs and issues stale
 add 4cfec76  Update to run if the stale.yml file has been updated

No new revisions were added by this update.

Summary of changes:
 .github/workflows/stale.yml | 3 +++
 1 file changed, 3 insertions(+)


[trafficserver] 01/01: Automatically marks PRs and issues stale

2021-04-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch bryancall-patch-2
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit d4e9ca1abf9e71999af5f5cd3272f0771fdc25e2
Author: Bryan Call 
AuthorDate: Sat Apr 3 08:38:28 2021 -0700

Automatically marks PRs and issues stale

Marks PRs and issues with the Stale tag and makes a comment on the PR and 
issue.
---
 .github/workflows/stale.yml | 24 
 1 file changed, 24 insertions(+)

diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml
new file mode 100644
index 000..9cf41c1
--- /dev/null
+++ b/.github/workflows/stale.yml
@@ -0,0 +1,24 @@
+name: Mark stale issues and pull requests
+
+on:
+  schedule:
+  - cron: "30 1 * * *"
+
+jobs:
+  stale:
+
+runs-on: ubuntu-latest
+
+steps:
+- uses: actions/stale@v3
+  with:
+repo-token: ${{ secrets.GITHUB_TOKEN }}
+stale-issue-message: 'This issue has been automatically marked as 
stale because it has not had recent activity. It will be closed if no further 
activity occurs.'
+stale-pr-message: 'This pull request has been automatically marked as 
stale because it has not had recent activity. It will be closed if no further 
activity occurs.'
+stale-issue-label: 'Stale'
+stale-pr-label: 'Stale'
+days-before-pr-stale: 90
+days-before-issue-stale: 365
+days-before-pr-close: -1
+days-before-issue-close: -1
+debug-only: true


[trafficserver] branch bryancall-patch-2 created (now d4e9ca1)

2021-04-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-2
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


  at d4e9ca1  Automatically marks PRs and issues stale

This branch includes the following new commits:

 new d4e9ca1  Automatically marks PRs and issues stale

The 1 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.



[trafficserver] branch master updated (10f7c51 -> f639621)

2021-03-24 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 10f7c51  Implement log throttling (#7279)
 add f639621  Typo in output when forcing kqueue for configure (#7617)

No new revisions were added by this update.

Summary of changes:
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


[trafficserver] branch master updated (f51f8ed -> 59097ff)

2021-03-04 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from f51f8ed  Make Allocator.h less silly (no creepy "proto" object). 
(#6241)
 add 59097ff  Updates the STATUS file with all recent releases (#7566)

No new revisions were added by this update.

Summary of changes:
 STATUS | 31 ---
 1 file changed, 24 insertions(+), 7 deletions(-)



[trafficserver] branch bryancall-patch-1 updated (74e1f52 -> 95233df)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from 74e1f52  added brew to install openssl
 add 95233df  Update c-cpp.yml

No new revisions were added by this update.

Summary of changes:
 .github/workflows/c-cpp.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch bryancall-patch-1 updated (36b39cb -> 74e1f52)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from 36b39cb  install automake
 add 74e1f52  added brew to install openssl

No new revisions were added by this update.

Summary of changes:
 .github/workflows/c-cpp.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch bryancall-patch-1 updated (2febbdd -> 36b39cb)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from 2febbdd  added brew install for aclocal
 add 36b39cb  install automake

No new revisions were added by this update.

Summary of changes:
 .github/workflows/c-cpp.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch bryancall-patch-1 updated (9e06e62 -> 2febbdd)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from 9e06e62  Update c-cpp.yml
 add 2febbdd  added brew install for aclocal

No new revisions were added by this update.

Summary of changes:
 .github/workflows/c-cpp.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch bryancall-patch-1 updated (b2f58cb -> 9e06e62)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


from b2f58cb  add GitHub workflow configuration
 add 9e06e62  Update c-cpp.yml

No new revisions were added by this update.

Summary of changes:
 .github/workflows/c-cpp.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch bryancall-patch-1 created (now b2f58cb)

2021-03-03 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch bryancall-patch-1
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


  at b2f58cb  add GitHub workflow configuration

No new revisions were added by this update.



[trafficserver-ci] branch main updated: remove rate limit script

2021-02-25 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git


The following commit(s) were added to refs/heads/main by this push:
 new d429990  remove rate limit script
d429990 is described below

commit d4299904758edbdfcc85503e7caa63cd39a00e51
Author: Bryan Call 
AuthorDate: Thu Feb 25 17:23:50 2021 -0800

remove rate limit script
---
 bin/rate_limit.sh | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/bin/rate_limit.sh b/bin/rate_limit.sh
deleted file mode 100755
index fe33225..000
--- a/bin/rate_limit.sh
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/sh
-
-curl -s -u psudaemon:7ba4b68c2dcbf525dd0db1f3cda09dd50a51ca22  
https://api.github.com/rate_limit



[trafficserver-ci] branch main updated: scripts for ci

2021-02-25 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git


The following commit(s) were added to refs/heads/main by this push:
 new e4814fd  scripts for ci
e4814fd is described below

commit e4814fd74457b56c2ca8c36e3d185100b7d9a3ce
Author: Bryan Call 
AuthorDate: Thu Feb 25 15:53:16 2021 -0800

scripts for ci
---
 bin/RTD.sh  |   8 +++
 bin/alpine38.sh |   3 +
 bin/ats_conf.pl | 103 +++
 bin/backtrace.sh|  13 
 bin/btrfs-snap  | 118 
 bin/build_master.sh |  13 
 bin/centos6.sh  |   3 +
 bin/centos7.sh  |   3 +
 bin/centos8.sh  |   3 +
 bin/clean   | 158 
 bin/clean-autest.sh |  17 ++
 bin/clean-clang.sh  |  27 +
 bin/clean-github.sh |   6 ++
 bin/cleanup.sh  |   6 ++
 bin/coad-6_1_x.pl   |  41 +
 bin/compare-yum.sh  |  21 +++
 bin/cov-submit.sh   |  13 
 bin/dbash   |   3 +
 bin/debian7.sh  |   3 +
 bin/debian8.sh  |   3 +
 bin/debian9.sh  |   3 +
 bin/docker  |   5 ++
 bin/docker-all.sh   |  25 
 bin/docker-clean.sh |   4 ++
 bin/fedora30.sh |   3 +
 bin/fedora31.sh |   3 +
 bin/fedora32.sh |   3 +
 bin/fedora33.sh |   3 +
 bin/freebsd.sh  |  32 ++
 bin/gd-sync.sh  |  11 
 bin/gh-mirror.sh|  62 +++
 bin/invoker_wrap.sh |   2 +
 bin/iptables.sh |  11 
 bin/make-machines.sh|  28 +
 bin/master-sync.sh  |   9 +++
 bin/on-all.sh   |   7 +++
 bin/on-docker.sh|  60 ++
 bin/on-gd.sh|  60 ++
 bin/pull-all.sh |  16 +
 bin/purge.sh|   6 ++
 bin/push-all.sh |  16 +
 bin/quic.sh |   3 +
 bin/rate_limit.sh   |   3 +
 bin/rax.sh  |  36 +++
 bin/rsync.sh|   7 +++
 bin/scp-all.sh  |   9 +++
 bin/site_sync.sh|   5 ++
 bin/src_watcher.sh  |   7 +++
 bin/start-ats.sh|  18 ++
 bin/start-registry.sh   |   9 +++
 bin/ubuntu1404.sh   |   3 +
 bin/ubuntu1604.sh   |   3 +
 bin/ubuntu1710.sh   |   3 +
 bin/ubuntu1804.sh   |   3 +
 bin/ubuntu1810.sh   |   3 +
 bin/ubuntu1904.sh   |   3 +
 bin/ubuntu1910.sh   |   3 +
 bin/ubuntu2004.sh   |   3 +
 bin/ubuntu2010.sh   |   3 +
 bin/update-rpm-links.sh |  13 
 bin/update.sh   |  16 +
 bin/version-all.sh  |  25 
 62 files changed, 1112 insertions(+)

diff --git a/bin/RTD.sh b/bin/RTD.sh
new file mode 100755
index 000..59ff33f
--- /dev/null
+++ b/bin/RTD.sh
@@ -0,0 +1,8 @@
+#!/bin/sh
+
+cd /home/jenkins/RTD
+
+wget -O MathJax.js 
'http://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS-MML_HTMLorMML'
+
+wget -O extensions/MathMenu.js 
'http://cdn.mathjax.org/mathjax/latest/extensions/MathMenu.js?config=TeX-AMS-MML_HTMLorMML'
+wget -O extensions/MathZoom.js 
'http://cdn.mathjax.org/mathjax/latest/extensions/MathZoom.js?config=TeX-AMS-MML_HTMLorMML'
diff --git a/bin/alpine38.sh b/bin/alpine38.sh
new file mode 100755
index 000..ba7f466
--- /dev/null
+++ b/bin/alpine38.sh
@@ -0,0 +1,3 @@
+#!/bin/sh
+
+docker run -v /home:/home -v /CA:/CA -v /var/tmp:/var/tmp -i -t 
docker.io/alpine:3.8  /bin/sh
diff --git a/bin/ats_conf.pl b/bin/ats_conf.pl
new file mode 100755
index 000..38c7309
--- /dev/null
+++ b/bin/ats_conf.pl
@@ -0,0 +1,103 @@
+#!/usr/bin/perl
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#  http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+use lib '/opt/ats/share/perl5';
+use Apache::TS::Config::Records;
+use File::Copy;
+
+#chdir("/usr/local");
+chdir("/opt/ats");
+
+my $recedit = new Apache::TS::Config::Records(file => 
"etc/trafficserver/records.config.default");
+
+$recedit->append(line => "");
+$recedit->append(line => "# My local stuff");
+$recedit->append(line => &quo

[trafficserver] branch master updated (e1f5cdb -> bb3b00f)

2021-02-19 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from e1f5cdb  Fix tls_session_reuse test (#7541)
 add bb3b00f  Fix origin_session_reuse test (#7542)

No new revisions were added by this update.

Summary of changes:
 tests/gold_tests/tls/tls_origin_session_reuse.test.py | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)



[trafficserver-ci] 01/01: first commit

2021-02-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git

commit f1ff5d41b1f8da90daa231ddc9e545fce5f50e44
Author: Bryan Call 
AuthorDate: Thu Feb 18 15:35:47 2021 -0800

first commit
---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
new file mode 100644
index 000..a9620d2
--- /dev/null
+++ b/README.md
@@ -0,0 +1 @@
+# trafficserver-ci



[trafficserver-ci] branch main created (now f1ff5d4)

2021-02-18 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/trafficserver-ci.git.


  at f1ff5d4  first commit

This branch includes the following new commits:

 new f1ff5d4  first commit

The 1 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.




[trafficserver] branch master updated (fa8b3f9 -> 7a7a899)

2021-02-16 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from fa8b3f9  Disable compiling Inline.cc on macOS (#7389)
 add 7a7a899  Fix out of bounds access error in jtest (#7526)

No new revisions were added by this update.

Summary of changes:
 tools/jtest/jtest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch master updated (d8f3970 -> dae3e76)

2021-02-16 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from d8f3970  Move has_request_body to ProxyTransaction (#7499)
 add dae3e76  Makes sure the types are correct, avoiding compiler warnings 
(#7523)

No new revisions were added by this update.

Summary of changes:
 proxy/hdrs/unit_tests/test_HdrUtils.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)



[trafficserver] branch master updated (f3eaf81 -> 5528ab6)

2021-02-11 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from f3eaf81  Disable client inactivity timeout while server is processing 
POST request (#7309)
 add 5528ab6  Updates the Dockerfile for debian (#7518)

No new revisions were added by this update.

Summary of changes:
 ci/docker/deb/Dockerfile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch master updated: Upgrade Catch.hpp to v2.13.4 (#7464)

2021-02-11 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 1a1a548  Upgrade Catch.hpp to v2.13.4 (#7464)
1a1a548 is described below

commit 1a1a548c61c176736bf058675153114a22e9b366
Author: Randall Meyer 
AuthorDate: Thu Feb 11 14:24:09 2021 -0800

Upgrade Catch.hpp to v2.13.4 (#7464)

* Upgrade Catch.hpp to v2.13.4

* fixup tests
---
 .../eventsystem/unit_tests/test_MIOBufferWriter.cc |   6 +-
 .../slice/unit-tests/test_content_range.cc |   2 +-
 proxy/http2/unit_tests/test_Http2Frame.cc  |   6 +-
 proxy/logging/unit-tests/test_LogUtils.cc  |   4 +-
 src/tscore/unit_tests/test_Errata.cc   |   2 +-
 src/tscore/unit_tests/test_IntrusiveHashMap.cc |   2 +-
 tests/include/catch.hpp| 892 ++---
 7 files changed, 599 insertions(+), 315 deletions(-)

diff --git a/iocore/eventsystem/unit_tests/test_MIOBufferWriter.cc 
b/iocore/eventsystem/unit_tests/test_MIOBufferWriter.cc
index 3479946..622d4ac 100644
--- a/iocore/eventsystem/unit_tests/test_MIOBufferWriter.cc
+++ b/iocore/eventsystem/unit_tests/test_MIOBufferWriter.cc
@@ -45,11 +45,11 @@ struct MIOBuffer {
 #include "MIOBufferWriter.cc"
 
 IOBufferBlock iobb[1];
-int iobbIdx{0};
+unsigned int iobbIdx{0};
 
-const int BlockSize = 11 * 11;
+const unsigned int BlockSize = 11 * 11;
 char block[BlockSize];
-int blockUsed{0};
+unsigned int blockUsed{0};
 
 std::int64_t
 IOBufferBlock::write_avail()
diff --git a/plugins/experimental/slice/unit-tests/test_content_range.cc 
b/plugins/experimental/slice/unit-tests/test_content_range.cc
index a987629..17b417f 100644
--- a/plugins/experimental/slice/unit-tests/test_content_range.cc
+++ b/plugins/experimental/slice/unit-tests/test_content_range.cc
@@ -47,7 +47,7 @@ TEST_CASE("content_range to/from string - valid", 
"[AWS][slice][utility]")
   bool const strstat(exprange.toStringClosed(gotbuf, ));
 
   CHECK(strstat);
-  CHECK(gotlen == expstr.size());
+  CHECK(gotlen == static_cast(expstr.size()));
   CHECK(expstr == std::string(gotbuf));
 
   ContentRange gotrange;
diff --git a/proxy/http2/unit_tests/test_Http2Frame.cc 
b/proxy/http2/unit_tests/test_Http2Frame.cc
index b957868..d30b07a 100644
--- a/proxy/http2/unit_tests/test_Http2Frame.cc
+++ b/proxy/http2/unit_tests/test_Http2Frame.cc
@@ -39,13 +39,13 @@ TEST_CASE("Http2Frame", "[http2][Http2Frame]")
 uint8_t hdr_block_len = sizeof(hdr_block);
 
 Http2PushPromiseFrame frame(id, flags, pp, hdr_block, hdr_block_len);
-uint64_t written = frame.write_to(miob);
+int64_t written = frame.write_to(miob);
 
-CHECK(written == HTTP2_FRAME_HEADER_LEN + sizeof(Http2StreamId) + 
hdr_block_len);
+CHECK(written == static_cast(HTTP2_FRAME_HEADER_LEN + 
sizeof(Http2StreamId) + hdr_block_len));
 CHECK(written == miob_r->read_avail());
 
 uint8_t buf[32] = {0};
-uint64_t read   = miob_r->read(buf, written);
+int64_t read= miob_r->read(buf, written);
 CHECK(read == written);
 
 uint8_t expected[] = {
diff --git a/proxy/logging/unit-tests/test_LogUtils.cc 
b/proxy/logging/unit-tests/test_LogUtils.cc
index 672aaef..cd87f67 100644
--- a/proxy/logging/unit-tests/test_LogUtils.cc
+++ b/proxy/logging/unit-tests/test_LogUtils.cc
@@ -50,7 +50,7 @@ test(const MIMEField *pairs, int numPairs, const char 
*asciiResult, int extraUnm
 
   int binAlignSize = marshalMimeHdr(numPairs ?  : nullptr, nullptr);
 
-  REQUIRE(binAlignSize < sizeof(binBuf));
+  REQUIRE(binAlignSize < static_cast(sizeof(binBuf)));
 
   hdr.reset();
 
@@ -72,7 +72,7 @@ test(const MIMEField *pairs, int numPairs, const char 
*asciiResult, int extraUnm
 
   char *bp = binBuf;
 
-  int asciiSize = unmarshalMimeHdr(, asciiBuf, std::strlen(asciiResult) + 
extraUnmarshalSpace);
+  unsigned int asciiSize = unmarshalMimeHdr(, asciiBuf, 
std::strlen(asciiResult) + extraUnmarshalSpace);
 
   REQUIRE(asciiSize == std::strlen(asciiResult));
 
diff --git a/src/tscore/unit_tests/test_Errata.cc 
b/src/tscore/unit_tests/test_Errata.cc
index a234f77..f19273a 100644
--- a/src/tscore/unit_tests/test_Errata.cc
+++ b/src/tscore/unit_tests/test_Errata.cc
@@ -49,7 +49,7 @@ TEST_CASE("Basic Errata test with id,code and text", 
"[errata]")
 {
   ts::Errata err;
   int id{1};
-  int code{2};
+  unsigned int code{2};
   std::string text{"Some error text"};
 
   err.push(id, code, text);
diff --git a/src/tscore/unit_tests/test_IntrusiveHashMap.cc 
b/src/tscore/unit_tests/test_IntrusiveHashMap.cc
index e182dd6..8223be3 100644
--- a/src/tscore/unit_tests/test_IntrusiveHashMap.cc
+++ b/src/tscore/unit_tests/test_IntrusiveHashMap.cc
@@ -99,7 +99,7 @@ TEST_CASE("IntrusiveHashMap", "[libts][Intr

[trafficserver] branch master updated: Fix spacing in clang-analyzer.sh script (#7480)

2021-02-10 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 18ceb28  Fix spacing in clang-analyzer.sh script (#7480)
18ceb28 is described below

commit 18ceb283a348dca5a92d613124cd9b553225feb5
Author: Brian Neradt 
AuthorDate: Wed Feb 10 12:46:30 2021 -0600

Fix spacing in clang-analyzer.sh script (#7480)

Co-authored-by: bneradt 
---
 ci/jenkins/bin/clang-analyzer.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci/jenkins/bin/clang-analyzer.sh b/ci/jenkins/bin/clang-analyzer.sh
index 5704fd8..5334b98 100755
--- a/ci/jenkins/bin/clang-analyzer.sh
+++ b/ci/jenkins/bin/clang-analyzer.sh
@@ -47,7 +47,7 @@ if [ "${JOB_NAME#*-github}" != "${JOB_NAME}" ]; then
 ATS_BRANCH="github"
 if [ -w "${OUTPUT_BASE}/${ATS_BRANCH}" ]; then
 output="${OUTPUT_BASE}/${ATS_BRANCH}/${ghprbPullId}"
-[ ! -d "${output}"] && mkdir "${output}"
+[ ! -d "${output}" ] && mkdir "${output}"
 fi
 github_pr=" PR #${ghprbPullId}"
 
results_url="https://ci.trafficserver.apache.org/clang-analyzer/${ATS_BRANCH}/${ghprbPullId}/;



[trafficserver] branch master updated: Fix out of bounds access error in ats_base64_decode (#7490)

2021-02-10 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 95b8699  Fix out of bounds access error in ats_base64_decode (#7490)
95b8699 is described below

commit 95b86998e37c57fb493a6d792d638e0368d7d80c
Author: Masakazu Kitajo 
AuthorDate: Thu Feb 11 03:45:45 2021 +0900

Fix out of bounds access error in ats_base64_decode (#7490)
---
 src/tscore/ink_base64.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tscore/ink_base64.cc b/src/tscore/ink_base64.cc
index 22cb11f..a1da352 100644
--- a/src/tscore/ink_base64.cc
+++ b/src/tscore/ink_base64.cc
@@ -136,7 +136,7 @@ ats_base64_decode(const char *inBuffer, size_t 
inBufferSize, unsigned char *outB
 
   // Ignore any trailing ='s or other undecodable characters.
   // TODO: Perhaps that ought to be an error instead?
-  while (printableToSixBit[static_cast(inBuffer[inBytes])] <= 
MAX_PRINT_VAL) {
+  while (inBytes < inBufferSize && 
printableToSixBit[static_cast(inBuffer[inBytes])] <= MAX_PRINT_VAL) {
 ++inBytes;
   }
 



[trafficserver] branch master updated: Updated to build lastest versions of Fedora and CentOS docker images (#7505)

2021-02-10 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 b9dad45  Updated to build lastest versions of Fedora and CentOS docker 
images (#7505)
b9dad45 is described below

commit b9dad45ebd862fcf885c7dfb75e4d81870f9210b
Author: Bryan Call 
AuthorDate: Wed Feb 10 10:43:37 2021 -0800

Updated to build lastest versions of Fedora and CentOS docker images (#7505)
---
 ci/docker/Makefile | 40 ++--
 1 file changed, 18 insertions(+), 22 deletions(-)

diff --git a/ci/docker/Makefile b/ci/docker/Makefile
index f092c50..0229a86 100644
--- a/ci/docker/Makefile
+++ b/ci/docker/Makefile
@@ -19,40 +19,36 @@
 .DEFAULT_GOAL := help
 
 help:
-   @echo 'fedora26 create ATS docker image for Fedora 26'
-   @echo 'fedora27 create ATS docker image for Fedora 27'
-   @echo 'fedora28 create ATS docker image for Fedora 28'
-   @echo 'fedora29 create ATS docker image for Fedora 29'
-   @echo 'fedora30 create ATS docker image for Fedora 30'
-   @echo 'centos6  create ATS docker image for Centos 6'
+   @echo 'fedora31 create ATS docker image for Fedora 31'
+   @echo 'fedora32 create ATS docker image for Fedora 32'
+   @echo 'fedora33 create ATS docker image for Fedora 33'
+   @echo 'fedora34 create ATS docker image for Fedora 34'
@echo 'centos7  create ATS docker image for Centos 7'
+   @echo 'centos8  create ATS docker image for Centos 8'
@echo 'all  build all images'
 
 all: fedora centos
 
 # Fedora Docker images
-fedora: fedora26 fedora27 fedora28
+fedora: fedora31 fedora32 fedora33 fedora34
 
-fedora26:
-   docker build -t ats_$@ --build-arg OS_VERSION=26 --build-arg 
OS_TYPE=fedora yum/
+fedora31:
+   docker build -t ats_$@ --build-arg OS_VERSION=31 --build-arg 
OS_TYPE=fedora yum/
 
-fedora27:
-   docker build -t ats_$@ --build-arg OS_VERSION=27 --build-arg 
OS_TYPE=fedora yum/
+fedora32:
+   docker build -t ats_$@ --build-arg OS_VERSION=32 --build-arg 
OS_TYPE=fedora yum/
 
-fedora28:
-   docker build -t ats_$@ --build-arg OS_VERSION=28 --build-arg 
OS_TYPE=fedora yum/
+fedora33:
+   docker build -t ats_$@ --build-arg OS_VERSION=33 --build-arg 
OS_TYPE=fedora yum/
 
-fedora29:
-   docker build -t ats_$@ --build-arg OS_VERSION=29 --build-arg 
OS_TYPE=fedora yum/
-
-fedora30:
-   docker build -t ats_$@ --build-arg OS_VERSION=30 --build-arg 
OS_TYPE=fedora yum/
+fedora34:
+   docker build -t ats_$@ --build-arg OS_VERSION=34 --build-arg 
OS_TYPE=fedora yum/
 
 # Centos Docker images
-centos: centos6 centos7
-
-centos6:
-   docker build -t ats_$@ --build-arg OS_VERSION=6 --build-arg 
OS_TYPE=centos yum/
+centos: centos7 centos8
 
 centos7:
docker build -t ats_$@ --build-arg OS_VERSION=7 --build-arg 
OS_TYPE=centos yum/
+
+centos8:
+   docker build -t ats_$@ --build-arg OS_VERSION=8 --build-arg 
OS_TYPE=centos yum/



[trafficserver] branch master updated: Fixed build issues with Fedora 34 (#7506)

2021-02-09 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

bcall 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 ef21e32  Fixed build issues with Fedora 34 (#7506)
ef21e32 is described below

commit ef21e325420a0e94548caefa24bee66e4ec43fca
Author: Bryan Call 
AuthorDate: Tue Feb 9 15:38:43 2021 -0800

Fixed build issues with Fedora 34 (#7506)
---
 include/tscore/Extendible.h   | 1 +
 include/tscpp/util/TextView.h | 1 +
 2 files changed, 2 insertions(+)

diff --git a/include/tscore/Extendible.h b/include/tscore/Extendible.h
index ea94a74..2fbfb7a 100644
--- a/include/tscore/Extendible.h
+++ b/include/tscore/Extendible.h
@@ -44,6 +44,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "tscore/AtomicBit.h"
 #include "tscore/ink_assert.h"
diff --git a/include/tscpp/util/TextView.h b/include/tscpp/util/TextView.h
index 27b09d9..b3b1eff 100644
--- a/include/tscpp/util/TextView.h
+++ b/include/tscpp/util/TextView.h
@@ -32,6 +32,7 @@
 #include 
 #include 
 #include 
+#include 
 
 /** Compare views with ordering, ignoring case.
  *



[trafficserver] branch master updated (13285d1 -> ff5abf6)

2021-02-05 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 13285d1  Perf: Replace casecmp with memcmp in HPACK static table 
lookup (#6521)
 add ff5abf6  update thread config tests (#7370)

No new revisions were added by this update.

Summary of changes:
 tests/gold_tests/thread_config/check_threads.py|  55 +-
 tests/gold_tests/thread_config/gold/http_200.gold  |   9 -
 .../gold_tests/thread_config/thread_100_0.test.py  |  65 --
 .../gold_tests/thread_config/thread_100_1.test.py  |  65 --
 .../gold_tests/thread_config/thread_100_10.test.py |  65 --
 tests/gold_tests/thread_config/thread_1_0.test.py  |  65 --
 tests/gold_tests/thread_config/thread_1_1.test.py  |  65 --
 tests/gold_tests/thread_config/thread_1_10.test.py |  65 --
 tests/gold_tests/thread_config/thread_2_0.test.py  |  65 --
 tests/gold_tests/thread_config/thread_2_1.test.py  |  65 --
 tests/gold_tests/thread_config/thread_2_10.test.py |  65 --
 tests/gold_tests/thread_config/thread_32_0.test.py |  65 --
 tests/gold_tests/thread_config/thread_32_1.test.py |  65 --
 .../gold_tests/thread_config/thread_32_10.test.py  |  65 --
 .../gold_tests/thread_config/thread_config.test.py | 218 +
 15 files changed, 263 insertions(+), 799 deletions(-)
 delete mode 100644 tests/gold_tests/thread_config/gold/http_200.gold
 delete mode 100644 tests/gold_tests/thread_config/thread_100_0.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_100_1.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_100_10.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_1_0.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_1_1.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_1_10.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_2_0.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_2_1.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_2_10.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_32_0.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_32_1.test.py
 delete mode 100644 tests/gold_tests/thread_config/thread_32_10.test.py
 create mode 100644 tests/gold_tests/thread_config/thread_config.test.py



[trafficserver] branch master updated (0e9d179 -> 152c011)

2021-02-01 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


from 0e9d179  Fixing compress expectation for new microserver (#7463)
 add 152c011  Cleans up duplicated TSOutboundConnectionMatchType definition 
(#7090)

No new revisions were added by this update.

Summary of changes:
 include/ts/apidefs.h.in|  3 ---
 proxy/http/HttpProxyAPIEnums.h | 15 ---
 2 files changed, 18 deletions(-)



[trafficserver] branch 9.0.x updated: Finished upgrading release notes for ATS 9.0.0

2021-01-20 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/9.0.x by this push:
 new 77e60f9  Finished upgrading release notes for ATS 9.0.0
77e60f9 is described below

commit 77e60f901325170ad2a10283154373bf9a098a64
Author: Bryan Call 
AuthorDate: Thu Jan 14 16:30:57 2021 -0800

Finished upgrading release notes for ATS 9.0.0
---
 doc/release-notes/upgrading.en.rst | 58 --
 1 file changed, 43 insertions(+), 15 deletions(-)

diff --git a/doc/release-notes/upgrading.en.rst 
b/doc/release-notes/upgrading.en.rst
index 6c8306f..4ad9f4f 100644
--- a/doc/release-notes/upgrading.en.rst
+++ b/doc/release-notes/upgrading.en.rst
@@ -51,7 +51,7 @@ Configuration Settings: records.config
 These are the changes that are most likely to cause problems during an 
upgrade. Take special care making sure you have updated your
 configurations accordingly.
 
-Connection management
+Connection Management
 ~
 
 The old settings for origin connection management included the following 
settings:
@@ -69,7 +69,13 @@ These are all gone, and replaced with the following set of 
configurations:
 * :ts:cv:`proxy.config.http.per_server.connection.queue_delay`
 * :ts:cv:`proxy.config.http.per_server.connection.min`
 
-Removed records.config settings
+
+Renamed records.config Settings
+~~~
+* `proxy.config.http.proxy_protocol_whitelist` was renamed to 
:ts:cv:`proxy.config.http.proxy_protocol_allowlist`
+* `proxy.config.net.max_connections_active_in` was renamed to 
:ts:cv:`proxy.config.net.max_requests_in`
+
+Removed records.config Settings
 ~~~
 
 The following settings are simply gone, and have no purpose:
@@ -95,7 +101,11 @@ longer a special case and are handled the same as a 
response with a non-zero len
 
 * `proxy.config.http.cache.allow_empty_doc`
 
-Deprecated records.config settings
+
+* `proxy.config.ssl.client.verify.server` was deprecated for ATS 8.x and has 
been removed.  Use
+:ts:cv:`proxy.config.ssl.client.verify.server.properties` instead.
+
+Deprecated records.config Settings
 ~~
 
 The following configurations still exist, and functions, but are considered 
deprecated and will be removed in a future release. We
@@ -113,7 +123,7 @@ The following configurations still exist, and functions, 
but are considered depr
   * ``proxy.config.cache.volume_filename``
   * ``proxy.config.dns.splitdns.filename``
 
-Settings with new defaults
+Settings With New Defaults
 ~~
 
 The following settings have changed from being `on` by default, to being off:
@@ -124,9 +134,16 @@ The following settings have changed from being `on` by 
default, to being off:
 * :ts:cv:`proxy.config.ssl.client.TLSv1_1`
 
 The default cipher list has changed:
-
 * :ts:cv:`proxy.config.ssl.server.cipher_suite`
 
+* :ts:cv:`proxy.config.exec_thread.autoconfig.scale` went from a value of 1.5 
to 1.  This controls the number of worker threads and
+the ratio is now worker thread 1 to 1 processing thread on the CPU or CPUs.
+
+
+Settings with new behavior
+~~
+* :ts:cv:`proxy.config.http.connect_attempts_max_retries_dead_server` 
specifies the exact number of times a dead server will be
+retried for each new request.  Before ATS tried 1 time more than this setting 
making it impossible to set the retry value to 0.
 
 Metrics
 ---
@@ -183,14 +200,15 @@ Our APIs are guaranteed to be compatible within major 
versions, but we do make c
 Removed APIs
 
 
-* ``TSHttpTxnRedirectRequest()``
+* :func:`TSHttpTxnRedirectRequest`
 
-Renamed or modified APIs
+Renamed or Modified APIs
 
 
-* ``TSVConnSSLConnectionGet()`` is renamed to be 
:c:func:`TSVConnSslConnectionGet`
-
-* ``TSHttpTxnServerPush()`` now returns a :c:type:`TSReturnCode`
+* :func:`TSVConnSSLConnectionGet` is renamed to be 
:func:`TSVConnSslConnectionGet`
+* :func:`TSHttpTxnServerPush` now returns a :c:type:`TSReturnCode`
+* :func:`TSContSchedule` and :func:`TSContScheduleAPI` by default will run on 
the same thread from which they are called.
+* :func:`TSFetchUrl` and :func:`TSFetchCreate` now return a :c:type:`TSFetchSM`
 
 
 Cache
@@ -243,13 +261,23 @@ Header Rewrite
 
 * `header-rewrite-expansion` was removed and replaced with 
`header-rewrite-concatenations`
 
-Library Dependencies
-
-TCL is no longer required to build ATS.
+Cache Key
+~
+* `--ua-blacklist` was renamed to `--ua-blocklist` and `--ua-whitelist` was 
renamed to `--ua-allowlist`
+
+Logging
+---
+* cqhv has been deprecated and cqpv should be used instead.
 
-The minium OpenSSL version to build ATS is now 1.0.2.
+Library Dependencies and Builds
+---
+* TCL is no longer required to build ATS

[trafficserver] branch 9.0.x updated: Updated upgrading release notes for ATS 9.0.0 - page 35

2021-01-13 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/9.0.x by this push:
 new b598741  Updated upgrading release notes for ATS 9.0.0 - page 35
b598741 is described below

commit b598741cc3f1f5b3602b66515bc5d13880f2bdbf
Author: Bryan Call 
AuthorDate: Wed Jan 13 16:24:16 2021 -0800

Updated upgrading release notes for ATS 9.0.0 - page 35
---
 doc/release-notes/upgrading.en.rst | 28 ++--
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/doc/release-notes/upgrading.en.rst 
b/doc/release-notes/upgrading.en.rst
index 59d1524..6c8306f 100644
--- a/doc/release-notes/upgrading.en.rst
+++ b/doc/release-notes/upgrading.en.rst
@@ -79,8 +79,8 @@ The following settings are simply gone, and have no purpose:
 * `proxy.config.http.server_tcp_init_cwnd` (see Solaris section below)
 * `proxy.config.http.parent_proxy_routing_enable` (implicit by use of 
:file:`parent.config`)
 
-All the `Vary` related configuration overrides were eliminated, prefer to set 
this via the origin server,
-or modify the `Vary` header on server responses instead.
+All the `Vary` related configuration overrides were eliminated, prefer to set 
this via the origin server, or modify the `Vary`
+header on server responses instead.
 
 * `proxy.config.http.cache.vary_default_text`
 * `proxy.config.http.cache.vary_default_images`
@@ -136,9 +136,9 @@ Renamed or Modified Metrics
 
 `proxy.process.http2.current_client_sessions` is renamed to be 
:ts:stat:`proxy.process.http2.current_client_connections`
 
-:ts:stat:`proxy.process.http.current_client_transactions` used to record both 
the number current of HTTP/1.1 and HTTP/2 requests.  Now it only records
-the number current of HTTP/1.1 client requests.  Please use 
:ts:stat:`proxy.process.http2.current_client_streams` to get the number of 
current HTTP/2
-client requests.
+:ts:stat:`proxy.process.http.current_client_transactions` used to record both 
the number current of HTTP/1.1 and HTTP/2 requests.
+Now it only records the number current of HTTP/1.1 client requests.  Please use
+:ts:stat:`proxy.process.http2.current_client_streams` to get the number of 
current HTTP/2 client requests.
 
 Removed Metrics
 ~~~
@@ -152,7 +152,12 @@ The following metrics have been removed.
 
 Command Line Options
 
-`--with-max-api-stats` was replace with `--maxRecords` to specify the total 
number of metric instead of just the total API metrics to use when running ATS.
+
+The following command line options were either renamed or removed.
+
+* `--with-max-api-stats` was replace with `--maxRecords` to specify the total 
number of metric instead of just the total API metrics
+to use when running ATS.
+* `--read_core` was removed and gdb should be used instead.
 
 Deprecated or Removed Features
 --
@@ -164,8 +169,11 @@ Removed the log collation feature along with its 
configuration settings (`proxy.
 
 Rollback Configuration
 ~~
-The rollback configuration code was removed in for ATS v9.0.0.  This featured 
copied the configuration files to have version
-numbers if the end user wanted to rollback to a previous version of the 
configuration.
+The rollback configuration code was removed in for ATS v9.0.0.  This featured 
copied the configuration files to have version numbers
+if the end user wanted to rollback to a previous version of the configuration.
+
+`records.config.shadow` and `records.config.snap` support has also been 
removed in order to clean up and simply loading of
+configuration files.
 
 API Changes
 ---
@@ -196,8 +204,8 @@ in a remap rule would get the pristine URL, and subsequent 
plugins would get the
 receive the remapped URL. If you are using a plugin that modifies the cache 
key, e.g. :ref:`admin-plugins-cachekey`, if it was
 evaluated first in a remap rule, the behavior (input) changes, and therefore, 
cache keys can change!
 
-Caches created with ATS v2.x are incompatible and can't be loading into ATS 
v9.0.0 or later. We feel that this is an unlikely scenario,
-but if you do run into this, clearing the cache is required.
+Caches created with ATS v2.x are incompatible and can't be loading into ATS 
v9.0.0 or later. We feel that this is an unlikely
+scenario, but if you do run into this, clearing the cache is required.
 
 Plugins
 ---



[trafficserver] branch 9.0.x updated: Updated upgrading release notes for ATS 9.0.0 - page 28

2021-01-13 Thread bcall
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/9.0.x by this push:
 new 640dfcd  Updated upgrading release notes for ATS 9.0.0 - page 28
640dfcd is described below

commit 640dfcd2128de09df358a95c438053863425b948
Author: Bryan Call 
AuthorDate: Wed Jan 13 08:13:32 2021 -0800

Updated upgrading release notes for ATS 9.0.0 - page 28
---
 doc/release-notes/upgrading.en.rst | 58 --
 1 file changed, 56 insertions(+), 2 deletions(-)

diff --git a/doc/release-notes/upgrading.en.rst 
b/doc/release-notes/upgrading.en.rst
index 33a7dfc..59d1524 100644
--- a/doc/release-notes/upgrading.en.rst
+++ b/doc/release-notes/upgrading.en.rst
@@ -43,7 +43,7 @@ We are moving configurations over to YAML, and thus far, the 
following configura
 
 In addition, a new file for TLS handshake negotiation configuration is added:
 
-* :file:`sni.yaml` (this was for a while named ssl_server_name.config in 
Github)
+* :file:`sni.yaml` (this was for a while named ssl_server_name.yaml in Github)
 
 Configuration Settings: records.config
 --
@@ -86,6 +86,15 @@ or modify the `Vary` header on server responses instead.
 * `proxy.config.http.cache.vary_default_images`
 * `proxy.config.http.cache.vary_default_other`
 
+SSLv3 support has been removed
+
+* `proxy.config.ssl.client.SSLv3`
+
+The option to disable the caching of an empty response (zero length body 
response) has been removed.  Zero length body responses are no
+longer a special case and are handled the same as a response with a non-zero 
length body.
+
+* `proxy.config.http.cache.allow_empty_doc`
+
 Deprecated records.config settings
 ~~
 
@@ -114,11 +123,15 @@ The following settings have changed from being `on` by 
default, to being off:
 * :ts:cv:`proxy.config.ssl.client.TLSv1`
 * :ts:cv:`proxy.config.ssl.client.TLSv1_1`
 
+The default cipher list has changed:
+
+* :ts:cv:`proxy.config.ssl.server.cipher_suite`
+
 
 Metrics
 ---
 
-Renamed or modified metrics
+Renamed or Modified Metrics
 ~~~
 
 `proxy.process.http2.current_client_sessions` is renamed to be 
:ts:stat:`proxy.process.http2.current_client_connections`
@@ -127,6 +140,20 @@ Renamed or modified metrics
 the number current of HTTP/1.1 client requests.  Please use 
:ts:stat:`proxy.process.http2.current_client_streams` to get the number of 
current HTTP/2
 client requests.
 
+Removed Metrics
+~~~
+
+The following metrics have been removed.
+
+* `proxy.process.ssl.ssl_error_want_read`
+* `proxy.process.ssl.ssl_error_want_write`
+* `proxy.process.ssl.ssl_error_want_x509_lookup`
+* `proxy.process.ssl.ssl_error_zero_return`
+
+Command Line Options
+
+`--with-max-api-stats` was replace with `--maxRecords` to specify the total 
number of metric instead of just the total API metrics to use when running ATS.
+
 Deprecated or Removed Features
 --
 
@@ -135,6 +162,11 @@ should be avoided, with the expectation that they will be 
removed in the next ma
 
 Removed the log collation feature along with its configuration settings 
(`proxy.local.log.collation*`).
 
+Rollback Configuration
+~~
+The rollback configuration code was removed in for ATS v9.0.0.  This featured 
copied the configuration files to have version
+numbers if the end user wanted to rollback to a previous version of the 
configuration.
+
 API Changes
 ---
 
@@ -170,6 +202,28 @@ but if you do run into this, clearing the cache is 
required.
 Plugins
 ---
 
+Promoted Plugins
+
+
+The following plugins have been promoted from experimental to stable plugins:
+* :ts:cv:`cache_range_requests`
+* :ts:cv:`certifier`
+* :ts:cv:`multiplexer`
+* :ts:cv:`prefetch`
+* :ts:cv:`remap_purge`
+
+Removed Plugins
+~~~
+
+The following plugins have been removed because they are no longer used or 
maintained:
+* `balancer`
+* `buffer_uplaod`
+* `header_normalize`
+* `hipes`
+* `memcached_remap`
+* `stale_while_revalidate`
+* `mysql_remap`
+
 The following plugins have changes that might require you to change
 configurations.
 



<    1   2   3   4   5   6   7   8   9   10   >