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.

Sign up for GitHub

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

Jump to bottom

Jetty 12 - Review MovedContextHandler #8858

Closed
sbordet opened this issue Nov 2, 2022 · 1 comment · Fixed by #8859 or #9090
Closed

Jetty 12 - Review MovedContextHandler #8858

sbordet opened this issue Nov 2, 2022 · 1 comment · Fixed by #8859 or #9090
Labels
Bug For general bugs on Jetty side

Comments

@sbordet
Copy link
Contributor

sbordet commented Nov 2, 2022

Jetty version(s)
12+

Description
MovedContextHandler is not tested and it's currently broken.
Methods are still named after Servlets, but should be renamed to a more generic naming.
@sbordet sbordet added the Bug For general bugs on Jetty side label Nov 2, 2022
sbordet added a commit that referenced this issue Nov 3, 2022
@sbordet
Fixes #8858 - Jetty 12 Review MovedContextHandler.
a3d70da 
Updated method names to avod references to Servlet concepts.
Added test cases.
Updated XML files.

Signed-off-by: Simone Bordet <[email protected]>
@sbordet sbordet linked a pull request Nov 3, 2022 that will close this issue
Fixes #8858 - Jetty 12 Review MovedContextHandler. #8859
Merged
sbordet added a commit that referenced this issue Nov 3, 2022
@sbordet
Fixes #8858 - Jetty 12 Review MovedContextHandler. (#8859)
41e9842 
* Fixes #8858 - Jetty 12 Review MovedContextHandler.

Updated method names to avod references to Servlet concepts.
Added test cases.
Updated XML files.

Signed-off-by: Simone Bordet <[email protected]>
@sbordet
Copy link
Contributor Author

sbordet commented Nov 9, 2022

Fixed by #8859.

@sbordet sbordet closed this as completed Nov 9, 2022
@olamy olamy mentioned this issue Dec 26, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For general bugs on Jetty side
Projects
None yet
Milestone
No milestone
1 participant
@sbordet