[trafficserver] branch master updated (35f352c -> 240a3f9)

2019-12-23 Thread masaori
This is an automated email from the ASF dual-hosted git repository.

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


from 35f352c  Fixes a few Sphinx build warnings (#6290)
 add 240a3f9  Perf: replace dynamic_cast with static_cast in this_thread()

No new revisions were added by this update.

Summary of changes:
 iocore/eventsystem/P_UnixEThread.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



[trafficserver] branch 9.0.x updated: Cleaned up indentation etc.

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 ddb5956  Cleaned up indentation etc.
ddb5956 is described below

commit ddb59564a3393ec22d0ca964b99f2b8fa34a4eb0
Author: Leif Hedstrom 
AuthorDate: Mon Dec 23 16:42:54 2019 -0700

Cleaned up indentation etc.
---
 doc/release-notes/upgrading.en.rst | 76 ++
 doc/release-notes/whats-new.en.rst | 41 ++--
 2 files changed, 56 insertions(+), 61 deletions(-)

diff --git a/doc/release-notes/upgrading.en.rst 
b/doc/release-notes/upgrading.en.rst
index 6ad85ab..a46bef5 100644
--- a/doc/release-notes/upgrading.en.rst
+++ b/doc/release-notes/upgrading.en.rst
@@ -28,16 +28,15 @@ Upgrading to ATS v9.x
 Remapping
 -
 
-One of the biggest changes in ATS v9.0.0 is that URL rewrites, as specified in 
a :file:`remap.config`
-rule, now always happens **before** all plugins are executed. This can have 
significant impact on
-behavior, since plugins might now see a different URL than they did in prior 
versions. In particular,
-plugins modifying the cache key could have serious problems (see the  section 
below for details).
+One of the biggest changes in ATS v9.0.0 is that URL rewrites, as specified in 
a :file:`remap.config` rule, now always happens
+**before** all plugins are executed. This can have significant impact on 
behavior, since plugins might now see a different URL than
+they did in prior versions. In particular, plugins modifying the cache key 
could have serious problems (see the section below for
+details).
 
 YAML
 
 
-We are moving configurations over to YAML, and thus far, the following 
configurations are now fully
-migrated over to YAML:
+We are moving configurations over to YAML, and thus far, the following 
configurations are now fully migrated over to YAML:
 
 * :file:`logging.yaml` (*was* `logging.config` or `logging.lua`)
 * :file:`ip_allow.yaml` (*was* `ip_allow.config`)
@@ -49,11 +48,12 @@ In addition, a new file for TLS handhsake negotiation 
configuration is added:
 New records.config settings
 
 
-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.
+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
 ~
+
 The old settings for origin connection management included the following 
settings:
 
 * `proxy.config.http.origin_max_connections`
@@ -79,9 +79,9 @@ The following settings are simply gone, and have no purpose:
 
 Deprecated records.config settings
 --
-The following configurations still exist, and functions, but are considered
-deprecated and will be removed in a future release. We **strongly** encourage
-you to avoid using any of these:
+
+The following configurations still exist, and functions, but are considered 
deprecated and will be removed in a future release. We
+**strongly** encourage you to avoid using any of these:
 
   * :ts:cv:`proxy.config.socks.socks_config_file`
   * :ts:cv:`proxy.config.log.config.filename`
@@ -97,23 +97,24 @@ you to avoid using any of these:
 
 Deprecated or Removed Features
 --
-The following features, configurations and plugins are either removed
-or deprecated in this version of ATS. Deprecated features should be
-avoided, with the expectation that they will be removed in the next major
-release of ATS.
+
+The following features, configurations and plugins are either removed or 
deprecated in this version of ATS. Deprecated features
+should be avoided, with the expectation that they will be removed in the next 
major release of ATS.
 
 
 API Changes
 ---
-Our APIs are guaranteed to be compatible within major versions, but we do
-make changes for each new major release.
+
+Our APIs are guaranteed to be compatible within major versions, but we do make 
changes for each new major release.
 
 Removed APIs
 
+
 * ``TSHttpTxnRedirectRequest()``
 
 Renamed or modified APIs
 
+
 * ``TSVConnSSLConnectionGet()`` is renamed to be 
:c:func:`TSVConnSslConnectionGet`
 
 * ``TSHttpTxnServerPush()`` now returns a :c:type:`TSReturnCode`
@@ -121,37 +122,32 @@ Renamed or modified APIs
 
 Cache
 -
-The cache in this releases of ATS is compatible with previous versions of ATS.
-You would not expect to lose your cache, or have to reinitialize the cache when
-upgrading.
-
-However, due to changes in how remap plugins are processed, your cache key
-*might* change. In versions to v9.0.0, the first plugin in a remap rule would
-get the pristine URL, and subsequent plugins 

[trafficserver] branch 9.0.x updated: Fixes a few Sphinx build warnings (#6290)

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 f4623de  Fixes a few Sphinx build warnings (#6290)
f4623de is described below

commit f4623de40e61152468571960b150e84a02e4377b
Author: Leif Hedstrom 
AuthorDate: Mon Dec 23 15:29:03 2019 -0800

Fixes a few Sphinx build warnings (#6290)

(cherry picked from commit 35f352c2bb7a6399a8908bf051dfb576bea4122c)
---
 doc/admin-guide/plugins/header_rewrite.en.rst | 14 ++
 doc/appendices/command-line/traffic_server.en.rst |  1 +
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/doc/admin-guide/plugins/header_rewrite.en.rst 
b/doc/admin-guide/plugins/header_rewrite.en.rst
index 527852e..93f134d 100644
--- a/doc/admin-guide/plugins/header_rewrite.en.rst
+++ b/doc/admin-guide/plugins/header_rewrite.en.rst
@@ -562,7 +562,7 @@ be specified multiple times, such as ``Set-Cookie``, but 
for headers which may
 only be specified once you may prefer to use `set-header`_ instead.
 
 The header's  may be specified as a literal string, or it may take
-advantage of :ref:`header-rewrite-concatenations` to calculate a dynamic value
+advantage of `String concatenations`_ to calculate a dynamic value
 for the header.
 
 counter
@@ -675,9 +675,8 @@ set-header
 Replaces the value of header  with , creating the header
 if necessary.
 
-The header's  may be specified according to `Header Values`_ or take
-advantage of :ref:`header-rewrite-concatenations` to calculate a dynamic value
-for the header.
+The header's  may be a literal string, or take advantage of
+`String concatenations`_ to calculate a dynamic value for the header.
 
 set-redirect
 
@@ -689,7 +688,7 @@ When invoked, sends a redirect response to the client, with 
HTTP status
 , and a new location of . If the ``QSA`` flag is
 enabled, the original query string will be preserved and added to the new
 location automatically. This operator supports
-:ref:`header-rewrite-concatenations` for .
+`String concatenations`_ for .
 
 set-status
 ~~
@@ -760,8 +759,6 @@ L  Last rule, do not continue.
 QSAAppend the results of the rule to the query string.
 == 
 
-.. _header-rewrite-concatenations:
-
 String concatenations
 -
 
@@ -769,7 +766,8 @@ You can concatenate values using strings, condition values 
and variable expansio
 
 add-header CustomHeader "Hello from %{IP:SERVER}:%{INBOUND:LOCAL-PORT}"
 
-String concatenation is not yet supported in condition testing.
+This is the new, generic form of setting header values. Unfortunately, string
+concatenation is not yet supported in conditionals.
 
 Note: In versions prior to ATS v9.0.0, an alternative string expansion was 
available. those
 expansions are no longer available, but the following table can help 
migrations:
diff --git a/doc/appendices/command-line/traffic_server.en.rst 
b/doc/appendices/command-line/traffic_server.en.rst
index 165267c..7cf4a29 100644
--- a/doc/appendices/command-line/traffic_server.en.rst
+++ b/doc/appendices/command-line/traffic_server.en.rst
@@ -79,6 +79,7 @@ actions or queries against |TS|. Note that some commands 
(such as ``help`` and
 argument, surround the ``CMD`` and its argument in quotes. For instance, to
 request help for the ``verify_global_plugin`` command, format your command like
 so::
+
 traffic_server -C "help verify_global_plugin"
 
 The following commands are supported:



[trafficserver] branch master updated: Fixes a few Sphinx build warnings (#6290)

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 35f352c  Fixes a few Sphinx build warnings (#6290)
35f352c is described below

commit 35f352c2bb7a6399a8908bf051dfb576bea4122c
Author: Leif Hedstrom 
AuthorDate: Mon Dec 23 15:29:03 2019 -0800

Fixes a few Sphinx build warnings (#6290)
---
 doc/admin-guide/plugins/header_rewrite.en.rst | 14 ++
 doc/appendices/command-line/traffic_server.en.rst |  1 +
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/doc/admin-guide/plugins/header_rewrite.en.rst 
b/doc/admin-guide/plugins/header_rewrite.en.rst
index 527852e..93f134d 100644
--- a/doc/admin-guide/plugins/header_rewrite.en.rst
+++ b/doc/admin-guide/plugins/header_rewrite.en.rst
@@ -562,7 +562,7 @@ be specified multiple times, such as ``Set-Cookie``, but 
for headers which may
 only be specified once you may prefer to use `set-header`_ instead.
 
 The header's  may be specified as a literal string, or it may take
-advantage of :ref:`header-rewrite-concatenations` to calculate a dynamic value
+advantage of `String concatenations`_ to calculate a dynamic value
 for the header.
 
 counter
@@ -675,9 +675,8 @@ set-header
 Replaces the value of header  with , creating the header
 if necessary.
 
-The header's  may be specified according to `Header Values`_ or take
-advantage of :ref:`header-rewrite-concatenations` to calculate a dynamic value
-for the header.
+The header's  may be a literal string, or take advantage of
+`String concatenations`_ to calculate a dynamic value for the header.
 
 set-redirect
 
@@ -689,7 +688,7 @@ When invoked, sends a redirect response to the client, with 
HTTP status
 , and a new location of . If the ``QSA`` flag is
 enabled, the original query string will be preserved and added to the new
 location automatically. This operator supports
-:ref:`header-rewrite-concatenations` for .
+`String concatenations`_ for .
 
 set-status
 ~~
@@ -760,8 +759,6 @@ L  Last rule, do not continue.
 QSAAppend the results of the rule to the query string.
 == 
 
-.. _header-rewrite-concatenations:
-
 String concatenations
 -
 
@@ -769,7 +766,8 @@ You can concatenate values using strings, condition values 
and variable expansio
 
 add-header CustomHeader "Hello from %{IP:SERVER}:%{INBOUND:LOCAL-PORT}"
 
-String concatenation is not yet supported in condition testing.
+This is the new, generic form of setting header values. Unfortunately, string
+concatenation is not yet supported in conditionals.
 
 Note: In versions prior to ATS v9.0.0, an alternative string expansion was 
available. those
 expansions are no longer available, but the following table can help 
migrations:
diff --git a/doc/appendices/command-line/traffic_server.en.rst 
b/doc/appendices/command-line/traffic_server.en.rst
index 165267c..7cf4a29 100644
--- a/doc/appendices/command-line/traffic_server.en.rst
+++ b/doc/appendices/command-line/traffic_server.en.rst
@@ -79,6 +79,7 @@ actions or queries against |TS|. Note that some commands 
(such as ``help`` and
 argument, surround the ``CMD`` and its argument in quotes. For instance, to
 request help for the ``verify_global_plugin`` command, format your command like
 so::
+
 traffic_server -C "help verify_global_plugin"
 
 The following commands are supported:



[trafficserver] branch 9.0.x updated: Fixes typos and wrong ref

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 2f1c8f5  Fixes typos and wrong ref
2f1c8f5 is described below

commit 2f1c8f5fb29037e7b21fa0356388077bb448a2ba
Author: Leif Hedstrom 
AuthorDate: Mon Dec 23 14:49:28 2019 -0700

Fixes typos and wrong ref
---
 doc/release-notes/whats-new.en.rst | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/doc/release-notes/whats-new.en.rst 
b/doc/release-notes/whats-new.en.rst
index 4c1d2d1..7fa7ec9 100644
--- a/doc/release-notes/whats-new.en.rst
+++ b/doc/release-notes/whats-new.en.rst
@@ -50,10 +50,10 @@ Developer features
 
 Command line tools
 ~~
-*  The :program:`traffic_ctl` tool now has two new sub-commands:
-  ``verify_global_plugin`` and ``verify_remap_plugin``. These commands load a
-  plugin's shared object file and verify it meets minimal global or remap
-  plugin API requirements.
+* The :program:`traffic_server` program now has two new maintenance
+  commands: ``verify_global_plugin`` and ``verify_remap_plugin``.
+  These commands load a  plugin's shared object file and verify it meets
+  minimal global or remap plugin API requirements.
 
 Incompatible records.config settings
 



[trafficserver] branch master updated (0330471 -> ed28293)

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

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


from 0330471  These features are in 9.0.x release notes (#6286)
 add ed28293  Moving / adding the Roadmap to the release notes (#6257)

No new revisions were added by this update.

Summary of changes:
 doc/release-notes/images/git-versions.png | Bin 0 -> 82979 bytes
 doc/release-notes/images/roadmap.png  | Bin 0 -> 226553 bytes
 doc/release-notes/index.en.rst|   1 +
 doc/release-notes/roadmap.en.rst  | 120 ++
 4 files changed, 121 insertions(+)
 create mode 100644 doc/release-notes/images/git-versions.png
 create mode 100644 doc/release-notes/images/roadmap.png
 create mode 100644 doc/release-notes/roadmap.en.rst



[trafficserver] branch master updated (25806af -> 0330471)

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

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


from 25806af  Removes the remaining references to TSQA
 add 0330471  These features are in 9.0.x release notes (#6286)

No new revisions were added by this update.

Summary of changes:
 doc/release-notes/whats-new.en.rst | 9 ++---
 1 file changed, 2 insertions(+), 7 deletions(-)



[trafficserver] branch 9.0.x updated: Removes the remaining references to TSQA

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 9797558  Removes the remaining references to TSQA
9797558 is described below

commit 97975584bc5b87e8e43e853a4358116e54bbaafe
Author: Randall Meyer 
AuthorDate: Fri Dec 20 16:59:17 2019 -0800

Removes the remaining references to TSQA

(cherry picked from commit 25806af21159f7996835d8ce2bf47b449d77c5b4)
---
 doc/developer-guide/testing/blackbox-testing.en.rst | 2 +-
 example/plugins/c-api/intercept/intercept.cc| 7 +++
 src/traffic_layout/info.cc  | 3 +--
 tests/README.md | 2 +-
 4 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/doc/developer-guide/testing/blackbox-testing.en.rst 
b/doc/developer-guide/testing/blackbox-testing.en.rst
index eba8240..07c8296 100644
--- a/doc/developer-guide/testing/blackbox-testing.en.rst
+++ b/doc/developer-guide/testing/blackbox-testing.en.rst
@@ -26,7 +26,7 @@ for functional testing. The current layout is:
 
 ::
 
-gold_tests/ - contains all the TSQA v4 based tests that run on the 
Reusable Gold Testing System (AuTest)
+gold_tests/ - contains all the tests that run on the Reusable Gold Testing 
System (AuTest)
 tools/ - contains programs used to help with testing.
 
 Scripts
diff --git a/example/plugins/c-api/intercept/intercept.cc 
b/example/plugins/c-api/intercept/intercept.cc
index 602857e..4d68be7 100644
--- a/example/plugins/c-api/intercept/intercept.cc
+++ b/example/plugins/c-api/intercept/intercept.cc
@@ -1,6 +1,6 @@
 /** @file
 
-  an example hello world plugin
+  an example intercept plugin
 
   @section license License
 
@@ -40,8 +40,7 @@
 // This plugin intercepts all cache misses and proxies them to a separate 
server
 // that is assumed to be running on localhost:6. The plugin does no HTTP
 // processing at all, it simply shuffles data until the client closes the
-// request. The TSQA test test-server-intercept exercises this plugin. You can
-// enable extensive logging with the "intercept" diagnostic tag.
+// request. You can enable extensive logging with the "intercept" diagnostic 
tag.
 
 #define PLUGIN_NAME "intercept"
 #define PORT 6
@@ -552,6 +551,6 @@ TSPluginInit(int /* argc */, const char * /* argv */[])
   InterceptHook = InterceptContCreate(InterceptInterceptionHook, nullptr, 
nullptr);
 
   // Wait until after the cache lookup to decide whether to
-  // intercept a request. For cache hits we will never intercept.
+  // intercept a request. For cache hits, we will never intercept.
   TSHttpHookAdd(TS_HTTP_CACHE_LOOKUP_COMPLETE_HOOK, TxnHook);
 }
diff --git a/src/traffic_layout/info.cc b/src/traffic_layout/info.cc
index 736e287..062aa7a 100644
--- a/src/traffic_layout/info.cc
+++ b/src/traffic_layout/info.cc
@@ -41,8 +41,7 @@
 #include 
 #endif
 
-// Produce output about compile time features, useful for checking how things 
were built, as well
-// as for our TSQA test harness.
+// Produce output about compile time features, useful for checking how things 
were built
 static void
 print_feature(std::string_view name, int value, bool json, bool last = false)
 {
diff --git a/tests/README.md b/tests/README.md
index 3024d94..e6bea58 100644
--- a/tests/README.md
+++ b/tests/README.md
@@ -6,7 +6,7 @@ This directory contains different tests for Apache 
Trafficserver. It is recommen
 ## Layout
 The current layout is:
 
-**gold_tests/** - contains all the TSQA v4 based tests that run on the 
Reusable Gold Testing System (AuTest)
+**gold_tests/** - contains all the tests that run on the Reusable Gold Testing 
System (AuTest)
 
 **tools/** - contains programs used to help with testing.
 



[trafficserver] branch 9.0.x updated: Updated CHANGELOG

2019-12-23 Thread zwoop
This is an automated email from the ASF dual-hosted git repository.

zwoop 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 a01608e  Updated CHANGELOG
a01608e is described below

commit a01608e9e2ab027d77fba11df84a16fdcdac181e
Author: Leif Hedstrom 
AuthorDate: Mon Dec 23 11:19:49 2019 -0700

Updated CHANGELOG
---
 CHANGELOG-9.0.0 | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/CHANGELOG-9.0.0 b/CHANGELOG-9.0.0
index 24443a5..a244b12 100644
--- a/CHANGELOG-9.0.0
+++ b/CHANGELOG-9.0.0
@@ -751,6 +751,7 @@ Changes with Apache Traffic Server 9.0.0
   #5964 - Fix a build issue on enabling FIPS
   #5969 - Change the ready checks for ats to use the actual ats port
   #5970 - "Plugin (un)used" post reload notification
+  #5972 - Avoid unnecesarry copy on POST request over HTTP/2
   #5973 - url_sig: fix memory leak with urlParse and pristine flag
   #5977 - YAML config:  output erroneous keyword and line number
   #5978 - Removed hardcoded logging.yaml filename in logs
@@ -761,6 +762,7 @@ Changes with Apache Traffic Server 9.0.0
   #5987 - Removing traffic_cop reference in admin guide introduction
   #5992 - Cache SSL EC explicitly
   #6000 - Add QUIC draft-23 support
+  #6001 - New TS maintenance commands: verify_global_plugin and 
verify_remap_plugin
   #6003 - Cleanup: do not re-define _proxy_ssn variable in local
   #6006 - Rewrote remap_stats plugin to use C++
   #6007 - Graduate TSHttpTxnServerPush to the stable API interface
@@ -771,6 +773,7 @@ Changes with Apache Traffic Server 9.0.0
   #6024 - Make proxy.config.http.request_buffer_enabled configurable and bug 
fix
   #6031 - listen on all the net threads
   #6049 - set host name in TLS extension for SNI Host check in service side 
with sni policy verify_with_name_source.
+  #6053 - Enhance Connection Collapse in ATS core
   #6054 - Ignore MgmtMessageHdr if ProcessManager is not running
   #6062 - Adding connection close headers to regex_remap test.
   #6066 - Fixing session sharing with IP matching.
@@ -825,6 +828,7 @@ Changes with Apache Traffic Server 9.0.0
   #6189 - Fixed issue with macOS Catalina and pcre 8.43 enabling pcre-jit
   #6190 - Fix ssl_session_reuse to compile on macOS and FreeBSD
   #6191 - Limit this check to Linux, where ldd always works
+  #6193 - Fixes various crashers loading/reloading parent.config
   #6198 - Move logging before session could be freed
   #6199 - Set END_STREAM flag when write_vio ntodo is 0
   #6201 - ssl_session_reuse optimization to check if diags is on
@@ -835,7 +839,13 @@ Changes with Apache Traffic Server 9.0.0
   #6218 - Fix stringstream crash during shutdown
   #6224 - Fixed build issue with clang5 and Extendible
   #6226 - Fix sni.yaml fqdn to match complete name string
+  #6227 - Remove never implemented regex descriptions
   #6231 - Adjust the refcounts to avoid Mutex leak
+  #6233 - auto delete rolled log file fixes
   #6247 - Add some stats collections to cache_promote.
   #6251 - Some tweaks to reloading-plugins.en.rst
+  #6260 - Fix TS_USE_DIAGS usage for --disable-diags option
   #6263 - TCL: cleanup in HostLookup.cc, make sure keys are stable.
+  #6264 - Change HTTP/2 error rate log to warning
+  #6265 - LGTM: Add header guards
+  #6266 - LGTM: Fix wrong type of arguments to formatting function