handle mixed-case HTTP headers correctly, per the spec #197

Merged
merged 1 commit into from Feb 23, 2012

Conversation

Projects
None yet
2 participants
Contributor

al2o3cr commented Feb 22, 2012

The exception-parsing code falls over if the WWW-Authenticate header is capitalized in a non-standard way (Www-Authenticate, for instance). The HTTP spec requires case-insensitive lookup for headers.

I ran into this issue when testing FbGraph code with WebMock, which oh-so-helpfully "normalizes" headers to only have capital letters at the beginning of each dash-delimited word.

@nov nov added a commit that referenced this pull request Feb 23, 2012

@nov nov Merge pull request #197 from al2o3cr/handle-header-capitalization
handle mixed-case HTTP headers correctly, per the spec
bb1ece0

@nov nov merged commit bb1ece0 into nov:master Feb 23, 2012

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