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

Support semicolons in query parameters as explain in the W3C recommen… #9701

Merged
merged 1 commit into from Oct 24, 2019

Conversation

@normanmaurer
Copy link
Member

normanmaurer commented Oct 22, 2019

…tation

Motivation:

Support semicolons in query parameters as explain in the W3C recommentation:
https://www.w3.org/TR/2014/REC-html5-20141028/forms.html#url-encoded-form-data

Modification:

  • Add a new constructor arg that can be used to "switch" modes for decoding ;
  • Add unit test

Result:

Fixes #8855

…tation

Motivation:

Support semicolons in query parameters as explain in the W3C recommentation:
https://www.w3.org/TR/2014/REC-html5-20141028/forms.html#url-encoded-form-data

Modification:

- Add a new constructor arg that can be used to "switch" modes for decoding ;
- Add unit test

Result:

Fixes #8855
@normanmaurer normanmaurer requested review from slandelle and ejona86 Oct 22, 2019
@normanmaurer normanmaurer added this to the 4.1.43.Final milestone Oct 22, 2019
Copy link
Member

carl-mastrangelo left a comment

LGTM

@normanmaurer normanmaurer merged commit 7d6d953 into 4.1 Oct 24, 2019
3 checks passed
3 checks passed
pull request validation (centos6-java11) Build finished.
Details
pull request validation (centos6-java12) Build finished.
Details
pull request validation (centos6-java8) Build finished.
Details
@normanmaurer normanmaurer deleted the query_decoder branch Oct 24, 2019
normanmaurer added a commit that referenced this pull request Oct 24, 2019
…tation (#9701)

Motivation:

Support semicolons in query parameters as explain in the W3C recommentation:
https://www.w3.org/TR/2014/REC-html5-20141028/forms.html#url-encoded-form-data

Modification:

- Add a new constructor arg that can be used to "switch" modes for decoding ;
- Add unit test

Result:

Fixes #8855
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.