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

Jump to bottom

Review URI encoding in ConcatServlet & WelcomeFilter #6263

Closed
lachlan-roberts opened this issue May 12, 2021 · 1 comment · Fixed by #6262 or #6261
Closed

Review URI encoding in ConcatServlet & WelcomeFilter #6263

lachlan-roberts opened this issue May 12, 2021 · 1 comment · Fixed by #6262 or #6261
Labels
Sponsored This issue affects a user with a commercial support agreement
Projects
Jetty 10.0.3/11.0.3 Jetty 9.4.41

Comments

@lachlan-roberts
Copy link
Contributor

Review of the URI encoding with regards to RequestDispatcher usage.
This was linked to pull requests May 12, 2021
Jetty 10 - Review URI encoding in ConcatServlet & WelcomeFilter and improve tesing #6262
Merged
Jetty 9.4 - Review URI encoding in ConcatServlet & WelcomeFilter and improve testing #6261
Merged
This was referenced May 12, 2021
Merged
Merged
@lachlan-roberts lachlan-roberts added this to To do in Jetty 10.0.3/11.0.3 via automation May 12, 2021
@lachlan-roberts lachlan-roberts added this to To do in Jetty 9.4.41 via automation May 12, 2021
@lachlan-roberts lachlan-roberts moved this from To do to In progress in Jetty 9.4.41 May 12, 2021
@lachlan-roberts lachlan-roberts moved this from To do to In progress in Jetty 10.0.3/11.0.3 May 12, 2021
sbordet added a commit that referenced this issue May 12, 2021
@sbordet
Issue #6263 - Review URI encoding in ConcatServlet & WelcomeFilter.
5d57248 
Fixed log statement.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue May 12, 2021
@sbordet
Issue #6263 - Review URI encoding in ConcatServlet & WelcomeFilter.
0489e04 
Small fix to a comment in HttpURI parsing.

Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
sbordet added a commit that referenced this issue May 12, 2021
@lachlan-roberts @sbordet
Fixes #6263 - Review URI encoding in ConcatServlet & WelcomeFilter.
1c05b0b 
Review URI encoding in ConcatServlet & WelcomeFilter and improve testing.

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
Co-authored-by: Simone Bordet <simone.bordet@gmail.com>
Jetty 10.0.3/11.0.3 automation moved this from In progress to Done May 12, 2021
Jetty 9.4.41 automation moved this from In progress to Done May 12, 2021
sbordet added a commit that referenced this issue May 12, 2021
@lachlan-roberts @sbordet
Fixes #6263 - Review URI encoding in ConcatServlet & WelcomeFilter.
f58dbed 
Review URI encoding in ConcatServlet & WelcomeFilter and improve testing.

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Simone Bordet <simone.bordet@gmail.com>
Co-authored-by: Simone Bordet <simone.bordet@gmail.com>
@gregw
Copy link
Contributor

gregw commented Aug 31, 2021

back port to 9.3

@gregw gregw reopened this Aug 31, 2021
@gregw gregw added this to To do in Jetty 9.3.30 FROZEN via automation Aug 31, 2021
@gregw gregw added the Sponsored This issue affects a user with a commercial support agreement label Aug 31, 2021
@olamy olamy removed this from To do in Jetty 9.3.30 FROZEN Sep 9, 2021
gregw added a commit that referenced this issue Sep 30, 2021
@gregw
backported fix for #6263 from 1c05b0b, but no tests
fd7e05c 
Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw mentioned this issue Sep 30, 2021
Merged
gregw added a commit that referenced this issue Sep 30, 2021
@gregw
backported fix for #6263 from 1c05b0b, but no tests (#6942)
ebf6d3d 
Signed-off-by: Greg Wilkins <gregw@webtide.com>
gregw added a commit that referenced this issue Sep 30, 2021
@gregw
backported fix for #6263 from 1c05b0b, but no tests
ec7dc0f 
Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw mentioned this issue Sep 30, 2021
Merged
joakime added a commit that referenced this issue Sep 30, 2021
@joakime
Merge pull request #6943 from eclipse/jetty-9.3.x-6263-welcomefilter
1846b28 
backported fix for welcome filter #6263
This was referenced Mar 4, 2022
Closed
Closed
Closed
This was referenced Sep 20, 2022
Closed
Closed
Closed
@cxronen cxronen mentioned this issue Dec 20, 2022
Open
@cxronen cxronen mentioned this issue Feb 17, 2023
Open
@cxronen cxronen mentioned this issue Nov 22, 2022
Open
This was referenced Feb 26, 2023
Open
Open
Open
@cxronen cxronen mentioned this issue Mar 15, 2023
Open
This was referenced Feb 15, 2023
Open
Open
This was referenced Jul 10, 2023
Open
Open
Open
@cxronen cxronen mentioned this issue May 25, 2023
Open
This was referenced Jul 10, 2023
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Sponsored This issue affects a user with a commercial support agreement
Projects
No open projects
Jetty 10.0.3/11.0.3
Done
Jetty 9.4.41
Done
Milestone
No milestone
2 participants
@gregw @lachlan-roberts