Skip to content

fix: send PROTOCOL_ERROR instead of REFUSED_STREAM for oversized headers#792

Merged
seanmonstar merged 1 commit intomasterfrom
oversize-headers-protocol-error
Aug 6, 2024
Merged

fix: send PROTOCOL_ERROR instead of REFUSED_STREAM for oversized headers#792
seanmonstar merged 1 commit intomasterfrom
oversize-headers-protocol-error

Conversation

@seanmonstar
Copy link
Member

Noticed during #791

cc @ajwerner

@seanmonstar seanmonstar force-pushed the oversize-headers-protocol-error branch from c011523 to ef1dc65 Compare August 5, 2024 14:31
@seanmonstar seanmonstar force-pushed the oversize-headers-protocol-error branch from ef1dc65 to dd4c53d Compare August 5, 2024 15:18
Copy link
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

LGTM

@seanmonstar seanmonstar merged commit e1693a7 into master Aug 6, 2024
@seanmonstar seanmonstar deleted the oversize-headers-protocol-error branch August 6, 2024 18:34
Cre3per pushed a commit to Cre3per/deno that referenced this pull request Jan 2, 2025
to get hyperium/h2#792

which fixes promises not being resolved when the server sends large
headers
denoland#26490
Cre3per added a commit to Cre3per/deno that referenced this pull request Jan 3, 2025
to get hyperium/h2#792

which fixes promises not being resolved when the server sends large
headers
denoland#26490
bartlomieju pushed a commit to denoland/deno that referenced this pull request Jan 26, 2025
bartlomieju pushed a commit to denoland/deno that referenced this pull request Jan 30, 2025
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.

2 participants