Skip to content

Fix N+1 issue for GraphQL VulnerabilityType

Jarka Košanová requested to merge fix-vuln-graphql-performance into master

What does this MR do and why?

This MR fixes the N+1 problem caused by authorization.

Basically, authorization is made for every single record that is found, even for collections. I moved the authorizations to resolvers/fields.

I think it is a general GraphQL problem, opened to discussion, maybe even as a follow-up issue.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Jarka Košanová

Merge request reports