Upgrade undici to 5.5.1 (#286)

* Upgrade undici to 5.5.1

* Remove header filtering hack, ref #288

Co-authored-by: MrBBot <[email protected]>
Co-authored-by: Cherry <[email protected]>
(cherry picked from commit 979e875)

Author: 3 people

package-lock.json

@@ -38,7 +38,7 @@
     "@miniflare/core": "2.5.0",
     "@miniflare/shared": "2.5.0",
     "http-cache-semantics": "^4.1.0",
-    "undici": "5.3.0"
+    "undici": "5.5.1"
   },
   "devDependencies": {
     "@miniflare/shared-test": "2.5.0",

packages/cache/package.json

@@ -45,7 +45,7 @@
     "dotenv": "^10.0.0",
     "kleur": "^4.1.4",
     "set-cookie-parser": "^2.4.8",
-    "undici": "5.3.0",
+    "undici": "5.5.1",
     "urlpattern-polyfill": "^4.0.3"
   },
   "devDependencies": {

packages/core/package.json

@@ -53,20 +53,6 @@ import {
 } from "./helpers";
 import { kContentLength } from "./streams";
 
-// `undici` restricts which headers can be added to/gotten from a `Headers`
-// object in `Request`/`Response`s. Whilst this restriction makes sense for
-// security reasons in the browser, it doesn't server side.
-//
-// Note: this is a massive hack. I'm fully expecting it to break in future
-// `undici` versions and need to be updated, but that's why we pin our `undici`
-// version and have tests... :) Right?
-const constants: {
-  readonly forbiddenHeaderNames: string[];
-  readonly forbiddenResponseHeaderNames: string[];
-} = require("undici/lib/fetch/constants.js");
-constants.forbiddenHeaderNames.length = 0;
-constants.forbiddenResponseHeaderNames.length = 0;
-
 // We need these for making Request's Headers immutable
 const fetchSymbols: {
   readonly kState: unique symbol;

packages/core/src/standards/http.ts

@@ -38,7 +38,7 @@
     "@miniflare/core": "2.5.0",
     "@miniflare/shared": "2.5.0",
     "@miniflare/storage-memory": "2.5.0",
-    "undici": "5.3.0"
+    "undici": "5.5.1"
   },
   "devDependencies": {
     "@miniflare/cache": "2.5.0",

packages/durable-objects/package.json

@@ -38,7 +38,7 @@
     "@miniflare/core": "2.5.0",
     "@miniflare/shared": "2.5.0",
     "html-rewriter-wasm": "^0.4.1",
-    "undici": "5.3.0"
+    "undici": "5.5.1"
   },
   "devDependencies": {
     "@miniflare/shared-test": "2.5.0"

packages/html-rewriter/package.json

@@ -40,7 +40,7 @@
     "@miniflare/web-sockets": "2.5.0",
     "kleur": "^4.1.4",
     "selfsigned": "^2.0.0",
-    "undici": "5.3.0",
+    "undici": "5.5.1",
     "ws": "^8.2.2",
     "youch": "^2.2.2"
   },

packages/http-server/package.json

@@ -63,7 +63,7 @@
     "kleur": "^4.1.4",
     "semiver": "^1.1.0",
     "source-map-support": "^0.5.20",
-    "undici": "5.3.0"
+    "undici": "5.5.1"
   },
   "devDependencies": {
     "@miniflare/shared-test": "2.5.0",

packages/miniflare/package.json

@@ -38,7 +38,7 @@
     "@miniflare/core": "2.5.0",
     "@miniflare/shared": "2.5.0",
     "ws": "^8.2.2",
-    "undici": "5.3.0"
+    "undici": "5.5.1"
   },
   "devDependencies": {
     "@miniflare/shared-test": "2.5.0",