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

Cache server-side, external API requests #1162

Open
antidipyramid opened this issue Oct 4, 2024 · 0 comments
Open

Cache server-side, external API requests #1162

antidipyramid opened this issue Oct 4, 2024 · 0 comments
Assignees

Comments

@antidipyramid
Copy link
Collaborator

There are a few places where we're making requests to Granicus on behalf of users (e.g. to verify meeting media links or get running events). These should all be cached so we're not making these requests to Granicus for every single user.

The homepage in particular slows down (>500 ms) on meeting days because of multiple Granicus requests. Just a 60 second server-side cache would speed up the homepage for the vast majority of requests.

We added some explicit caching for running events requests in #1111. For code clarity, it's probably a good idea to do this caching in the timed_get function so we're not having to manually set/clear the cache for every request.

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

No branches or pull requests

1 participant