envelop feat(response-cache): added getScope callback in buildResponseCacheKey

feat(response-cache): added getScope callback in buildResponseCacheKey

Open Dunk14 opened this issue 10 months ago • 1 comments

Issue related : https://github.com/n1ru4l/envelop/issues/2181

Description

Include a getScope callback in buildResponseCacheKey to enable an Apollo like behavior, where PUBLIC scopes have only one cache key to get better performances on larger scales.

Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?

I added some unit tests to check that PRIVATE scope is correctly retrieved from a query, when directive @cacheControl(scope: PRIVATE) is put on query, field and subfield.

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules