Skip to content
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

build: update envoy and disable h3 (for now) #1326

Merged
merged 4 commits into from
Mar 30, 2021
Merged

build: update envoy and disable h3 (for now) #1326

merged 4 commits into from
Mar 30, 2021

Conversation

goaway
Copy link
Contributor

@goaway goaway commented Mar 29, 2021

Description: Adds new upstream flag for temporarily disabling HTTP/3.
Risk Level: Low
Testing: Local and CI

Signed-off-by: Mike Schore [email protected]

@rebello95
Copy link
Contributor

@junr03 since you have #1325

buildbreaker
buildbreaker previously approved these changes Mar 29, 2021
@goaway
Copy link
Contributor Author

goaway commented Mar 29, 2021

There's a quiche bug in the current upstream dependency (https://envoyproxy.slack.com/archives/C9UGU858E/p1617048013001500), but it's being updated.

Signed-off-by: Mike Schore <[email protected]>
buildbreaker
buildbreaker previously approved these changes Mar 29, 2021
@goaway
Copy link
Contributor Author

goaway commented Mar 29, 2021

quiche dependency fix is merged, now waiting on this PR I opened upstream with a couple more fixes: envoyproxy/envoy#15751

Signed-off-by: Mike Schore <[email protected]>
@junr03
Copy link
Member

junr03 commented Mar 30, 2021

This update also includes the fix in envoyproxy/envoy#15719

@goaway goaway merged commit 10ffa05 into main Mar 30, 2021
@goaway goaway deleted the ms/disable-http3 branch March 30, 2021 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants