Skip to content

Fix default config#7223

Closed
shinrich wants to merge 718 commits intoapache:masterfrom
shinrich:fix-default-config
Closed

Fix default config#7223
shinrich wants to merge 718 commits intoapache:masterfrom
shinrich:fix-default-config

Conversation

@shinrich
Copy link
Copy Markdown
Member

@djcarlin noticed that our latest ATS9 builds were complaining on start up

[Sep 28 16:06:30.528] traffic_server WARNING: Unrecognized configuration value 'proxy.config.net.max_connections_active_in'

PR #6968 changed the name of that setting to max_requests_in, but the default records.config.in did not get updated.

sudheerv and others added 30 commits May 14, 2020 10:50
When doing redirect  and get cache read miss, we need to get cache write  lock first other than directly do cache write directly.

(cherry picked from commit 36ebb2d)
Fixed ODR violation in test_RemapPluginInfo when using test plugin
plugin_testing_calls.so. Also fixed a typo in specifying plugin
factory build dependencies.

(cherry picked from commit 02e8d5e)
(cherry picked from commit 9f71a80)
…apache#6755)"

This reverts commit 4582cbe.

I don't know what happened here, but the same change came in twice ...
…che#6781)

Set a non-zero default value for TLS Client Handshake Timeout

doc updates to clarify that no activity timeouts will use default
inactivity timeout value when they are set to 0

(cherry picked from commit 2188810)
(cherry picked from commit a01b94b)
(cherry picked from commit 3376d43)

 Conflicts:
	proxy/http2/unit_tests/test_HpackIndexingTable.cc
This change adds tls information nodes like the following:

    "tls": {
	"sni": "<SNI>",
	"verify_mode": "<SSL_VERIFY_MODE_VALUE>"
    },

Co-authored-by: bneradt <bneradt@verizonmedia.com>
(cherry picked from commit bfafd91)
This refactor is occasioned by the need to make transaction writes
atomic. Mainly, this encapsulates session and transaction handling
in SessionData and TransactionData classes.

(cherry picked from commit 59ebfd8)
ssl_session_ticket_index is registerd but unused.

(cherry picked from commit e5d51e4)
…ches (apache#6483)

Rework server side SSL_CTX creation to better handle dual_cert name mismatches

(cherry picked from commit f729c9d)
This leak was introduced by f729c9d

(cherry picked from commit 9fb6f96)
Co-authored-by: Susan Hinrichs <shinrich@verizonmedia.com>
(cherry picked from commit 0265ac7)
This crash was introduced by f729c9d

(cherry picked from commit 96e1f46)
* Add TXN_CLOSE hook to CPPAPI TransactionPlugin

* Clean up TransactionPlugin object and associated Continuation in txn_close

* Address review comments

* More review comments

(cherry picked from commit 34b57fc)
(cherry picked from commit c179620)

 Conflicts:
	iocore/net/UnixNetAccept.cc
… scheduler

update docs

(cherry picked from commit 6a1c5f7)
zwoop and others added 24 commits September 4, 2020 09:22
The MicroDNS extension supports populating the 'otherwise' node with a
list of hosts via the default parameter.  The Traffic Server tests only
pass a string to default, but tests could also pass a list. This makes
the extension deal with either of these input types. As it is, without
this change, tests cannot pass a list because the list itself would be
wrapped as a list.

This change fixes the use of MicroDNS for plugins that use the
Traffic Server extensions, such as txn_box.so.

(cherry picked from commit c637048)
Many AuTests have test plugins or test executables that are used as a
part of the test. Before this change these were built by ad hoc calls to
g++ or gcc via shell executions from within the test or one of a couple
AuTest extensions. This patch removes the building of these files from
AuTest and instead uses the Automake build system to build these. This
will build these utilities in a way that is consistent with the rest of
the system, using the same warning and error flags, the same
clang tool configurations, etc.

(cherry picked from commit 61f8ee3)
Co-authored-by: bneradt <bneradt@verizonmedia.com>
(cherry picked from commit b527464)
)

Susan observed a couple issues with certain configurations now that
we are building AuTest plugins via the Automake system:

1. Without ` --enable-experimental-plugins` and
`--enable-example-plugins`: A couple tests used to build some
experimental plugins by themselves with a copy-and-pasted version of the
experimental plugin in the tools directory. This way the test would run
regardless of whether the user configured their make with
--experimental-plugins. That clearly won't work now that the standard
installed plugins are built with Automake. To address this, I'm adding a
skip check associated with these plugins.

2. Susan also executed `make install` via sudo because she was
installing in the root filesystem and needed root privilege. The
PreparePlugin script copies down the plugins into the sandbox directory.
For all the test plugins this is fine, but for the standard ATS plugins,
this attempt to copy down upon a root sym link would fail with an EPERM.
I'm addressing this by providing both a PrepareTestPlugin and
PrepareInstalledPlugin in which the latter does not copy down the plugin
`.so` file because it is already installed via `make install`.

(cherry picked from commit 7ef20c6)
…rting duplicates to pending dns (apache#7166)

(cherry picked from commit decc63c)
This also adds a pair of log messages for the initial OCSP update.
ATS doesn't respond until this is complete and may give operators an
understanding where it on load.

Fixes issue apache#6801

(cherry picked from commit b353df2)
Before this change, every call to autest.sh would re-install the pipenv
virtual environment. This reuses the virtual environment on subsequent
runs to autest.sh.

(cherry picked from commit 3cfebc9)
(cherry picked from commit e4e3ea3)
…he#7193)

The custom header name is passed to the xdebug plugin as plugin parameter.  This change makes it available
as a global TS API user parameter.

(cherry picked from commit b09e043)
This is a simple plugin (based off of the `generator` plugin) to
serve static content from a local filesystem. It shares some
of the same functionality as the `healthchecks` plugin,
but can be used in the remap context, making it reloadable.

(cherry picked from commit e158590)
Remove references to ATS 2.x and 4.x, update examples for logging to YAML

(cherry picked from commit b8eb810)
It was found that if someone has non-commited Python files, such as can
happen if they have a virtual environment in their source tree, autopep8
will inspect those as well. This is slow and probably not desired by the
user.

This patch only runs autopep8 on files tracked by git. It will also
enable autopep8 to check .test.ext extensions, thus the updates to those
files.

It also has autopep8 run silently so it doesn't produce as much noise.

(cherry picked from commit a815016)
@shinrich shinrich added this to the 10.0.0 milestone Sep 28, 2020
@shinrich shinrich requested a review from sudheerv September 28, 2020 20:24
@shinrich shinrich self-assigned this Sep 28, 2020
@shinrich shinrich closed this Sep 28, 2020
@zwoop zwoop removed this from the 10.0.0 milestone Sep 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.