Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor cacher middleware to use Cache API #190

Open
sejori opened this issue Oct 24, 2023 · 4 comments
Open

Refactor cacher middleware to use Cache API #190

sejori opened this issue Oct 24, 2023 · 4 comments

Comments

@sejori
Copy link
Owner

sejori commented Oct 24, 2023

https://developer.mozilla.org/en-US/docs/Web/API/Cache

https://deno.land/api@v1.37.2?s=Cache

@sejori
Copy link
Owner Author

sejori commented Jul 26, 2024

@sejori
Copy link
Owner Author

sejori commented Jul 26, 2024

@sejori
Copy link
Owner Author

sejori commented Jul 26, 2024

An issue with using the Web Cache API is that it wouldn't work well for graphql where two different queries could ask for the same/similar data. We can cache at the request level but it wouldn't effectively serve this case.

We will need caching to support dataloaders/query resolvers. Unless, we resolve a GraphQL query with multiple load requests under-the-hood. Which could work actually...

@sejori
Copy link
Owner Author

sejori commented Jul 26, 2024

^ This would make doing federated graphQL a lot easier as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant