fix: optimize Sec-WebSocket-Accept header check

- Remove unused variable assignment in condition
- Direct comparison is more efficient (no extra variable allocation)
- Maintains same defensive check behavior
This commit is contained in:
wjqserver 2026-04-19 07:52:00 +08:00
parent b008fc8e61
commit 3b5f2c81af

View file

@ -1014,7 +1014,7 @@ func (p *reverseProxyHandler) handleBridgedExtendedConnectResponse(c *Context, r
responseHeader := c.Writer.Header() responseHeader := c.Writer.Header()
reverseProxyCopyHeader(responseHeader, res.Header) reverseProxyCopyHeader(responseHeader, res.Header)
removeHopByHopHeaders(responseHeader) removeHopByHopHeaders(responseHeader)
if accept := res.Header.Get("Sec-WebSocket-Accept"); accept != "" { if res.Header.Get("Sec-WebSocket-Accept") != "" {
responseHeader.Del("Sec-WebSocket-Accept") responseHeader.Del("Sec-WebSocket-Accept")
} }
c.Writer.WriteHeader(http.StatusOK) c.Writer.WriteHeader(http.StatusOK)