Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #6001 separate compliance modes for ambiguous URI segments, params and separators #6005

Merged
merged 2 commits into from
Feb 24, 2021

Conversation

gregw
Copy link
Contributor

@gregw gregw commented Feb 24, 2021

#6001 #4275 default modes allows both ambiguous separators and segments, but still forbids ambiguous parameters

…parators

default modes allows both ambiguous separators and segments, but still forbids ambiguous parameters
@gregw gregw requested a review from joakime February 24, 2021 18:19
Copy link
Contributor

@joakime joakime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with this.
Just going to let CI have a go at it first. (before I merge and restage)

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@gregw gregw requested review from sbordet and joakime February 24, 2021 19:11
@gregw
Copy link
Contributor Author

gregw commented Feb 24, 2021

@sbordet thanks for catching those!

@joakime joakime merged commit 0603b13 into jetty-9.4.x Feb 24, 2021
@joakime joakime deleted the jetty-9.4.x-6001-default-accept-ambiguous-uris branch February 24, 2021 20:12
This was referenced Mar 10, 2021
This was referenced Mar 10, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants