Ryan Hamilton

Results 13 issues of Ryan Hamilton

Add a filter for applying Android socket tags Testing: Need help Docs Changes: Release Notes:

Signed-off-by: Ryan Hamilton Risk Level: Low Testing: N/A Docs Changes: N/A Release Notes: N/A

Since certificate verification can be quite slow and can be on the critical path for an app, it would be great to use a simple memcmp instead. To do this,...

perf

Signed-off-by: Ryan Hamilton Risk Level: Low Testing: Lots more tests :) Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A

The system that Envoy Mobile uses to generate Envoy's config is fairly complicated. It involves snippets of YAML defined as C strings in [library/common/config/config.cc](https://sourcegraph.com/github.com/envoyproxy/envoy-mobile/-/blob/library/common/config/config.cc). In order to avoid duplication, these...

Support for Android socket tagging is added in PR #2423. However, the test coverage is not ideal. [Chromium's unit tests](https://source.chromium.org/chromium/chromium/src/+/main:net/socket/socket_tag_unittest.cc;l=81) for the socket tagging implementation looks at `/proc` to [check...

quic: remove duplciate code in QUIC encodeData() implementations Add a QuicSpdyStream& member to EnvoyQuicStream so that the encodeData implementation can be moved to EnvoyQuicStream and out of the subclasses. Risk...

http3: Add support for HTTP/3 METADATA Adds a new `allow_metadata` option to Http3ProtocolOptions. Risk Level: Low, protected by new config option Testing: New integration tests Docs Changes: N/A Release Notes:...

https://github.com/envoyproxy/envoy-mobile/actions/runs/3363462624/jobs/5576640354

area/testing

TestConfig.StringAccessors seems to be flaky, or at least fragile. I got a test failure in an unrelated PR recently: [2022-10-31 13:02:05.770][13641667][critical][assert] [external/envoy/test/test_listener.cc:21] assert failure: !Thread::MainThread::isMainThreadActive(). Details: MainThreadLeak: [TestConfig.StringAccessors] test exited...

area/testing