Skip to content

WebServer should match headers using equalsIgnoreCase #2131

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

Closed
enzosagretti opened this issue Jun 12, 2016 · 1 comment · Fixed by #2474
Closed

WebServer should match headers using equalsIgnoreCase #2131

enzosagretti opened this issue Jun 12, 2016 · 1 comment · Fixed by #2474

Comments

@enzosagretti
Copy link

enzosagretti commented Jun 12, 2016

I found an issue in "Parsing.cpp". (line 164)
else if (headerName == "Content-Length"){ contentLength = headerValue.toInt();

The compare between "headerName" and "Content-Length" must not be case sensitive. For this problem i can´t read HTTP_POST payload. I change this in my enviroment and work fine!!
else if (headerName == "content-length"){ contentLength = headerValue.toInt();

Could you change all HTTP header compare to not be case sensitive?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@igrr igrr added this to the 2.3.0 milestone Jun 12, 2016
@igrr igrr changed the title ISSSUE in parsing.cpp (line 164) WebServer should match headers using equalsIgnoreCase Jun 12, 2016
@igrr igrr modified the milestones: 2.4.0, 2.3.0 Jun 12, 2016
me-no-dev pushed a commit to me-no-dev/Arduino that referenced this issue Sep 2, 2016
me-no-dev added a commit that referenced this issue Sep 2, 2016
@me-no-dev
Copy link
Collaborator

Should be fixed in the above commit

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

Successfully merging a pull request may close this issue.

3 participants