Add lookahead to dastSiteValidations resolver
The following discussion from !50635 (merged) should be addressed:
-
@reprazent started a discussion: (+2 comments) Could we make sure that this query is protected against N+1 queries when requesting the field multiple times in a single GraphQL query: We'll probably need to wrap this in a
BatchLoader
: https://docs.gitlab.com/ee/development/api_graphql_styleguide.html#resolversThis can go into a separate issue, since the code is already merged.
Implementation
-
Add lookahead -
Add test to check for n + 1
issues
Edited by Philip Cunningham