Skip to content
This repository has been archived by the owner on Dec 18, 2018. It is now read-only.

HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block #2071

Closed
wants to merge 1 commit into from
Closed

HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block #2071

wants to merge 1 commit into from

Conversation

cesarblum
Copy link
Contributor

@cesarblum cesarblum commented Sep 21, 2017 •
edited
Loading

http://httpwg.org/specs/rfc7540.html#rfc.section.4.3

A receiver MUST terminate the connection with a connection error (Section 5.4.1) of type COMPRESSION_ERROR if it does not decompress a header block.

@cesarblum cesarblum force-pushed the cesarbs/http2-conn-err-bad-header-block branch from d1ade24 to 3b7804e Compare September 21, 2017 00:51
@cesarblum cesarblum force-pushed the cesarbs/http2-conn-err-bad-header-block branch from 3b7804e to 223ed47 Compare September 21, 2017 01:12
@cesarblum cesarblum changed the title Send a GOAWAY frame with COMPRESSION_ERROR on incomplete header block HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block Sep 21, 2017
@cesarblum cesarblum force-pushed the cesarbs/http2-conn-err-bad-header-block branch from 223ed47 to 278e346 Compare September 21, 2017 01:13
@cesarblum cesarblum changed the title HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block fragment Sep 21, 2017
@cesarblum cesarblum changed the title HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block fragment HTTP/2: close connection with COMPRESSION_ERROR on incomplete header block Sep 21, 2017
@cesarblum cesarblum force-pushed the cesarbs/http2-conn-err-bad-header-block branch from 278e346 to 9e49b34 Compare September 21, 2017 01:14
@cesarblum cesarblum force-pushed the cesarbs/http2-conn-err-bad-header-block branch from 9e49b34 to f45e141 Compare September 26, 2017 21:42
@cesarblum
Copy link
Contributor Author

cesarblum commented Sep 29, 2017 •
edited
Loading

Closing this in favor of #2084 (merged in 0c8d198).

@cesarblum cesarblum closed this Sep 29, 2017
@cesarblum cesarblum deleted the cesarbs/http2-conn-err-bad-header-block branch September 29, 2017 01:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@halter73 halter73 Awaiting requested review from halter73

@davidfowl davidfowl Awaiting requested review from davidfowl

@Tratcher Tratcher Awaiting requested review from Tratcher

@pakrym pakrym Awaiting requested review from pakrym

@natemcmaster natemcmaster Awaiting requested review from natemcmaster

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cesarblum @dnfclas