diff options
author | Bert Belder <bertbelder@gmail.com> | 2023-02-10 20:41:20 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-10 19:41:20 +0000 |
commit | cf06a7c7e672880e1b38598fe445e2c50b4a9d06 (patch) | |
tree | 3d54861fd742d903e92971a6711f3479620d606c /ext/http/01_http.js | |
parent | 68008bee515e47fd9258d4f110dea6f352f2cd5b (diff) |
refactor(ext/http): use `String.prototype.trim()` instead of regex (#17722)
Diffstat (limited to 'ext/http/01_http.js')
-rw-r--r-- | ext/http/01_http.js | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/ext/http/01_http.js b/ext/http/01_http.js index 6807ad973..1da371e8d 100644 --- a/ext/http/01_http.js +++ b/ext/http/01_http.js @@ -51,6 +51,7 @@ const { StringPrototypeIncludes, StringPrototypeToLowerCase, StringPrototypeSplit, + StringPrototypeTrim, Symbol, SymbolAsyncIterator, TypeError, @@ -393,8 +394,9 @@ function upgradeWebSocket(request, options = {}) { const upgrade = request.headers.get("upgrade"); const upgradeHasWebSocketOption = upgrade !== null && ArrayPrototypeSome( - StringPrototypeSplit(upgrade, /\s*,\s*/), - (option) => StringPrototypeToLowerCase(option) === "websocket", + StringPrototypeSplit(upgrade, ","), + (option) => + StringPrototypeToLowerCase(StringPrototypeTrim(option)) === "websocket", ); if (!upgradeHasWebSocketOption) { throw new TypeError( @@ -405,8 +407,9 @@ function upgradeWebSocket(request, options = {}) { const connection = request.headers.get("connection"); const connectionHasUpgradeOption = connection !== null && ArrayPrototypeSome( - StringPrototypeSplit(connection, /\s*,\s*/), - (option) => StringPrototypeToLowerCase(option) === "upgrade", + StringPrototypeSplit(connection, ","), + (option) => + StringPrototypeToLowerCase(StringPrototypeTrim(option)) === "upgrade", ); if (!connectionHasUpgradeOption) { throw new TypeError( |