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.

Sign up for GitHub

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

Jump to bottom

Header names should not be case sensitive #1230

Open
deki opened this issue Feb 4, 2025 · 0 comments
Open

Header names should not be case sensitive #1230

deki opened this issue Feb 4, 2025 · 0 comments

Comments

@deki
Copy link
Contributor

deki commented Feb 4, 2025 •
edited
Loading

ServerlessHttpServletRequest currently stores and retrieves headers in a case sensitive way. Header names should not not be treated case sensitive though.
From RFC 2616 - "Hypertext Transfer Protocol -- HTTP/1.1", Section 4.2, "Message Headers":

Each header field consists of a name followed by a colon (":") and the field value. Field names are case-insensitive.

The updating RFC 7230 does not list any changes from RFC 2616 at this part.

Examples: request.getHeader("User-Agent") works, request.getHeader("user-agent") returns no result and vice versa.

Tomcat converts all headers to lower case directly but in my view making lookup case insensitive would already fix it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@deki