Don't make the upgrader global.

x/websocket/endpoint.go

@@ -32,10 +32,6 @@ import (
 	"github.com/gorilla/websocket"
 )
 
-// defaultUpgrader is a pre-configured instance which provides reasonable default
-// values for various WebSocket upgrade parameters.
-var defaultUpgrader = websocket.Upgrader{}
-
 // NewStreamEndpoint creates a new Websocket Stream Endpoint. Streams are sent over
 // Websockets, with each Write becoming a separate message. Half-close is supported:
 // CloseRead will not close the Websocket connection, while CloseWrite sends a Websocket
@@ -203,7 +199,8 @@ func (c *gorillaConn) Close() error {
 // [transport.StreamConn] representing the WebSocket connection, or an error if
 // the upgrade fails.
 func Upgrade(w http.ResponseWriter, r *http.Request, responseHeader http.Header) (transport.StreamConn, error) {
-	wsConn, err := defaultUpgrader.Upgrade(w, r, responseHeader)
+	upgrader := websocket.Upgrader{}
+	wsConn, err := upgrader.Upgrade(w, r, responseHeader)
 	if err != nil {
 		return nil, err
 	}