Skip to content

Preserve URI user-info delimiters and encode appended paths - #1391

Merged
asika32764 merged 2 commits into
masterfrom
copilot/fix-uri-bugs
Nov 8, 2026
Merged

asika32764 merged 2 commits into
masterfrom
copilot/fix-uri-bugs

Conversation

Copilot AI commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

URI rendering could omit @ when excluding the password, and appended path segments were not encoded. URI-template extraction mismatch behavior remains a documented dependency limitation.

  • User-info rendering: Keep the delimiter when rendering the username without the password:
    $uri->toString(Uri::ALL & ~Uri::PASSWORD); // https://admin@example.com/foo
  • Path appending: Encode appended path characters while preserving existing percent escapes.

Copilot AI linked an issue Nov 7, 2026 that may be closed by this pull request
Co-authored-by: asika32764 <1639206+asika32764@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix bugs in URI handling and extraction Preserve URI user-info delimiters and encode appended paths Nov 7, 2026
Copilot AI requested a review from asika32764 November 7, 2026 17:48
@asika32764
asika32764 marked this pull request as ready for review November 8, 2026 04:02
@asika32764
asika32764 merged commit b5ce686 into master Nov 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[URI] bugs

2 participants