TLS: expose connection reset errors in SslSocket#44297
TLS: expose connection reset errors in SslSocket#44297TAOXUY wants to merge 3 commits intoenvoyproxy:mainfrom
Conversation
…test coverage Signed-off-by: Xuyang Tao <taoxuy@google.com>
Signed-off-by: Xuyang Tao <taoxuy@google.com>
Signed-off-by: Xuyang Tao <taoxuy@google.com>
|
/retest |
|
@TAOXUY your approach seems like it's solving an issue I had: Your Your PR missing the following behavior from what I can see: I would love to combine forces and use your PR if that's ok with you for the scenario I am missing - WDYT? |
Of course, please go ahead! Remember also copy the integration test on *_DETECTED_CLOSE_TYPE as well. |
Commit Message: currently only plain text connection will expose the reset type in DOWNSTREAM|UPSTREAM_DETECTED_CLOSE_TYPE but tls connection won't. This PR exposes reset errors to sslSocket caller(connectionImpl) so that RESET can be exposed to DETECED_CLOSE_TYPE in stream_info metadata.
Risk Level: lo
Testing: added
Docs Changes: no needed
Release Notes: updated
Platform Specific Features: no