Skip to content

go.mod: Bump up quic-go to v0.28.0, fixes for BC breaks #4867

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

Merged
merged 1 commit into from
Jul 6, 2022
Merged

Conversation

francislavoie
Copy link
Member

@francislavoie francislavoie commented Jul 6, 2022

Also closes #4819

The logger was never used by quic-go, and we should have been passing through MaxHeaderBytes (oversight that we weren't).

@francislavoie francislavoie added the upstream ⬆️ Relates to some dependency of this project label Jul 6, 2022
@francislavoie francislavoie added this to the v2.5.2 milestone Jul 6, 2022
@francislavoie francislavoie requested a review from mholt July 6, 2022 04:31
Copy link
Member

@mholt mholt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch -- I was waiting for the v0.28 tag, so this LGTM! Really appreciate the thorough fix.

@mholt mholt merged commit 8bac134 into master Jul 6, 2022
@mholt mholt deleted the bump-quic-go branch July 6, 2022 18:14
@caddyserver caddyserver deleted a comment Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream ⬆️ Relates to some dependency of this project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

caddy 2.5.1 + experimental_http3 + fastcgi reverse_proxy always returns 500
2 participants