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

Fix #6870 Encode control characters (#6874) #6895

Merged
merged 1 commit into from Sep 23, 2021

Conversation

gregw
Copy link
Contributor

@gregw gregw commented Sep 23, 2021

Fix #6870 URIUtil.encodePath encodes control characters
cherry pick from 9

  • Better test for wider range of characters
  • Encode all control characters

Signed-off-by: Greg Wilkins gregw@webtide.com

Fix #6870 URIUtil.encodePath encodes control characters

* Better test for wider range of characters
* Encode all control characters

Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw added this to In progress in Jetty 10.0.7/11.0.7 FROZEN via automation Sep 23, 2021
@gregw gregw requested review from joakime and lachlan-roberts and removed request for joakime September 23, 2021 00:21
@joakime joakime moved this from In progress to Review in progress in Jetty 10.0.7/11.0.7 FROZEN Sep 23, 2021
Jetty 10.0.7/11.0.7 FROZEN automation moved this from Review in progress to Reviewer approved Sep 23, 2021
@gregw gregw merged commit d9d8919 into jetty-10.0.x Sep 23, 2021
Jetty 10.0.7/11.0.7 FROZEN automation moved this from Reviewer approved to Done Sep 23, 2021
@gregw gregw deleted the jetty-10.0.x-6870-encode-control branch September 23, 2021 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Encode control characters in URIUtil.encodePath
2 participants