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 bug where valid mixed case headers fails parse #23

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@mwindle
Contributor

mwindle commented Jan 23, 2014

Server parse fails when explicitly provided headers include upper case
characters. Update lib/parser.js to look for the lower case version of
the header. Add a new test case in test/parser.test.js to check for
valid mixed case headers. Verified new test failure without change, all
tests passing with change.

Fix bug where valid mixed case headers fails parse
Server parse fails when explicitly provided headers include upper case
characters. Update lib/parser.js to look for the lower case version of
the header. Add a new test case in test/parser.test.js to check for
valid mixed case headers. Verified new test failure without change, all
tests passing with change.
@arekinath

This comment has been minimized.

Show comment
Hide comment
@arekinath

arekinath Aug 25, 2017

Contributor

Merged as aaf66f5. Thanks!

Contributor

arekinath commented Aug 25, 2017

Merged as aaf66f5. Thanks!

@arekinath arekinath closed this Aug 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment