envoy icon indicating copy to clipboard operation
envoy copied to clipboard

Published 20 hours ago • verified publisher icon envoyproxy

http: move HTTP Datagram support check to constructor.

Open jeongseokson opened this issue 1 year ago • 1 comments

Commit Message: Moved the runtime flag check for HTTP Datagram support to QUIC session constructors. This will eliminate the repeated runtime flag check, saving some CPU cycles. Additional Description: Risk Level: Low, the change is protected with the runtime flag. Testing: Existing unit tests and integration tests cover this change. Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A

jeongseokson avatar May 30 '24 23:05 jeongseokson

/assign @danzh2010

Hi Dan, can you review this PR? Thank you!

jeongseokson avatar May 30 '24 23:05 jeongseokson

Please merge main & fix CI. Thanks!

/wait

jmarantz avatar May 31 '24 20:05 jmarantz

/retest

jeongseokson avatar Jun 03 '24 20:06 jeongseokson

Assigning Greg as !Google and senior-maintainer reviewer. /assign @ggreenway

adisuissa avatar Jun 04 '24 13:06 adisuissa