-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Configurable URL decoding of HTTP headers #1789
Comments
@artpaul thank you for the report. This decoding was introduced at the following issue and commit. #576 I am ok with "global or server-level option to disable the url decoding". If we take this option, are there any possible side effects that you can think of? Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @yhirose
While parsing HTTP headers the library unconditionally applies an url decoding to all headers except for the Location header.
https://github.com/yhirose/cpp-httplib/blob/master/httplib.h#L3809
In my specific use case there is a custom header that contains url-like data (server side). The decoding makes hardly possible the further processing of the data.
What do you thing about adding some flexibility to the function? Suggestions:
The text was updated successfully, but these errors were encountered: