fix: define request.context as read only (#133)

unjs · Feb 10, 2025 · f8372f8 · f8372f8
1 parent 52f5538
commit f8372f8
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/hooks.ts b/src/hooks.ts
@@ -41,13 +41,21 @@ export class AdapterHookable {
   }
 
   async upgrade(
-    request: UpgradeRequest & { context?: Peer["context"] },
+    request: UpgradeRequest & { readonly context?: Peer["context"] },
   ): Promise<{
     upgradeHeaders?: HeadersInit;
     endResponse?: Response;
     context: Peer["context"];
   }> {
-    const context = (request.context ??= {});
+    let context = request.context;
+    if (!context) {
+      context = {};
+      Object.defineProperty(request, "context", {
+        enumerable: true,
+        value: context,
+      });
+    }
+
     try {
       const res = await this.callHook(
         "upgrade",