Support for Query Complexity Analysis about lighthouse HOT 7 CLOSED

It might also be nice to add native support for the other https://webonyx.github.io/graphql-php/security/ features too.

from lighthouse.

What about a @queryDepth(maxDepth: 10) directive we can set on each query? :)
And maybe also in conf add a queryDepth so we can set it as global

from lighthouse.

@olivernybroe Sounds good. I'm way more interested in the Query Complexity Analysis than Limiting Query Depth, but yeah, both features are definitely needed.

from lighthouse.

@TGNThump Implementing them would be really similiar as they are both just Rules. But I am totally up for having Complexy, QueryDepth and prob. also Introspection disabling.

from lighthouse.

@olivernybroe All three of those are definitely items that should be available in Lighthouse... added to v2.x label. First beta of subscriptions is done, I need to add some initial docs for it then I'll cut v2.1 of Lighthouse and I'll get started on new items (such as #101, #99, #98) for v2.2. I'll try to keep this issue updated as these items get added in. Thanks @TGNThump @olivernybroe!!

from lighthouse.

Does the SecurityDirective resolve this issue or does it need anything else?

from lighthouse.

Closing this, as SecurityDirective and the ComplexityDirective should cover it. @TGNThump if you are still missing something, let us know.

from lighthouse.