-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Invalidate Repo and Permission Cache #1232
Milestone
Comments
implemented a 24 hour cache and informational messages in the UI in case the cache is stale |
changed to 15 minute cache a few weeks ago until we have a way to force-refresh from the UI |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The list of user repositories and the permissions to those repositories are cached in a simple LRU cache without expiration (due to lack of time on my part to focus on caching). We need a TTL to avoid rendering stale data in the user interface, otherwise this can result in the following behavior:
There will still be situations where our cache is stale, but we can handle this well enough in the user interface. For example, when I search for a repository that isn't in our cache (perhaps was recently created) we can show a message that says something like this:
We can also provide a button to manually flush the cache in order to force a refresh.
The text was updated successfully, but these errors were encountered: