sync from master. by wangfakang · Pull Request #9 · wangfakang/envoy

wangfakang · 2021-03-31T06:30:37Z

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

…ess log, tracing, admin) (#15139) * Re-purpose alt_stat_name for other observability use-cases: access logging, tracing, and admin dumps through a new ClusterInfo field, `observabilityName()`. Signed-off-by: Asra Ali <[email protected]>

… connection creation. (#15005) Checking the runtime feature on every packet sent results in excessive lock contention and could trigger incorrect termination of HTTP2 upstream connections if the value of the runtime feature changes halfway through the lifetime of the upstream connection. Signed-off-by: Antonio Vicente <[email protected]>

Moved the db generation into a bazel job - which will allow other tasks that also require this data to make use of it more easily/bazely this also adds buildozer into the bazel env, so we might want to make use of that elsewhere and potentially remove buildozer from the build-image env Touch #13229 Signed-off-by: Ryan Northey <[email protected]>

Introduce user space io socket handle on top of user space event at #14712 Userspace Io socket handles are always created with a peering handle. The write() methods populate the buffer in the peer handle. The read() methods consume the owned buffer. Signed-off-by: Yuchen Dai <[email protected]>

Commit Message: fix some issues around duplicated http headers: Quiche doesn't coalesce duplicated headers, so we need to coalesce them in Envoy. Cookie headers after coalescing are semi-colon separated, while Set-Cookie headers shouldn't be coalesced. Risk Level: low Testing: added unit tests and integration tests. Part of #2557 Co-authored-by: Dan Zhang <[email protected]>

When filter doesn't support virtual host-specific typed per filter config, the envoy should reject any per filter config for that filter. Also, adds new runtime config envoy.reloadable_features.check_unsupported_typed_per_filter_config for backward-compatible. The default value is true, only when the value is true, the check will be executed. Risk Level: low Testing: unit test added Docs Changes: n/a Release Notes: note new feature Runtime guard: envoy.reloadable_features.check_unsupported_typed_per_filter_config Fixes #15043 Signed-off-by: He Jie Xu <[email protected]>

Avoiding some (expected) 5s test timeouts and future-proofing. Risk Level: n/a (test only) Testing: yes Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <[email protected]>

- Add DependencyManager class to handle http filterchain validation - This class currently only supports decode path validation, but encode path validation will be trivial to add afterwards - This class is directly scoped to the hcm extension and http filters, but it can be abstracted out when filterchain dependency is implemented for l4. Risk Level: Low, this only adds a class and unit tests See design doc attached to #14470 for more details. Signed-off-by: Auni Ahsan <[email protected]>

Signed-off-by: Alyssa Wilk <[email protected]>

This patch adds a new custom flag `%j` to the log pattern to printthe actual message to log, but as escaped JSON string. Signed-off-by: Dhi Aurrahman <[email protected]>

Signed-off-by: Takeshi Yoneda <[email protected]>

…15237) Currently ClusterManagerInitHelper.secondary_init_clusters_ is a list. Upon every insert the list gets searched for an already added cluster with the same name. Since in normal circumstances all new clusters have unique names the worst case scenario is triggered and all elements of the list are checked sequentially upon every cluster added. Replace the list with a hash map. Signed-off-by: Dmitry Rozhkov <[email protected]>

Signed-off-by: Yuchen Dai <[email protected]>

…trust domain support (#14884) Adds the extension point for certificate validations, and its first implementation for SPIFFE multi trust domain support in a single listener or cluster. Resolves #14614 and #9284. Risk Level: low (only adding the new extension point and one implementation for it) Testing: unit tests and integration tests. Docs Changes: Release Notes: tls: implement SPIFFE Certificate Validator for independent multiple trust domain support. Signed-off-by: Takeshi Yoneda <[email protected]>

This is working end to end for a number of the protocol integration tests, but still needs a bunch of work before it's production ready (I triaged some of the excludes but not all, watermarks not working etc). I think given this works for the happy path it's worth checking in, and I can iterate on getting individual tests working and doing the TODOs. Risk Level: low (minor core refactors, major changes are all behind hidden config) Testing: integration tests, some unit tests Docs Changes: n/a Release Notes: n/a #14829 Signed-off-by: Alyssa Wilk <[email protected]>

Commit Message: delete unreferenced conn_manager_impl_common.h Additional Description: This file doesn't appear in any BUILD files and isn't referenced by any #includes in Envoy. Risk Level: low Testing: git grep Docs Changes: n/a Release Notes: n/a Platform Specific Features: n/a Signed-off-by: Alex Konradi <[email protected]>

Commit Message: Add pragma once almost every header. Risk Level: LOW Testing: self-contained Docs Changes: Release Notes: Platform Specific Features: [Optional Runtime guard:] Part of #15264 [Optional Deprecated:] [Optional API Considerations:] Additional Description: Signed-off-by: Yuchen Dai <[email protected]>

Commit Message: fix tlsVersion() and typo Additional Description: Risk Level: Testing: Docs Changes: Signed-off-by: Yuchen Dai <[email protected]>

Commit Message: update QUICHE to commit 6fec8e97a1ab1a945b86bd6b7252666294448ddb and boring SSL commit b049eae83d25977661556dcd913b35fbafb3a93a. Additional Description: the latest QUICHE commit depends on broingSSL commit: https://boringssl.googlesource.com/boringssl/+/3d8b8c3df27cc780a49607be6a27f2c518f5c082 which is in chrome dev channel. Risk Level: low Testing: existing tests passed Signed-off-by: Dan Zhang <[email protected]> Co-authored-by: Dan Zhang <[email protected]>

afaict the current rules_python has quite a few things that dont work correctly - namespaced packages, entrypoints and wheels eg version 0.1.0 at least seems to fix the issue of namespaced packages, and possibly other problems too Signed-off-by: Ryan Northey <[email protected]>

This allows us to more easily test response crashes. Risk Level: low (this is a test-only filter AFAIK) Testing: unit test, integration test Docs Changes: Included Release Notes: Included Platform Specific Features: N/A Related Issue #14249 Signed-off-by: Kevin Baichoo <[email protected]>

This lets a custom handshaker configure the SSL_CTX object after creation, but before it is used to create any SSL objects. Per the BoringSSL docs, "|SSL_CTX| are reference-counted and may be shared by connections across multiple threads. Once shared, functions which change the |SSL_CTX|'s configuration may not be used." Additionally, the SSL_CTX object is complex enough and has enough methods that, rather than expose each of them via another method on some wrapper class, it is convenient to just allow mutating access via a callback during socket creation (as in tls/context_impl.cc` in this PR). This PR also provides a reference to the HandshakerFactoryContext to this sslctx_cb callback at runtime so that a custom handshaker can further inject SSL_CTX configuration behavior. Risk Level: Low Testing: n/a Docs Changes: n/a Release Notes: n/a Platform Specific Features: n/a Signed-off-by: James Buckland <[email protected]>

Risk Level: n/e (test only) Testing: n/a Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <[email protected]>

Signed-off-by: Alyssa Wilk <[email protected]>

The "json_serialize_arena" test was non-representative, and it was running out of memory in WasmVM during benchmarks, leading to invalid results. The "json_deserialize_arena" test was measuring exactly the same process as the "json_deserialize" test, so it was redundant. The "json_serialize_deserialize" test was a combinantion of "json_serialize" and "json_deserialize" tests, so it was redundant. While there, make sure that inputs are not used as outputs, and vice-versa. Signed-off-by: Piotr Sikora <[email protected]>

… integers (#15256) Risk Level: low Unit Tests: updated Docs: n/a Release Notes: updated Fixes #15255 Signed-off-by: Derek Argueta <[email protected]>

Commit Message: Provide the host info in the onLoadDnsCacheComplete callback so that the worker threads do not have to do an additional lookup to get the resolved IP. Additional Description: Risk Level: Low - no change to functionality and code modifications are minimal Testing: Unit testing Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A Signed-off-by: Craig Radcliffe <[email protected]>

) Signed-off-by: Kevin Baichoo <[email protected]>

Due to #14204, doing static cluster checks for AsyncClientFactory can only be done on the main thread. Signed-off-by: Michael Wozniak <[email protected]>

Signed-off-by: Alyssa Wilk <[email protected]>

Bumps [jinja2](https://github.com/pallets/jinja) from 2.11.2 to 2.11.3. - [Release notes](https://github.com/pallets/jinja/releases) - [Changelog](https://github.com/pallets/jinja/blob/master/CHANGES.rst) - [Commits](pallets/jinja@2.11.2...2.11.3) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

This adds the foundations of support for glob collections in CDS/SRDS (in addition to LDS from #14311) and singleton collections in RDS/EDS. xdstp:// URLs are not normalized across the internal resource graph yet, so this is only safe to use for sorted context parameters (followup PRs planned on this). Part of #11264. Risk level: Low (opt-in) Testing: ADS/SRDS integration tests added, various unit tests in //test/common/config/... Signed-off-by: Harvey Tuch <[email protected]>

Move active_tcp_listener out of conn handler. Similar to #15349 but for tcp listener. Fixes #15126 Signed-off-by: Yuchen Dai <[email protected]>

Signed-off-by: Sotiris Nanopoulos <[email protected]>

Signed-off-by: Ryan Northey <[email protected]>

1) Add new stateful header formatter extension point 2) Add preserve case formatter extension Fixes #14363 Signed-off-by: Matt Klein <[email protected]>

…ity (#15719) Signed-off-by: Jose Nino <[email protected]>

Signed-off-by: mindyor <[email protected]>

Signed-off-by: Piotr Sikora <[email protected]>

…gy (#15721) Signed-off-by: Sotiris Nanopoulos <[email protected]>

* docs: add comments about inherited options on SPIFFE validator Signed-off-by: Takeshi Yoneda <[email protected]>

Part of #12829. The transport extension will stay a built-in extension since it fits well. UDP listener and UDP writer extension points have been removed. GSO is still only enabled for QUIC because it currently depends on QUICHE, has some obvious perf issues, and is failing non-QUIC integration tests. Futher work is needed to remove codec extension factories. Part of #12829 Signed-off-by: Matt Klein <[email protected]>

Signed-off-by: Mike Schore <[email protected]>

… caught on main thread (#15251) Signed-off-by: chaoqin-li1123 <[email protected]> Commit Message: Define a macros that wrap try with an assertion that exceptions are only caught in main thread. Currently, envoy use c++ exception for error propagation. This PR is one of the steps to address #14320. The long term goal is to disallow raw try in envoy core code and eliminate c++ exception from data plane, which can improve exception safety. The try in the PR happen on main thread and can be wrapped in main thread assertion without breaking any existing test. In the following PR, raw try that can not be replaced by TRY_ASSERT_MAIN_THREAD will be removed from core codebase with other error propagation. Additional Description: Risk Level: Testing: none Docs Changes: none Release Notes: none Platform Specific Features: none

Signed-off-by: Dan Zhang <[email protected]>

… on main thread. (#15548) During the v2 -> v3 migration, some uses of this were added inside filter factory lambdas and could occur on worker threads, rejecting config on the data plane. Relevant to #10943 Fixes #15083 Risk level: Low Testing: New assertion in method caused existing tests to fail, fix addresses these. Also manually audited via grep. Signed-off-by: Harvey Tuch <[email protected]>

Naive connectivity grid, which works for TCP/UDP and only does serial connections. Risk Level: n/a (unused code) Testing: new unit tests Docs Changes: n/a Release Notes: n/a Part of #15649 Signed-off-by: Alyssa Wilk <[email protected]>

This was missed in #14512. Signed-off-by: Raul Gutierrez Segales <[email protected]>

Tagging a bunch of quic test failures with more details. Adapting the status code tests to QUIC and fixing QUIC code to validate. minor refactors to core code. Risk Level: low (minor core refactors) Testing: new integration tests Docs Changes: n/a Release Notes: n/a part of #14829 Signed-off-by: Alyssa Wilk <[email protected]>

Signed-off-by: Alyssa Wilk <[email protected]>

Signed-off-by: Ryan Northey <[email protected]>

…voyproxy#22856) `//test/integration:tcp_proxy_odcds_integration_test` was observed to fail as follows: ``` ==================== Test output for //test/integration:tcp_proxy_odcds_integration_test: [==========] Running 24 tests from 1 test suite. [----------] Global test environment set-up. [----------] 24 tests from IpVersionsClientType/TcpProxyOdcdsIntegrationTest [ RUN ] IpVersionsClientType/TcpProxyOdcdsIntegrationTest.SingleTcpClient/0 [2022-08-25 20:22:46.750][3969][critical][assert] [test/integration/fake_upstream.cc:832] assert failure: !dispatcher_->isThreadSafe(). [2022-08-25 20:22:46.752][3969][critical][backtrace] [./source/server/backtrace.h:104] Caught Aborted, suspect faulting address 0x6b00000f81 [2022-08-25 20:22:46.752][3969][critical][backtrace] [./source/server/backtrace.h:91] Backtrace (use tools/stack_decode.py to get line numbers): [2022-08-25 20:22:46.752][3969][critical][backtrace] [./source/server/backtrace.h:92] Envoy version: 0/1.24.0-dev/test/DEBUG/BoringSSL [2022-08-25 20:22:46.773][3969][critical][backtrace] [./source/server/backtrace.h:96] #0: Envoy::SignalAction::sigHandler() [0x4c46a78]->[0x2cf2a78] external/com_google_googletest/googlemock/include/gmock/gmock-spec-builders.h:1267 [2022-08-25 20:22:46.773][3969][critical][backtrace] [./source/server/backtrace.h:96] #1: __restore_rt [0x7ffbdaa79420]->[0x7ffbd8b25420] ??:0 [2022-08-25 20:22:46.802][3969][critical][backtrace] [./source/server/backtrace.h:96] #2: Envoy::FakeUpstream::assertPendingConnectionsEmpty() [0x245bf0b]->[0x507f0b] ??:0 [2022-08-25 20:22:46.846][3969][critical][backtrace] [./source/server/backtrace.h:96] #3: Envoy::(anonymous namespace)::TcpProxyOdcdsIntegrationTest_SingleTcpClient_Test::TestBody() [0x1f596cb]->[0x56cb] ??:0 [2022-08-25 20:22:46.877][3969][critical][backtrace] [./source/server/backtrace.h:96] #4: testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x588b61b]->[0x393761b] ??:0 [2022-08-25 20:22:46.924][3969][critical][backtrace] [./source/server/backtrace.h:96] #5: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x587c2bd]->[0x39282bd] ??:0 [2022-08-25 20:22:46.966][3969][critical][backtrace] [./source/server/backtrace.h:96] #6: testing::Test::Run() [0x5864ba3]->[0x3910ba3] /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_vector.h:0 [2022-08-25 20:22:47.003][3969][critical][backtrace] [./source/server/backtrace.h:96] #7: testing::TestInfo::Run() [0x586576a]->[0x391176a] external/com_google_absl/absl/container/internal/raw_hash_set.h:1259 [2022-08-25 20:22:47.037][3969][critical][backtrace] [./source/server/backtrace.h:96] #8: testing::TestSuite::Run() [0x5865fbb]->[0x3911fbb] /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/stl_algo.h:1925 [2022-08-25 20:22:47.089][3969][critical][backtrace] [./source/server/backtrace.h:96] #9: testing::internal::UnitTestImpl::RunAllTests() [0x5874a28]->[0x3920a28] envoy/registry/registry.h:509 [2022-08-25 20:22:47.114][3969][critical][backtrace] [./source/server/backtrace.h:96] #10: testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x588ddbb]->[0x3939dbb] envoy/registry/registry.h:0 [2022-08-25 20:22:47.160][3969][critical][backtrace] [./source/server/backtrace.h:96] #11: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x587e683]->[0x392a683] /usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/bits/unique_ptr.h:154 [2022-08-25 20:22:47.191][3969][critical][backtrace] [./source/server/backtrace.h:96] #12: testing::UnitTest::Run() [0x5874568]->[0x3920568] envoy/registry/registry.h:508 [2022-08-25 20:22:47.237][3969][critical][backtrace] [./source/server/backtrace.h:96] #13: RUN_ALL_TESTS() [0x4878d51]->[0x2924d51] external/com_google_googletest/googlemock/include/gmock/gmock-spec-builders.h:1203 [2022-08-25 20:22:47.284][3969][critical][backtrace] [./source/server/backtrace.h:96] #14: Envoy::TestRunner::RunTests() [0x48783b1]->[0x29243b1] external/com_google_googletest/googlemock/include/gmock/gmock-actions.h:485 [2022-08-25 20:22:47.316][3969][critical][backtrace] [./source/server/backtrace.h:96] #15: main [0x4874c3a]->[0x2920c3a] external/com_google_googletest/googlemock/include/gmock/gmock-spec-builders.h:1181 [2022-08-25 20:22:47.316][3969][critical][backtrace] [./source/server/backtrace.h:96] #16: __libc_start_main [0x7ffbda897083]->[0x7ffbd8943083] ??:0 ================================================================================ ``` This is due to the race described by envoyproxy#22855. Making sure the dispatcher thread is running before starting the test avoids this problem. Signed-off-by: Benjamin Peterson <[email protected]>

This test sends a large number of metadata frames in order to trigger a disconnect. However, it was possible for the disconnect to happen and the connection to be torn down before all the metadata frames had been sent. If that happened, ASAN detected a UAF: ``` ==95==ERROR: AddressSanitizer: heap-use-after-free on address 0x60700037e5a0 at pc 0x000004811f9e bp 0x7ffc903af990 sp 0x7ffc903af988 READ of size 8 at 0x60700037e5a0 thread T0 #0 0x4811f9d in Envoy::IntegrationCodecClient::sendMetadata(Envoy::Http::RequestEncoder&, Envoy::Http::MetadataMap) /proc/self/cwd/test/integration/http_integration.cc:168:3 #1 0x46ed711 in Envoy::Http2FloodMitigationTest_RequestMetadata_Test::TestBody() /proc/self/cwd/test/integration/http2_flood_integration_test.cc:1486:20 wangfakang#2 0xd380e64 in void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2580:10 wangfakang#3 0xd348dc2 in testing::Test::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2655:5 wangfakang#4 0xd34a927 in testing::TestInfo::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2832:11 wangfakang#5 0xd34ccc4 in testing::TestSuite::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2986:28 wangfakang#6 0xd36f07a in testing::internal::UnitTestImpl::RunAllTests() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5697:44 wangfakang#7 0xd384e63 in bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) /proc/self/cwd/external/com_google_googletest/googletest/src/ gtest.cc:2580:10 wangfakang#8 0xd36dd86 in testing::UnitTest::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5280:10 wangfakang#9 0xa0e53a4 in Envoy::TestRunner::RunTests(int, char**) /proc/self/cwd/external/com_google_googletest/googletest/include/gtest/gtest.h:2485:46 wangfakang#10 0xa0e0af7 in main /proc/self/cwd/test/main.cc:34:10 wangfakang#11 0x7f442ef69082 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x24082) (BuildId: 1878e6b475720c7c51969e69ab2d276fae6d1dee) wangfakang#12 0x45ed36d in _start (/mnt/ssd/cas/work/1/exec/bazel-out/k8-dbg/bin/test/integration/http2_flood_integration_test.runfiles/envoy/test/integration/http2_flood_integration_test+0x45ed36d) 0x60700037e5a0 is located 48 bytes inside of 80-byte region [0x60700037e570,0x60700037e5c0) freed by thread T0 here: #0 0x466f7d2 in free /local/mnt/workspace/bcain_clang_hu-bcain-lv_22036/final/llvm-project/compiler-rt/lib/asan/asan_malloc_linux.cpp:52:3 #1 0x831dde8 in Envoy::Http::CodecClient::ActiveRequest::~ActiveRequest() /proc/self/cwd/./source/common/http/codec_client.h:220:10 wangfakang#2 0x5aa33f9 in std::__1::unique_ptr<Envoy::Event::DeferredDeletable, std::__1::default_delete<Envoy::Event::DeferredDeletable> >::reset(Envoy::Event::DeferredDeletable*) /opt/llvm/bin/../include/c++/v1/__memory/unique_ptr.h:54:5 wangfakang#3 0xa3218e8 in Envoy::Event::DispatcherImpl::clearDeferredDeleteList() /proc/self/cwd/source/common/event/dispatcher_impl.cc:142:21 wangfakang#4 0xa3348df in void std::__1::__invoke_void_return_wrapper<void, true>::__call<Envoy::Event::DispatcherImpl::DispatcherImpl(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, Envoy::Thread::ThreadFactory&, Envoy::TimeSource& , Envoy::Random::RandomGenerator&, Envoy::Filesystem::Instance&, Envoy::Event::TimeSystem&, std::__1::function<std::__1::unique_ptr<Envoy::Event::ScaledRangeTimerManager, std::__1::default_delete<Envoy::Event::ScaledRangeTimerManager> > (Envoy::Event::Dispatcher&)> const& , std::__1::shared_ptr<Envoy::Buffer::WatermarkFactory> const&)::$_2&>(Envoy::Event::DispatcherImpl::DispatcherImpl(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, Envoy::Thread::ThreadFactory&, Envoy::TimeSource&, Envoy::Rand om::RandomGenerator&, Envoy::Filesystem::Instance&, Envoy::Event::TimeSystem&, std::__1::function<std::__1::unique_ptr<Envoy::Event::ScaledRangeTimerManager, std::__1::default_delete<Envoy::Event::ScaledRangeTimerManager> > (Envoy::Event::Dispatcher&)> const&, std::__1::s hared_ptr<Envoy::Buffer::WatermarkFactory> const&)::$_2&) /proc/self/cwd/source/common/event/dispatcher_impl.cc:79:30 wangfakang#5 0xa334603 in std::__1::__function::__func<Envoy::Event::DispatcherImpl::DispatcherImpl(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, Envoy::Thread::ThreadFactory&, Envoy::TimeSource&, Envoy::Random::RandomGenerator&, Envoy::Filesystem::Instance&, Envoy::Event::TimeSystem&, std::__1::function<std::__1::unique_ptr<Envoy::Event::ScaledRangeTimerManager, std::__1::default_delete<Envoy::Event::ScaledRangeTimerManager> > (Envoy::Event::Dispatcher&)> const&, std::__1::shared_ptr<Envoy::Buffe r::WatermarkFactory> const&)::$_2, std::__1::allocator<Envoy::Event::DispatcherImpl::DispatcherImpl(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, Envoy::Thread::ThreadFactory&, Envoy::TimeSource&, Envoy::Random::RandomGenera tor&, Envoy::Filesystem::Instance&, Envoy::Event::TimeSystem&, std::__1::function<std::__1::unique_ptr<Envoy::Event::ScaledRangeTimerManager, std::__1::default_delete<Envoy::Event::ScaledRangeTimerManager> > (Envoy::Event::Dispatcher&)> const&, std::__1::shared_ptr<Envoy: :Buffer::WatermarkFactory> const&)::$_2>, void ()>::operator()() /opt/llvm/bin/../include/c++/v1/__functional/function.h:180:16 wangfakang#6 0x4897039 in std::__1::__function::__value_func<void ()>::operator()() const /opt/llvm/bin/../include/c++/v1/__functional/function.h:507:16 wangfakang#7 0xa8e6aa4 in Envoy::Event::SchedulableCallbackImpl::SchedulableCallbackImpl(Envoy::CSmartPtr<event_base, &(event_base_free)>&, std::__1::function<void ()>)::$_0::__invoke(int, short, void*) /opt/llvm/bin/../include/c++/v1/__functional/function.h:1184:12 wangfakang#8 0xb557c5e in event_process_active_single_queue /mnt/ssd/cas/work/2/exec/external/com_github_libevent_libevent/event.c:1713:4 wangfakang#9 0xb539252 in event_process_active /mnt/ssd/cas/work/2/exec/external/com_github_libevent_libevent/event.c wangfakang#10 0xb539252 in event_base_loop /mnt/ssd/cas/work/2/exec/external/com_github_libevent_libevent/event.c:2047:12 wangfakang#11 0xa8e1e3c in Envoy::Event::LibeventScheduler::run(Envoy::Event::Dispatcher::RunType) /proc/self/cwd/source/common/event/libevent_scheduler.cc:60:3 wangfakang#12 0xa32bd94 in Envoy::Event::DispatcherImpl::run(Envoy::Event::Dispatcher::RunType) /proc/self/cwd/source/common/event/dispatcher_impl.cc:299:19 wangfakang#13 0x480faad in Envoy::IntegrationCodecClient::flushWrite() /proc/self/cwd/test/integration/http_integration.cc:100:29 wangfakang#14 0x4811e94 in Envoy::IntegrationCodecClient::sendMetadata(Envoy::Http::RequestEncoder&, Envoy::Http::MetadataMap) /proc/self/cwd/test/integration/http_integration.cc:169:3 wangfakang#15 0x46ed711 in Envoy::Http2FloodMitigationTest_RequestMetadata_Test::TestBody() /proc/self/cwd/test/integration/http2_flood_integration_test.cc:1486:20 wangfakang#16 0xd380e64 in void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2580:10 wangfakang#17 0xd348dc2 in testing::Test::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2655:5 wangfakang#18 0xd34a927 in testing::TestInfo::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2832:11 wangfakang#19 0xd34ccc4 in testing::TestSuite::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2986:28 wangfakang#20 0xd36f07a in testing::internal::UnitTestImpl::RunAllTests() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5697:44 wangfakang#21 0xd384e63 in bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:2580:10 wangfakang#22 0xd36dd86 in testing::UnitTest::Run() /proc/self/cwd/external/com_google_googletest/googletest/src/gtest.cc:5280:10 wangfakang#23 0xa0e53a4 in Envoy::TestRunner::RunTests(int, char**) /proc/self/cwd/external/com_google_googletest/googletest/include/gtest/gtest.h:2485:46 wangfakang#24 0xa0e0af7 in main /proc/self/cwd/test/main.cc:34:10 wangfakang#25 0x7f442ef69082 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x24082) (BuildId: 1878e6b475720c7c51969e69ab2d276fae6d1dee) ``` To fix that, write all metadata frames at once. Signed-off-by: Benjamin Peterson <[email protected]>

asraa and others added 30 commits March 2, 2021 12:52

test: avoiding some long timeouts (#15261)

460cbc5

Avoiding some (expected) 5s test timeouts and future-proofing. Risk Level: n/a (test only) Testing: yes Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <[email protected]>

ci: disable wasm for osx (#15262)

1af68f6

Signed-off-by: Alyssa Wilk <[email protected]>

log: Add "%j" to print message as JSON escaped string (#15226)

08d01f1

This patch adds a new custom flag `%j` to the log pattern to printthe actual message to log, but as escaped JSON string. Signed-off-by: Dhi Aurrahman <[email protected]>

wasm: refactor common codes around configurations (#15210)

3f54cd8

Signed-off-by: Takeshi Yoneda <[email protected]>

test: split tcp_proxy test into config test and data test (#15067)

abba6e4

Signed-off-by: Yuchen Dai <[email protected]>

doc: lua filter (#15277)

d83c3dd

Commit Message: fix tlsVersion() and typo Additional Description: Risk Level: Testing: Docs Changes: Signed-off-by: Yuchen Dai <[email protected]>

http: adding another smuggling test (#15244)

98c9dc7

Risk Level: n/e (test only) Testing: n/a Docs Changes: n/a Release Notes: n/a Signed-off-by: Alyssa Wilk <[email protected]>

test: fixing CI (#15282)

5e0a9e2

Signed-off-by: Alyssa Wilk <[email protected]>

access_log: fix substition formatter to recognize commands containing…

671e436

… integers (#15256) Risk Level: low Unit Tests: updated Docs: n/a Release Notes: updated Fixes #15255 Signed-off-by: Derek Argueta <[email protected]>

HTTP: Associate Response crashes with the corresponding Request. (#15029

15f6b7a

) Signed-off-by: Kevin Baichoo <[email protected]>

AsyncClientFactory: use getThreadLocalCluster for thread safety (#15227)

296d106

Due to #14204, doing static cluster checks for AsyncClientFactory can only be done on the main thread. Signed-off-by: Michael Wozniak <[email protected]>

quic: disabling one more flake (#15294)

2a701b7

Signed-off-by: Alyssa Wilk <[email protected]>

dependabot bot and others added 27 commits March 25, 2021 11:50

listener: move active_tcp_listener out of conn handler (#15553)

fff2cf8

Move active_tcp_listener out of conn handler. Similar to #15349 but for tcp listener. Fixes #15126 Signed-off-by: Yuchen Dai <[email protected]>

Adds Access loggers to admin interface (#15461)

0564107

Signed-off-by: Sotiris Nanopoulos <[email protected]>

deps: Add dependabot.yml (#15651)

2eb6359

Signed-off-by: Ryan Northey <[email protected]>

deps: Resolve conflicting python updates (#15703)

fcdaf20

Signed-off-by: Ryan Northey <[email protected]>

http: add HTTP/1.1 case preservation (#15619)

2a4d97c

1) Add new stateful header formatter extension point 2) Add preserve case formatter extension Fixes #14363 Signed-off-by: Matt Klein <[email protected]>

RaiiMapOfListElement: fix invalid reference due to rehashing instabil…

5025348

…ity (#15719) Signed-off-by: Jose Nino <[email protected]>

lua: timestamp in millis (#14522)

486cd88

Signed-off-by: mindyor <[email protected]>

wasm: allow using empty vm_id to indicate the current WasmVM. (#15359)

87f9309

Signed-off-by: Piotr Sikora <[email protected]>

wasm: add support for resuming downstream connection. (#15393)

d118f80

Signed-off-by: Piotr Sikora <[email protected]>

Rename access log streams to make it more friendly to posix terminolo…

98fb5d6

…gy (#15721) Signed-off-by: Sotiris Nanopoulos <[email protected]>

docs: add comments about inherited options on SPIFFE validator (#15621)

b778119

* docs: add comments about inherited options on SPIFFE validator Signed-off-by: Takeshi Yoneda <[email protected]>

conn_pool_base: minor fix for warnings building on android-x86 (#15751)

29739fc

Signed-off-by: Mike Schore <[email protected]>

quiche: prevent nested buffer watermark accounting (#14259)

964e2a0

Signed-off-by: Dan Zhang <[email protected]>

access log: support formatter extensions from json_format (#15646)

be51d87

This was missed in #14512. Signed-off-by: Raul Gutierrez Segales <[email protected]>

connect: docs update (#15775)

19e98a1

Signed-off-by: Alyssa Wilk <[email protected]>

lint: All text files for general formatting (#15641)

20e4dd2

Signed-off-by: Ryan Northey <[email protected]>

wangfakang merged commit 95c509b into wangfakang:master Mar 31, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

sync from master.#9

sync from master.#9
wangfakang merged 408 commits intowangfakang:masterfrom
envoyproxy:main

wangfakang commented Mar 31, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Conversation

wangfakang commented Mar 31, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants