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

Use UriUtils to process static resource paths #33859

Closed
rstoyanchev opened this issue Nov 8, 2024 · 0 comments
Closed

Use UriUtils to process static resource paths #33859

rstoyanchev opened this issue Nov 8, 2024 · 0 comments
Assignees
@rstoyanchev
Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Milestone
6.1.15

Comments

@rstoyanchev
Copy link
Contributor

Currently we use URLDecoder to process the path initially in order to normalize and perform checks on it. However, URLDecoder is for decoding form data rather, not paths. We should use UriUtils instead to ensure handling of raw characters, and for consistency with how we handle paths generally.
@rstoyanchev rstoyanchev added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement labels Nov 8, 2024
@rstoyanchev rstoyanchev added this to the 6.1.15 milestone Nov 8, 2024
@rstoyanchev rstoyanchev self-assigned this Nov 8, 2024
rstoyanchev added a commit that referenced this issue Nov 12, 2024
@rstoyanchev
Decode static resource path with UriUtils
cbe2f36 
See gh-33859
lijunyzzZ pushed a commit to succsoft/spring-framework that referenced this issue Dec 4, 2024
@rstoyanchev @lijunyzzZ
Decode static resource path with UriUtils
39f0773 
Closes spring-projectsgh-33859
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rstoyanchev rstoyanchev

Labels
in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement
Projects
None yet
Milestone
6.1.15
Development

No branches or pull requests
1 participant
@rstoyanchev