@graphql-tools/executor-http 3.0.7-rc-8db5f91d48d70343835f0385391ceedca11067e4 → 3.0.7-rc-fb75c546d23aab2b1d2b4679334d64d4ee47d013

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # @graphql-tools/executor-http
 
-## 3.0.7-rc-8db5f91d48d70343835f0385391ceedca11067e4
+## 3.0.7-rc-fb75c546d23aab2b1d2b4679334d64d4ee47d013
 ### Patch Changes
 
 
@@ -13,8 +13,12 @@
 - [#1691](https://github.com/graphql-hive/gateway/pull/1691) [`7ecaf7e`](https://github.com/graphql-hive/gateway/commit/7ecaf7e8f658c4e4c1a91d1e8db3c1a8ceca51cb) Thanks [@dependabot](https://github.com/apps/dependabot)! - dependencies updates:
   
   - Updated dependency [`@graphql-tools/utils@^10.10.3` ↗︎](https://www.npmjs.com/package/@graphql-tools/utils/v/10.10.3) (from `^10.10.1`, in `dependencies`)
+
+
+- [#1347](https://github.com/graphql-hive/gateway/pull/1347) [`2993f24`](https://github.com/graphql-hive/gateway/commit/2993f24f8e46853c0d2667c0cd2bf96190f3a2b2) Thanks [@ardatan](https://github.com/ardatan)! - Avoid shared AbortController instance on CloudflareWorkers because it gives `Cannot perform I/O on behalf of a different request.` error.
+  This change ensures that the AbortController is only created when not running in a Cloudflare Workers environment.
 - Updated dependencies [[`7ecaf7e`](https://github.com/graphql-hive/gateway/commit/7ecaf7e8f658c4e4c1a91d1e8db3c1a8ceca51cb)]:
-  - @graphql-tools/executor-common@1.0.5-rc-8db5f91d48d70343835f0385391ceedca11067e4
+  - @graphql-tools/executor-common@1.0.5-rc-fb75c546d23aab2b1d2b4679334d64d4ee47d013
 
 ## 3.0.6
 ### Patch Changes

package/dist/index.cjs

@@ -543,8 +543,10 @@ function buildHTTPExecutor(options) {
   const printFn = options?.print ?? executorCommon.defaultPrintFn;
   let disposeCtrl;
   const baseExecutor = (request, excludeQuery) => {
-    disposeCtrl ||= new AbortController();
-    if (disposeCtrl.signal.aborted) {
+    if (!request.context?.request?.cf) {
+      disposeCtrl ||= new AbortController();
+    }
+    if (disposeCtrl?.signal.aborted) {
       return createResultForAbort(disposeCtrl.signal.reason);
     }
     const fetchFn = request.extensions?.fetch ?? options?.fetch ?? fetch.fetch;
@@ -598,7 +600,10 @@ function buildHTTPExecutor(options) {
       Object.assign(headers, headersFromExtensions);
       request.extensions = restExtensions;
     }
-    const signals = [disposeCtrl.signal];
+    const signals = [];
+    if (disposeCtrl?.signal) {
+      signals.push(disposeCtrl.signal);
+    }
     const signalFromRequest = request.signal || request.info?.signal;
     if (signalFromRequest) {
       if (signalFromRequest.aborted) {
@@ -612,7 +617,7 @@ function buildHTTPExecutor(options) {
     if (subscriptionCtrl) {
       signals.push(subscriptionCtrl.signal);
     }
-    const signal$1 = signal.abortSignalAny(signals);
+    const signal$1 = signals.length ? signal.abortSignalAny(signals) : void 0;
     const upstreamErrorExtensions = {
       request: {
         method

package/dist/index.js

@@ -541,8 +541,10 @@ function buildHTTPExecutor(options) {
   const printFn = options?.print ?? defaultPrintFn;
   let disposeCtrl;
   const baseExecutor = (request, excludeQuery) => {
-    disposeCtrl ||= new AbortController();
-    if (disposeCtrl.signal.aborted) {
+    if (!request.context?.request?.cf) {
+      disposeCtrl ||= new AbortController();
+    }
+    if (disposeCtrl?.signal.aborted) {
       return createResultForAbort(disposeCtrl.signal.reason);
     }
     const fetchFn = request.extensions?.fetch ?? options?.fetch ?? fetch;
@@ -596,7 +598,10 @@ function buildHTTPExecutor(options) {
       Object.assign(headers, headersFromExtensions);
       request.extensions = restExtensions;
     }
-    const signals = [disposeCtrl.signal];
+    const signals = [];
+    if (disposeCtrl?.signal) {
+      signals.push(disposeCtrl.signal);
+    }
     const signalFromRequest = request.signal || request.info?.signal;
     if (signalFromRequest) {
       if (signalFromRequest.aborted) {
@@ -610,7 +615,7 @@ function buildHTTPExecutor(options) {
     if (subscriptionCtrl) {
       signals.push(subscriptionCtrl.signal);
     }
-    const signal = abortSignalAny(signals);
+    const signal = signals.length ? abortSignalAny(signals) : void 0;
     const upstreamErrorExtensions = {
       request: {
         method

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-tools/executor-http",
-  "version": "3.0.7-rc-8db5f91d48d70343835f0385391ceedca11067e4",
+  "version": "3.0.7-rc-fb75c546d23aab2b1d2b4679334d64d4ee47d013",
   "type": "module",
   "description": "A set of utils for faster development of GraphQL tools",
   "repository": {
@@ -40,7 +40,7 @@
   },
   "dependencies": {
     "@graphql-hive/signal": "^2.0.0",
-    "@graphql-tools/executor-common": "1.0.5-rc-8db5f91d48d70343835f0385391ceedca11067e4",
+    "@graphql-tools/executor-common": "1.0.5-rc-fb75c546d23aab2b1d2b4679334d64d4ee47d013",
     "@graphql-tools/utils": "^10.10.3",
     "@repeaterjs/repeater": "^3.0.4",
     "@whatwg-node/disposablestack": "^0.0.6",