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

Improve the efficiency of the service with caching #11

Open
jon-betts opened this issue Nov 10, 2020 · 0 comments
Open

Improve the efficiency of the service with caching #11

jon-betts opened this issue Nov 10, 2020 · 0 comments
Assignees

Comments

@jon-betts
Copy link
Contributor

jon-betts commented Nov 10, 2020

We can improve the speed and cost efficiency by using caching in certain circumstances:

  • Resources which are flagged by our internal process can be cached for a long time
  • Resources which are flagged by external services can be cached for a shorter time
  • Resources which are not flagged by anything cannot be cached as they could prove bad at any time (This could be cached for a short time, at the cost of a delay in responsiveness)

Implementation notes

Depending on whether we end up protecting our end-points with a blanket block, or a more tailored approach in #9, Cloudflare might not be in the mix from a point of view of caching.

In that case we would probably have to fallback to local NGINX caches and client cache, which isn't as good. And as our services are the clients, they won't cache.

@jon-betts jon-betts added this to the Improve the checking service milestone Nov 10, 2020
@jon-betts jon-betts self-assigned this Nov 10, 2020
@jon-betts jon-betts modified the milestones: 2 - Improve the checking service, 4 - Idea factory Jan 5, 2021
@seanh seanh removed the tech debt label Feb 16, 2021
@seanh seanh removed this from the Ideas: Misc milestone Feb 16, 2021
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

2 participants