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

Backport to v2.2.x: [http2] delay processing requests upon observing suspicious behavior #3293

Merged
merged 1 commit into from Oct 15, 2023

Conversation

rgacogne
Copy link
Contributor

@rgacogne rgacogne commented Oct 11, 2023

Hi!

This is a backport of 94fbc54 from #3291 to the v2.2.x branch.

I know that all the tagged releases should be considered deprecated, and that no new version will be released in the future, therefore I don't expect this pull request to be merged. While we are transitioning from libh2o-evloop to nghttp2 in DNSdist, we unfortunately need to keep our h2o support alive for a little bit longer while the transition is complete, so we have been working on backporting the HTTP2 rapid reset fix on top of 2.2.6, and we decided to share the result in case it helps others.
Please note that we unfortunately had to break the ABI to introduce the new timer, so this change will require a rebuild of applications dynamically linked against libh2o.
Any feedback would be of course appreciated!

Best regards,

Remi Gacogne
PowerDNS.COM BV - https://www.powerdns.com/

@dch
Copy link
Collaborator

dch commented Oct 11, 2023

thanks @rgacogne we'll be backporting this to FreeBSD as well, to support dndist, urbit, and freeswitch, which rely on libh2o from 2.2.6.

@kazuho
Copy link
Member

kazuho commented Oct 15, 2023

Thank you for taking care of downstream projects.

With the caveat that we longer maintain 2.2.x, I do not think there is a reason to not merge this PR.

@kazuho kazuho merged commit 094a6f9 into h2o:v2.2.x Oct 15, 2023
@rgacogne rgacogne deleted the 2.2.x-h2-rapid-reset branch October 16, 2023 13:20
freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this pull request Oct 28, 2023
- downstream dnsdist project has backported a fix for this specific issue
- deprecation is still planned, and port should not be considered secure
- pet port in line with www/h2o-devel

See h2o/h2o#3293 for further details

Obtained from:	Remi Gacogne <remi.gacogne@powerdns.com>
Security:	CVE-2023-44487
Security:	GHSA-2m7v-gc89-fjqf
freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this pull request Oct 28, 2023
- downstream dnsdist project has backported a fix for this specific issue
- deprecation is still planned, and port should not be considered secure
- pet port in line with www/h2o-devel

See h2o/h2o#3293 for further details

Obtained from:	Remi Gacogne <remi.gacogne@powerdns.com>
Security:	CVE-2023-44487
Security:	GHSA-2m7v-gc89-fjqf

(cherry picked from commit dcd7c23)
lsalvadore pushed a commit to lsalvadore/freebsd-ports that referenced this pull request Oct 29, 2023
- downstream dnsdist project has backported a fix for this specific issue
- deprecation is still planned, and port should not be considered secure
- pet port in line with www/h2o-devel

See h2o/h2o#3293 for further details

Obtained from:	Remi Gacogne <remi.gacogne@powerdns.com>
Security:	CVE-2023-44487
Security:	GHSA-2m7v-gc89-fjqf
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.

None yet

3 participants