Skip to content

ext/uri: Fix GH-22628 Percent-encoding of caret in WHATWG URL paths is not performed#22700

Merged
TimWolla merged 1 commit into
php:PHP-8.5from
kocsismate:gh-22628-fix-caret
Jul 12, 2026
Merged

ext/uri: Fix GH-22628 Percent-encoding of caret in WHATWG URL paths is not performed#22700
TimWolla merged 1 commit into
php:PHP-8.5from
kocsismate:gh-22628-fix-caret

Conversation

@kocsismate

Copy link
Copy Markdown
Member

The caret (^) is part of the path percent-encode set:

"The path percent-encode set is a percent-encode set consisting of the query percent-encode set and U+003F (?), U+005E (^), U+0060 (`), U+007B ({), and U+007D (})."

Until now, this character wasn't percent-encoded in the path likely due to a copy-paste error. This mistake is fixed by adding LXB_URL_MAP_PATH to the lxb_url_map entry for the caret.
9682ea0

@kocsismate kocsismate requested a review from TimWolla as a code owner July 11, 2026 19:49
@kocsismate kocsismate requested review from iluuu1994 and ndossche and removed request for TimWolla July 11, 2026 19:49
…s is not performed

The caret (^) is part of the path percent-encode set:

"The path percent-encode set is a percent-encode set consisting of the query percent-encode set and U+003F (?), U+005E (^), U+0060 (`), U+007B ({), and U+007D (})."

Until now, this character wasn't percent-encoded in the path likely due to a copy-paste error. This mistake is fixed by adding LXB_URL_MAP_PATH to the lxb_url_map entry for the caret.
@TimWolla TimWolla force-pushed the gh-22628-fix-caret branch from 9682ea0 to 60fe63c Compare July 12, 2026 17:58
@TimWolla TimWolla merged commit c025bc4 into php:PHP-8.5 Jul 12, 2026
18 checks passed
TimWolla added a commit that referenced this pull request Jul 12, 2026
* PHP-8.5:
  ext/uri: Fix GH-22628 Percent-encoding of caret in WHATWG URL paths is not performed (#22700)
@kocsismate kocsismate deleted the gh-22628-fix-caret branch July 12, 2026 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants