From f6278b27cf8b88535bae16041eb76a764317f10e Mon Sep 17 00:00:00 2001 From: Sebastian Serth Date: Wed, 10 May 2023 00:26:27 +0200 Subject: [PATCH] Determine WebSocket connection without Upgrade header The HTTP Upgrade header is hop-by-hop only and therefore removed by a reverse proxy. Read more: http://nginx.org/en/docs/http/websocket.html --- lib/middleware/websocket_sentry_headers.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/middleware/websocket_sentry_headers.rb b/lib/middleware/websocket_sentry_headers.rb index f2bbd995..c7b50322 100644 --- a/lib/middleware/websocket_sentry_headers.rb +++ b/lib/middleware/websocket_sentry_headers.rb @@ -15,8 +15,7 @@ module Middleware private def websocket_upgrade?(request) - request.get_header('HTTP_CONNECTION')&.casecmp?('Upgrade') && - request.get_header('HTTP_UPGRADE')&.casecmp?('websocket') + request.has_header?('HTTP_SEC_WEBSOCKET_VERSION') end def extract_sentry_parameters(request)