@AlliBalliBaba suggestion, empty out script_name if it's not in server root

Author: henderkes

caddy/caddy_test.go

@@ -639,9 +639,9 @@ func TestWorkerPHPServerGlobals(t *testing.T) {
 	tester.AssertGetResponse(
 		"http://localhost:"+testPortThree+"/en",
 		http.StatusOK,
-		"SCRIPT_NAME: "+documentRoot+"/server-globals.php<br>"+
+		"SCRIPT_NAME: <br>"+
 			"SCRIPT_FILENAME: "+scriptFilename+"<br>"+
-			"PHP_SELF: "+documentRoot+"/server-globals.php<br>"+
+			"PHP_SELF: <br>"+
 			"PATH_INFO: <br>"+
 			"DOCUMENT_ROOT: "+documentRoot2+"<br>"+
 			"REQUEST_URI: /en<br>",
@@ -650,9 +650,9 @@ func TestWorkerPHPServerGlobals(t *testing.T) {
 	tester.AssertGetResponse(
 		"http://localhost:"+testPortThree+"/server-globals.php/en",
 		http.StatusOK,
-		"SCRIPT_NAME: "+documentRoot+"/server-globals.php<br>"+
+		"SCRIPT_NAME: <br>"+
 			"SCRIPT_FILENAME: "+scriptFilename+"<br>"+
-			"PHP_SELF: "+documentRoot+"/server-globals.php/en<br>"+
+			"PHP_SELF: /en<br>"+
 			"PATH_INFO: /en<br>"+
 			"DOCUMENT_ROOT: "+documentRoot2+"<br>"+
 			"REQUEST_URI: /server-globals.php/en<br>",

cgi.go

@@ -213,7 +213,9 @@ func splitCgiPath(fc *frankenPHPContext) {
 	// If a worker is already assigned explicitly, derive SCRIPT_NAME from its filename
 	if fc.worker != nil {
 		fc.scriptFilename = fc.worker.fileName
-		fc.scriptName = filepath.ToSlash(strings.TrimPrefix(fc.worker.fileName, fc.documentRoot))
+		if strings.HasPrefix(fc.worker.fileName, fc.documentRoot) {
+			fc.scriptName = filepath.ToSlash(strings.TrimPrefix(fc.worker.fileName, fc.documentRoot))
+		}
 		return
 	}