HoudiniGraphql / houdini

The disappearing GraphQL framework
http://www.houdinigraphql.com
MIT License
913 stars 98 forks source link

Add configurable timeout to fetching the remote schema #1324

Closed SeppahBaws closed 4 months ago

SeppahBaws commented 4 months ago

Fixes #1302

This adds a timeout to pullSchema. In the rare event that the remote API is unreachable (the request is not cancelled immediately, but just never resolves), pullSchema will show a warning that the remote API is unreachable, instead of hanging indefinitely before nodejs cancels the fetch automatically. (which can sometimes take minutes!)

To help everyone out, please make sure your PR does the following:

changeset-bot[bot] commented 4 months ago

🦋 Changeset detected

Latest commit: 661487129c9160364cb323702e706d39b31a4c1a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages | Name | Type | | ----------------------------------- | ----- | | houdini | Patch | | houdini-adapter-auto | Patch | | houdini-adapter-cloudflare | Patch | | houdini-adapter-node | Patch | | houdini-react | Patch | | houdini-svelte | Patch | | houdini-plugin-svelte-global-stores | Patch | | create-houdini | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

netlify[bot] commented 4 months ago

Deploy Preview for houdinigraphql canceled.

Name Link
Latest commit 661487129c9160364cb323702e706d39b31a4c1a
Latest deploy log https://app.netlify.com/sites/houdinigraphql/deploys/668f879f147af100088d0ec4
netlify[bot] commented 4 months ago

Deploy Preview for houdini-docs-next canceled.

Name Link
Latest commit 661487129c9160364cb323702e706d39b31a4c1a
Latest deploy log https://app.netlify.com/sites/houdini-docs-next/deploys/668f879ff36bd900081b9256