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

fix(context-http2): don't guard headers received as part of a PUSH_PROMISE #67

Merged
merged 2 commits into from
Dec 5, 2019

Conversation

stefan-guggisberg
Copy link
Contributor

Headers received as part of a PUSH_PROMISE should not be subject to forbidden headers check

fix #66

… not be guarded

Headers received as part of a PUSH_PROMISE should not be subject to forbidden headers check

fix grantila#66
@coveralls
Copy link

coveralls commented Dec 5, 2019

Coverage Status

Coverage decreased (-0.1%) to 83.564% when pulling df64569 on stefan-guggisberg:relax-forbidden-header into c2dbbbd on grantila:master.

80 chars/line fix
@grantila
Copy link
Owner

grantila commented Dec 5, 2019

Thanks @stefan-guggisberg!

@grantila grantila merged commit b940584 into grantila:master Dec 5, 2019
@grantila
Copy link
Owner

grantila commented Dec 5, 2019

🎉 This PR is included in version 2.2.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@stefan-guggisberg stefan-guggisberg deleted the relax-forbidden-header branch December 6, 2019 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Cannot set forbidden header for requests" on pushed response
3 participants