fix $_SERVER['SCRIPT_NAME'] and $_SERVER['PHP_SELF'] in php_server workers

apache passes PHP_SELF as SCRIPT_NAME + REQUEST_URL, but the docs say it's the same as SCRIPT_NAME and that's how Caddy+fpm behave too

Author: henderkes

.gitignore

@@ -1,6 +1,8 @@
 /caddy/frankenphp/Build
+/caddy/frankenphp/Caddyfile.test
 /caddy/frankenphp/frankenphp
 /caddy/frankenphp/frankenphp.exe
+/caddy/frankenphp/public
 /dist
 /github_conf
 /internal/testserver/testserver

cgi.go

@@ -111,8 +111,6 @@ func addKnownVariablesToServer(fc *frankenPHPContext, trackVarsArray *C.zval) {
 		requestURI = fc.requestURI
 	}
 
-	requestPath := ensureLeadingSlash(request.URL.Path)
-
 	C.frankenphp_register_server_vars(trackVarsArray, C.frankenphp_server_vars{
 		// approximate total length to avoid array re-hashing:
 		// 28 CGI vars + headers + environment
@@ -129,8 +127,8 @@ func addKnownVariablesToServer(fc *frankenPHPContext, trackVarsArray *C.zval) {
 		document_root_len:   C.size_t(len(fc.documentRoot)),
 		path_info:           toUnsafeChar(fc.pathInfo),
 		path_info_len:       C.size_t(len(fc.pathInfo)),
-		php_self:            toUnsafeChar(requestPath),
-		php_self_len:        C.size_t(len(requestPath)),
+		php_self:            toUnsafeChar(fc.scriptName),
+		php_self_len:        C.size_t(len(fc.scriptName)),
 		document_uri:        toUnsafeChar(fc.docURI),
 		document_uri_len:    C.size_t(len(fc.docURI)),
 		script_filename:     toUnsafeChar(fc.scriptFilename),

context.go

@@ -89,6 +89,7 @@ func NewRequestWithContext(r *http.Request, opts ...RequestOption) (*http.Reques
 	// If a worker is already assigned explicitly, use its filename and skip parsing path variables
 	if fc.worker != nil {
 		fc.scriptFilename = fc.worker.fileName
+		fc.scriptName = strings.TrimPrefix(fc.worker.fileName, fc.documentRoot)
 	} else {
 		// If no worker was assigned, split the path into the "traditional" CGI path variables.
 		// This needs to already happen here in case a worker script still matches the path.