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 18 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 12 days ago
henderkes henderkes marked this pull request as ready for review 12 days ago
henderkes -failfast for tests, so we can actually see what failed
1935dc9f
henderkes henderkes marked this pull request as draft 12 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' 12 days ago
henderkes henderkes marked this pull request as ready for review 12 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 11 days ago
henderkes henderkes requested a review from dunglas dunglas 11 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' 9 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
dunglas
dunglas approved these changes on 2026-03-27
dunglas dunglas requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 17 hours ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-04-08
henderkes safety check for separator, in case someone uses matchers
b632a5ee
henderkes Update caddy/caddy_test.go
c5481d00
henderkes Update caddy/caddy_test.go
6fc5c153
alexandre-daubois
alexandre-daubois commented on 2026-04-08
alexandre-daubois
alexandre-daubois approved these changes on 2026-04-08
henderkes use gotestsum for tests (shows all failed test outputs only)
7375dcee
henderkes use gotestsum for other tests too (except fuzz test)
2dc72701
henderkes Merge branch 'main' into fix/script_name
44519f51
henderkes fix "Set Set CGO flags"
5f9c77f9
henderkes reset CI test changes, refactor out to next PR
786b6be4
AlliBalliBaba
AlliBalliBaba commented on 2026-04-08

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone