@graphql-tools/executor-http - npm Package Compare versions

Comparing version 1.1.10-rc-3b80293b218b95108fd6a43eb6ce5dd9e20080eb to 1.1.10-rc-3c1b0d738213eb198826a03ab0566c9ebeab1e2f

CHANGELOG.md

 # @graphql-tools/executor-http
 
-## 1.1.10-rc-3b80293b218b95108fd6a43eb6ce5dd9e20080eb
+## 1.1.10-rc-3c1b0d738213eb198826a03ab0566c9ebeab1e2f
 
@@ -5,0 +5,0 @@ ### Patch Changes

dist/index.js

@@ -1,2 +0,2 @@
-import { isAsyncIterable, isPromise, fakePromise, inspect, mapAsyncIterator, mergeIncrementalResult, memoize1, getOperationASTFromRequest, createGraphQLError } from '@graphql-tools/utils';
+import { isAsyncIterable, isPromise, fakePromise, inspect, mapAsyncIterator, mergeIncrementalResult, memoize1, getOperationASTFromRequest, mapMaybePromise, createGraphQLError } from '@graphql-tools/utils';
 import { File, FormData, TextDecoder, fetch } from '@whatwg-node/fetch';
@@ -91,7 +91,14 @@ import { ValueOrPromise } from 'value-or-promise';
   }
-  return ValueOrPromise.all(
-    uploads.map(
-      (upload, i) => new ValueOrPromise(() => handleUpload(upload, i))
-    )
-  ).then(() => form).resolve();
+  const jobs = [];
+  for (const i in uploads) {
+    const upload = uploads[i];
+    const job = handleUpload(upload, Number(i));
+    if (isPromise(job)) {
+      jobs.push(job);
+    }
+  }
+  if (jobs.length > 0) {
+    return Promise.all(jobs).then(() => form);
+  }
+  return form;
 }
@@ -318,28 +325,29 @@ function isBlob(obj) {
           upstreamErrorExtensions.request.body = body;
-          return new ValueOrPromise(
-            () => createFormDataFromVariables(body, {
+          return mapMaybePromise(
+            createFormDataFromVariables(body, {
               File: options?.File,
               FormData: options?.FormData
-            })
-          ).then((body2) => {
-            if (typeof body2 === "string" && !headers["content-type"]) {
-              upstreamErrorExtensions.request.body = body2;
-              headers["content-type"] = "application/json";
+            }),
+            (body2) => {
+              if (typeof body2 === "string" && !headers["content-type"]) {
+                upstreamErrorExtensions.request.body = body2;
+                headers["content-type"] = "application/json";
+              }
+              const fetchOptions = {
+                method: "POST",
+                body: body2,
+                headers,
+                signal
+              };
+              if (options?.credentials != null) {
+                fetchOptions.credentials = options.credentials;
+              }
+              return fetchFn(
+                endpoint,
+                fetchOptions,
+                request.context,
+                request.info
+              );
             }
-            const fetchOptions = {
-              method: "POST",
-              body: body2,
-              headers,
-              signal
-            };
-            if (options?.credentials != null) {
-              fetchOptions.credentials = options.credentials;
-            }
-            return fetchFn(
-              endpoint,
-              fetchOptions,
-              request.context,
-              request.info
-            );
-          }).resolve();
+          );
         }
@@ -463,3 +471,3 @@ }
         }
-        return new ValueOrPromise(() => baseExecutor(request)).then((res) => {
+        return mapMaybePromise(baseExecutor(request), (res) => {
           result = res;
@@ -470,3 +478,3 @@ if (result?.errors?.length) {
           return result;
-        }).resolve();
+        });
       }
@@ -473,0 +481,0 @@ return retryAttempt();

package.json

 {
   "name": "@graphql-tools/executor-http",
-  "version": "1.1.10-rc-3b80293b218b95108fd6a43eb6ce5dd9e20080eb",
+  "version": "1.1.10-rc-3c1b0d738213eb198826a03ab0566c9ebeab1e2f",
   "type": "module",
@@ -5,0 +5,0 @@ "description": "A set of utils for faster development of GraphQL tools",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

62817

increased by0.36%

Lines of code

1191

increased by1.36%

No dependency changes