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

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

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
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>
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 the Sponsored This issue affects a user with a commercial support agreement label Aug 31, 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
Closed
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
None yet
Milestone
No milestone
2 participants
@gregw @lachlan-roberts