-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Quiche roll 20230206164719 #25394
Quiche roll 20230206164719 #25394
Conversation
https://github.com/google/quiche/compare/bebdfcd7e..b901e844b ``` $ git log bebdfcd7e..b901e844b --date=short --no-merges --format="%ad %al %s" 2023-02-03 martinduke Internal change 2023-02-03 martinduke Remove redundant code to report ECN marks. 2023-02-02 quiche-dev Fix Windows build error due to implicit cast of uint64_t to WebTransportStreamError which is uint8_t. 2023-02-02 vasilvv Fix QUICHE bazel build by adding a missing dependency 2023-02-02 danzh Internal change 2023-02-01 vasilvv Simplify capsule.h to reduce the amount of code per individual capsule. ``` Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, assuming it passes CI.
I presume you need to change the Decrypters too. |
interestingly (to me) Null Decrypters weren't used (according to both grep and CI) :-P |
Risk Level: low
Testing: updated tests
Docs Changes: n/a
Release Notes: n/a