refactor(platform-server): clean up and simplify parseUrl utility#68983
Open
alan-agius4 wants to merge 1 commit into
Open
refactor(platform-server): clean up and simplify parseUrl utility#68983alan-agius4 wants to merge 1 commit into
alan-agius4 wants to merge 1 commit into
Conversation
cbc32af to
2a31fb4
Compare
521b25e to
2768415
Compare
2768415 to
8be0b85
Compare
Trims leading/trailing whitespaces in resolveUrl to normalize input. Leverages the native new URL() constructor with a non-standard base (resolve://) under strict mode to detect and reject malformed absolute targets (like double ports) during bootstrap, rather than raw regular expressions. Preserves relative sub-paths safely by only collapsing consecutive leading slashes/backslashes when a slash is present, ensuring trailing-slash contexts are resolved properly against parent domains. Adds and updates comprehensive unit tests covering all relative pathing, malformed structures, and potential hijacking edge cases.
8be0b85 to
1f3725e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Trims leading/trailing whitespaces in parseUrl to normalize input.
Leverages the native new URL() constructor with a non-standard base (resolve://) under strict mode to detect and reject malformed absolute targets (like double ports) during bootstrap, rather than raw regular expressions.
Preserves relative sub-paths safely by only collapsing consecutive leading slashes/backslashes when a slash is present, ensuring trailing-slash contexts are resolved properly against parent domains.
Adds and updates comprehensive unit tests covering all relative pathing, malformed structures, and potential hijacking edge cases.