[SLING-9753] Return a 500 status code if the persisted query cannot be stored in the cache

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: GraphQL Core 0.0.6
• Component/s: GraphQL
• Assignee: Radu Cotescu
• Reporter: Radu Cotescu
• Created: 21/Sep/20 16:17
• Updated: 09/Oct/20 10:21
• Resolved: 22/Sep/20 21:47

Description

SLING-9655 added support for GraphQL persisted queries. However, the current implementation still returns a query's cache and allows a client to perform cache retrievals even if the query was not actually stored in the cache.

This happens due to the LRU cache implemented by org.apache.sling.graphql.core.cache.SimpleGraphQLCacheProvider, which evicts the oldest entry without checking if the new entry can actually be stored when using a cache memory limit configuration.