Skip to content

[pull] master from php:master#776

Merged
pull[bot] merged 3 commits into
ConnectionMaster:masterfrom
php:master
May 29, 2026
Merged

[pull] master from php:master#776
pull[bot] merged 3 commits into
ConnectionMaster:masterfrom
php:master

Conversation

@pull

@pull pull Bot commented May 29, 2026 •
edited
Loading

Copy link
Copy Markdown

See Commits and Changes for more details.

Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

devnexen added 3 commits May 29, 2026 19:28
@devnexen
ext/standard: http(s) wrapper corrupts the basic auth header on perce…
263743b 
…nt-encoded userinfo.

php_url_decode() returns the shorter decoded length but ZSTR_LEN() is left
untouched, so smart_str_append() carries the stale [decoded][NUL][undecoded
tail] bytes into the base64 credentials.

Fix #22171

close GH-22172
@devnexen
ext/standard: http(s) wrapper corrupts the basic auth header on perce…
f357164 
…nt-encoded userinfo.

php_url_decode() returns the shorter decoded length but ZSTR_LEN() is left
untouched, so smart_str_append() carries the stale [decoded][NUL][undecoded
tail] bytes into the base64 credentials.

Fix #22171

close GH-22172
@devnexen
Merge branch 'PHP-8.5'
ce4adcc 
* PHP-8.5:
  ext/standard: http(s) wrapper corrupts the basic auth header on percent-encoded userinfo.
@pull pull Bot locked and limited conversation to collaborators May 29, 2026
@pull pull Bot added the ⤵️ pull label May 29, 2026
@pull pull Bot merged commit ce4adcc into ConnectionMaster:master May 29, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

⤵️ pull

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@devnexen