graphile / crystal

🔮 Graphile's Crystal Monorepo; home to Grafast, PostGraphile, pg-introspection, pg-sql2 and much more!
https://graphile.org/
Other
12.62k stars 571 forks source link

Enable stabilization of objects/lists going through loadMany/loadOne multistep #2193

Closed benjie closed 1 month ago

benjie commented 1 month ago

Description

Fixes https://github.com/graphile/crystal/issues/2170

Performance impact

Marginal performance cost for loadOne/loadMany calls that accept object/list specs.

Security impact

Generally an improvement I hope, though there's a risk of memory exhaustion attacks if this is not used sensibly.

Checklist

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: b77f6e909c1c0f1921f75726384472e945ea698b

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

This PR includes changesets to release 15 packages | Name | Type | | -------------------------- | ----- | | @dataplan/pg | Patch | | grafast | Patch | | graphile-build-pg | Patch | | graphile-utils | Patch | | pgl | Patch | | postgraphile | Patch | | @localrepo/grafast-bench | Patch | | @dataplan/json | Patch | | @grafserv/persisted | Patch | | grafserv | Patch | | ruru-components | Patch | | @localrepo/grafast-website | Patch | | graphile-build | Patch | | graphile-export | Patch | | graphile | 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