sync from master. #108
Merged
sync from master. #108
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously I thought it wasn't safe to pass this because it wasn't in a shipped version of clang yet but it turns out I wasn't using `-Xclang`. This solves an issue where if you build a pch, then `touch` a header file that's included in it without actually changing contents, including the pch would fail. Now we let bazel handle that and since it doesn't use timestamps it's a bit smarter about it. Signed-off-by: Keith Smiley <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Commit Message: This PR updates cpp2sky to 0.2.2 and enable to use Apache SkyWalking v8.6.0 Additional Description: Risk Level: Low Testing: N/A Docs Changes: Release Notes: Signed-off-by: Shikugawa <[email protected]>
Commit Message: Validate postgres messages before parsing. Additional Description: Introduced InSync and OutOfSync states in decoder. When decoder detects a wrongly formatted message, it stops parsing and moves to OutOfSync state. Continuing parsing after encountering message with wrong syntax may lead to interpreting random bytes as length of the message and possibly causing OOM. Risk Level: Low Testing: Added unit tests and run full regression tests. Docs Changes: No. Release Notes: No. Platform Specific Features: Fixes #12340 Signed-off-by: Christoph Pakulski <[email protected]>
Remove useless func, now it is only used in test: Signed-off-by: Long Dai <[email protected]>
…ontext` (#16978) This field has been deprecated in favor of of the match_subject_alt_names field which provides more flexible matching. SAN list can still be specified via transport_socket_options's SAN list override and verified via verifySubjectAltName Risk Level: Low Testing: local test on linux (bazel test //test/...) Signed-off-by: Tianyu Xia <[email protected]>
Signed-off-by: Long Dai <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
…construction (#16891) Commit Message: this PR adds the ability to prefetch hostnames to resolve during cache construction. Additional Description: This feature can be used as a performance improvement for setups that use dns cache, but know a priori some of the hostnames that will be used. Risk Level: low -- adds a feature that is off by default. Testing: new tests Docs Changes: added documentation in the API Release Notes: inline Signed-off-by: Jose Nino <[email protected]>
Signed-off-by: giantcroc <[email protected]>
…ctions (#16975) Previously, L7 connections forwarded over TCP didn't get connection failure details (auto_http before codec assignment, CONNECT requests during the entire lifetime) but now they should. Risk Level: low Testing: unit test. Docs Changes: n/a Release Notes: n/a Fixes #/16881 Signed-off-by: Alyssa Wilk <[email protected]>
Signed-off-by: Kevin Baichoo <[email protected]>
Also a small Refactor of StatsHandler::handlerStats(). Signed-off-by: Ryan Hamilton <[email protected]>
…us::operator<< (#17014) Signed-off-by: Yan Avlasov <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
There seem to be some concerns about getenvoy's distributions, and most of them are moot at this point. As the getenvoy project stops building and publishing envoy, we should make sure users are aware where "official" things are. Right now, there are official and more maintained alternatives for everything except system packages. This removes the website links of getenvoy for everything except system packages. It also removes the section about nightlies as I don't think they are being maintained and certainly won't be moving forward. See: #16867 #16830 (comment) Signed-off-by: Adrian Cole <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Commit Message: pass dispatcher and transport socket factory into createEnvoyQuicCryptoServerStream(). These two parameters are not used in the default extension, but can be used for other extensions if needed. Risk Level: low Testing: existing test Signed-off-by: Dan Zhang <[email protected]>
Commit Message: add quic version counters in http3 codec stats. Additional Description: Risk Level: Low Testing: Integration tests Docs Changes: docs/root/configuration/http/http_conn_man/stats.rst Signed-off-by: Renjie Tang <[email protected]>
Feature unsupported on Windows Signed-off-by: Abhay Narayan Katare <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Rama Chavali <[email protected]>
docs: link additional filters that emit dynamic metadata Additional Description: N/A Risk Level: N/A Testing: Manually build docs and verified link targets Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A Signed-off-by: James Peach <[email protected]>
The listener filter may add event on the new socket, but it won't cleanup the event. If continue_on_listener_filters_timeout is set, the new connection may add same event to the socket. So reset the file event before initialize new one. Signed-off-by: He Jie Xu <[email protected]>
#16816) When SAN(SubjectAltName) matching with the provided matchers is the only verification performed and the matching succeeds, the `validated` flag is not updated properly(left as `NotValidated`) and the function return wrong statuscode. Signed-off-by: Tianyu Xia <[email protected]>
Raising error bar to silence flakes until #17067 is fully addressed Signed-off-by: Alyssa Wilk <[email protected]>
Signed-off-by: chaoqin-li1123 <[email protected]>
Signed-off-by: wbpcode <[email protected]> Sub-PR of #16049. Check #16049 get more background information. This PR designed a new generic abstraction `TraceContext` to replace `Http::RequestHeaderMap` to provide tracing context to the tracer driver. `Http::RequestHeaderMapImpl` already inherits from TraceContext and implements the relevant interfaces. Next, we just need simply replace `Http::RequestHeaderMap` with `TraceContext` in all tracer drivers implementations. After that, the main body of the entire general tracing system is completed. I'm not sure, whether the replacement step requires a separate PR. Because it does not involve any new logic, but there will be a lot of file changes. Risk Level: Low. Testing: Added. Docs Changes: N/A Release Notes: N/A Signed-off-by: wbpcode <[email protected]>
Signed-off-by: Matt Klein <[email protected]>
) Signed-off-by: Piotr Sikora <[email protected]>
Signed-off-by: Piotr Sikora <[email protected]>
This is a new extension point that will allow for dynamic cluster selection. The plugin configured by it will be responsible for returning a cluster to Envoy through its API (to be defined). Since the cluster returned is dynamic, a new field is added to Route to list all the clusters it may return. This allows for proxy implementations that are not using wildcard CDS queries to pre-fetch clusters. This feature may ultimately replace the FilterAction mechanism, as it provides the same functionality for known use cases, but is simpler to implement and use. Signed-off-by: Doug Fawley <[email protected]>
Signed-off-by: Alyssa Wilk <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Mark D. Roth <[email protected]>
Prior to this PR the MatchTreeFactory requried a ServerFactoryContext in order to pass this to the action factories. This is not available in all contexts, and there are other possible use cases where we might need to pass additional information to the match tree factory (e.g. the router might need to pass the parent vhost configuration). This PR introduces a paramterized action context, allowing each usage of the MatchTree to define its own data that should be presented to the action factories. This also has the nice benefit of partitioning the action factories per context: each unique ActionFactoryContext defines a new factory type, ensuring that actions defined for a HTTP filter context won't be conflated with actions defined for another context. Signed-off-by: Snow Pettersen <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
…" (#17191) This reverts commit be1cd7b. Signed-off-by: Matt Klein <[email protected]>
…etwork instead of bare new() (#17177) Risk Level: Low Testing: None - no behavior change Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A Signed-off-by: Ryan Hamilton <[email protected]>
Before the connection reach to TCPPorxy filter or HCM, the server name won't be populated into stream info. It leads to the access log get empty server name. This patch populate the server name in the path of closing socket. Signed-off-by: He Jie Xu <[email protected]>
…red (#17195) Signed-off-by: qinggniq <[email protected]>
Signed-off-by: Justin Ely <[email protected]>
Signed-off-by: Sotiris Nanopoulos <[email protected]>
Moves the format tests to the tooling job - this prevents eg the tests being called repeatedly in fix/test calls, speeds up precheck a bit (by making pre-check jobs more parallel) and will make it easier to optimize these tests Signed-off-by: Ryan Northey <[email protected]>
If apt can't update the package index, it doesn't exit with an error by default, so add the `--error-on=any` flag to force an error. Pass the `--yes` flag to the install to force non-interactive mode. Signed-off-by: James Peach <[email protected]>
Signed-off-by: Kevin Baichoo <[email protected]>
Increase common/common coverage by: Removing CompiledStdMatcher partially cleaned by regex: remove parseStdRegexAsCompiledMatcher #17061 Add test for ContainsReference in stl_helpers Signed-off-by: Sotiris Nanopoulos <[email protected]>
…17118) Signed-off-by: Greg Greenway <[email protected]>
) During dynamic endpoint updates via EDS, `BaseDynamicClusterImpl::updateDynamicHostList` method tries to minimize deletions/insertions. If only metadata was changed, endpoint is updated in-place by calling `HostDescriptionImpl::metadata`, which updates metadata but not fields dependent on metadata. In particular, if metadata's transport socket matcher was changed, socket_factory was not updated. Risk Level: Low Testing: added unit test Docs Changes: no Release Notes: no Platform Specific Features: Fixes #16536 Signed-off-by: Christoph Pakulski <[email protected]>
Previously, the setBuffer API was misused and the size of a new data was passed instead of the size of data to replace. Signed-off-by: Piotr Sikora <[email protected]>
Signed-off-by: Paul Gallagher <[email protected]>
Signed-off-by: Kuat Yessenov <[email protected]>
…7127) Signed-off-by: Shikugawa <[email protected]>
Signed-off-by: Abhay Narayan Katare <[email protected]>
…16752) Signed-off-by: Xuyang Tao <[email protected]>
…#17179) Commit Message: upstream -- allow clusters to skip waiting on warmup for initialization Additional Description: this reduces server startup time, but might cause initial 500s for requests made on unresolved clusters. Further work might be needed there if operators use this new config option. Risk Level: low - new config option. Default uses legacy behavior. Testing: added unit tests Docs Changes: added. Release Notes: updated Signed-off-by: Jose Nino <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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:]