Skip to content
This repository was archived by the owner on Jul 6, 2018. It is now read-only.

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented May 16, 2017

Connection-specific headers are forbidden by HTTP/2. Make sure we're filtering them out.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

http2

mcollina

This comment was marked as off-topic.

@jasnell jasnell closed this May 17, 2017
@jasnell jasnell force-pushed the more-connection-headers branch from b1c4561 to bd57c68 Compare May 17, 2017 14:55
jasnell added a commit that referenced this pull request May 17, 2017
These are forbidden by HTTP/2.

PR-URL: #128
Reviewed-By: Matteo Collina <[email protected]>
@jasnell
Copy link
Member Author

jasnell commented May 17, 2017

Landed!

jasnell added a commit that referenced this pull request May 19, 2017
These are forbidden by HTTP/2.

PR-URL: #128
Reviewed-By: Matteo Collina <[email protected]>
jasnell added a commit that referenced this pull request May 31, 2017
These are forbidden by HTTP/2.

PR-URL: #128
Reviewed-By: Matteo Collina <[email protected]>
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jun 22, 2017
These are forbidden by HTTP/2.

PR-URL: nodejs#128
Reviewed-By: Matteo Collina <[email protected]>
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jul 10, 2017
These are forbidden by HTTP/2.

PR-URL: nodejs#128
Reviewed-By: Matteo Collina <[email protected]>
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jul 14, 2017
These are forbidden by HTTP/2.

PR-URL: nodejs#128
Reviewed-By: Matteo Collina <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants