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

Enforce content-type header parsing #87

Closed
wants to merge 3 commits into from
Closed

Enforce content-type header parsing #87

wants to merge 3 commits into from

Conversation

ebuildy
Copy link
Contributor

@ebuildy ebuildy commented May 9, 2018

Use Netty HttpUtil getMimeType method to get the content-type, instead of the raw HTTP header.

This should fix: #35

Use Netty HttpUtil getMimeType to get the real query content type.
@ebuildy
Copy link
Contributor Author

ebuildy commented May 9, 2018

maybe I should try first with header, then if no codec is found, try with getMimeType? (for good performance)

@jsvd
Copy link
Member

jsvd commented May 10, 2018

merged in 1b76537

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

Successfully merging this pull request may close these issues.

Support "Content-Type: application/json; charset=utf-8"
2 participants