HoudiniGraphql / houdini

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

Fix memory leak in error path #1296

Closed AlecAivazis closed 7 months ago

AlecAivazis commented 7 months ago

This PR fixes a memory leak in the cache that was found as part of the investigation into #1294. This PR does not fix the original issue since it requires some additional thinking and there is a work-around

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

changeset-bot[bot] commented 7 months ago

🦋 Changeset detected

Latest commit: e5240dfa09f87f7699892cda982345f06ed20d8d

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 7 months ago

Deploy Preview for houdinigraphql canceled.

Name Link
Latest commit e5240dfa09f87f7699892cda982345f06ed20d8d
Latest deploy log https://app.netlify.com/sites/houdinigraphql/deploys/6629f215aade93000837a2f1
netlify[bot] commented 7 months ago

Deploy Preview for houdini-docs-next canceled.

Name Link
Latest commit e5240dfa09f87f7699892cda982345f06ed20d8d
Latest deploy log https://app.netlify.com/sites/houdini-docs-next/deploys/6629f215b9ec5f00071880fa