Skip to content

fix(platform-server): normalize path parsing in ServerPlatformLocation#68928

Open
alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:pathname-normalize
Open

fix(platform-server): normalize path parsing in ServerPlatformLocation#68928
alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:pathname-normalize

Conversation

@alan-agius4
Copy link
Copy Markdown
Contributor

@alan-agius4 alan-agius4 commented May 26, 2026

Normalizes request path parsing in ServerPlatformLocation by collapsing multiple consecutive leading slashes and backslashes (e.g. // or /) down to a single forward slash. This ensures consistent resolution of relative path segments and prevents unexpected absolute path overrides during URL parsing.

@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels May 26, 2026
@pullapprove pullapprove Bot requested a review from kirjs May 26, 2026 09:50
@angular-robot angular-robot Bot added the area: server Issues related to server-side rendering label May 26, 2026
Normalizes request path parsing in ServerPlatformLocation by collapsing multiple consecutive leading slashes and backslashes (e.g. // or /\) down to a single forward slash. This ensures consistent resolution of relative path segments and prevents unexpected absolute path overrides during URL parsing.
@ngbot ngbot Bot added this to the Backlog milestone May 26, 2026
@alan-agius4 alan-agius4 force-pushed the pathname-normalize branch from 04e796a to b3096ed Compare May 26, 2026 09:50
@alan-agius4 alan-agius4 requested review from AndrewKushnir and removed request for kirjs May 26, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer area: server Issues related to server-side rendering target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant