expvar: avoid conflict with user-defined "GET /" route.

With the new routing style in go 1.22, declaring

    http.Handle("GET /", h)

generates a conflict with route "/debug/vars" declared in the expvar
package. You get an error such as:

panic: pattern "GET /" (registered at ...) conflicts with pattern
"/debug/vars" (registered at ...expvar.go:384): GET / matches fewer
methods than /debug/vars, but has a more general path pattern

This patch prevents that error.  Adding GET is correct because no other
method makes sense with /debug/vars.

We preserve the traditional behaviour when GODEBUG=httpmuxgo121=1 is
specified.

Fixes #65723

Change-Id: Id2b963ebad41a1ebdcceb73baf3436d59aac73a0
GitHub-Last-Rev: 9c2b9f7
GitHub-Pull-Request: #65745
Reviewed-on: https://go-review.googlesource.com/c/go/+/564735
Reviewed-by: Damien Neil <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Jonathan Amsterdam <[email protected]>

Author: xpmatteo

src/expvar/expvar.go

@@ -4,7 +4,8 @@
 
 // Package expvar provides a standardized interface to public variables, such
 // as operation counters in servers. It exposes these variables via HTTP at
-// /debug/vars in JSON format.
+// /debug/vars in JSON format. As of Go 1.22, the /debug/vars request must
+// use GET.
 //
 // Operations to set or modify these public variables are atomic.
 //
@@ -23,6 +24,7 @@ package expvar
 
 import (
 	"encoding/json"
+	"internal/godebug"
 	"log"
 	"math"
 	"net/http"
@@ -381,7 +383,11 @@ func memstats() any {
 }
 
 func init() {
-	http.HandleFunc("/debug/vars", expvarHandler)
+	if godebug.New("httpmuxgo121").Value() == "1" {
+		http.HandleFunc("/debug/vars", expvarHandler)
+	} else {
+		http.HandleFunc("GET /debug/vars", expvarHandler)
+	}
 	Publish("cmdline", Func(cmdline))
 	Publish("memstats", Func(memstats))
 }