From ff00fdfdbeeb72ca8981a04290765452d6a31b26 Mon Sep 17 00:00:00 2001 From: Cuong Manh Le Date: Sat, 26 Aug 2023 08:08:30 +0700 Subject: [PATCH] Revert "net/http/cgi: workaround for closure inlining issue" This reverts CL 522935. Issue #62277 is fixed, the workaround can be dropped. Updates #62277 Change-Id: I7c69e35248942b4d4fcdd81121051cca9b098980 Reviewed-on: https://go-review.googlesource.com/c/go/+/523175 Run-TryBot: Cuong Manh Le TryBot-Result: Gopher Robot Auto-Submit: Cuong Manh Le Reviewed-by: Matthew Dempsky Reviewed-by: Dmitri Shuralyov --- src/net/http/cgi/host.go | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/net/http/cgi/host.go b/src/net/http/cgi/host.go index a3fba4b413..073952a7bd 100644 --- a/src/net/http/cgi/host.go +++ b/src/net/http/cgi/host.go @@ -35,10 +35,7 @@ import ( var trailingPort = regexp.MustCompile(`:([0-9]+)$`) -var osDefaultInheritEnv = getOSDefaultInheritEnv() - -// TODO(mdempsky): Revert CL 522935 after #62277 is fixed. -func getOSDefaultInheritEnv() []string { +var osDefaultInheritEnv = func() []string { switch runtime.GOOS { case "darwin", "ios": return []string{"DYLD_LIBRARY_PATH"} @@ -54,7 +51,7 @@ func getOSDefaultInheritEnv() []string { return []string{"SystemRoot", "COMSPEC", "PATHEXT", "WINDIR"} } return nil -} +}() // Handler runs an executable in a subprocess with a CGI environment. type Handler struct { -- 2.50.0