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

Issue #7748 - Merge PathSpec improvements made in 9.4.x #7845

Merged
merged 2 commits into from Apr 6, 2022

Conversation

gregw
Copy link
Contributor

@gregw gregw commented Apr 5, 2022

cherry pick forward of pathInfo fixes in 9 for #7748

+ Backport of #7748
+ Fix RegexPathSpec pathInfo
+ Fix UriTemplatePathSpec pathInfo
+ Test regression option to 93 behaviour

Signed-off-by: Greg Wilkins <gregw@webtide.com>
Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw requested a review from joakime April 5, 2022 17:16
@joakime joakime added this to In progress in Jetty 10.0.10/11.0.10 - 🧊 FROZEN 🥶 via automation Apr 5, 2022
@joakime joakime changed the title Jetty 10.0.x 7748 updates from 9 Issue #7748 - Merge PathSpec improvements made in 9.4.x Apr 5, 2022
@joakime joakime moved this from In progress to Review in progress in Jetty 10.0.10/11.0.10 - 🧊 FROZEN 🥶 Apr 5, 2022
@gregw
Copy link
Contributor Author

gregw commented Apr 6, 2022

@joakime review please

Jetty 10.0.10/11.0.10 - 🧊 FROZEN 🥶 automation moved this from Review in progress to Reviewer approved Apr 6, 2022
@gregw gregw merged commit 5fc63bf into jetty-10.0.x Apr 6, 2022
Jetty 10.0.10/11.0.10 - 🧊 FROZEN 🥶 automation moved this from Reviewer approved to Done Apr 6, 2022
@gregw gregw deleted the jetty-10.0.x-7748-updates-from-9 branch April 6, 2022 09:14
gregw added a commit that referenced this pull request Apr 6, 2022
* Fixes to backport of #7748

+ Backport of #7748
+ Fix RegexPathSpec pathInfo
+ Fix UriTemplatePathSpec pathInfo
+ Test regression option to 93 behaviour

* small optimization

Signed-off-by: Greg Wilkins <gregw@webtide.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants