frankenphp
fix: set $_SERVER variables: 'SCRIPT_NAME', 'PHP_SELF', and 'PATH_INFO'
#2317
Open

fix: set $_SERVER variables: 'SCRIPT_NAME', 'PHP_SELF', and 'PATH_INFO' #2317

henderkes wants to merge 10 commits into main from fix/script_name
henderkes
henderkes fix $_SERVER['SCRIPT_NAME'] and $_SERVER['PHP_SELF'] in php_server wo…
708b2336
henderkes henderkes requested a review from AlliBalliBaba AlliBalliBaba 9 days ago
henderkes henderkes marked this pull request as ready for review 9 days ago
henderkes -failfast for tests, so we can actually see what failed
1935dc9f
henderkes henderkes marked this pull request as draft 9 days ago
henderkes fix pathInfo being empty
1eb691d8
henderkes henderkes changed the title fix $_SERVER['SCRIPT_NAME'] and $_SERVER['PHP_SELF'] in php_server workers fix $_SERVER['SCRIPT_NAME'], 'PHP_SELF', and 'PATH_INFO' 9 days ago
henderkes henderkes marked this pull request as ready for review 9 days ago
AlliBalliBaba
AlliBalliBaba commented on 2026-03-28
AlliBalliBaba
AlliBalliBaba commented on 2026-03-28
henderkes add tests (worker and non-worker separately because the global worker…
3708b41b
henderkes fix on windows
2a32861c
henderkes
henderkes commented on 2026-03-28
henderkes use existing ensureLeadingSlash function to
652322b0
henderkes henderkes requested a review from AlliBalliBaba AlliBalliBaba 8 days ago
henderkes henderkes requested a review from dunglas dunglas 8 days ago
AlliBalliBaba
AlliBalliBaba commented on 2026-03-29
henderkes test for match * worker
fd6595e3
henderkes @AlliBalliBaba suggestion, empty out script_name if it's not in serve…
b8a981d9
henderkes henderkes changed the title fix $_SERVER['SCRIPT_NAME'], 'PHP_SELF', and 'PATH_INFO' fix: set $_SERVER variables: 'SCRIPT_NAME', 'PHP_SELF', and 'PATH_INFO' 6 days ago
AlliBalliBaba
AlliBalliBaba commented on 2026-04-04
henderkes
henderkes commented on 2026-04-05
henderkes
henderkes commented on 2026-04-05
henderkes Apply suggestions from code review
91a69681
henderkes Merge branch 'main' into fix/script_name
0a294aca

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone