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

upd: cache logic update #18

Merged
merged 1 commit into from
Jan 7, 2025
Merged

upd: cache logic update #18

merged 1 commit into from
Jan 7, 2025

Conversation

muse254
Copy link
Owner

@muse254 muse254 commented Jan 7, 2025

Fixes for this PR:

  1. The cache_limit is respected and accurate; the bug was we were looking at the size of the buffer and not the string serialised buffer
  2. The expiry is also respected; the bug was it cleared the cache if the whole site was refreshed

@muse254 muse254 marked this pull request as ready for review January 7, 2025 09:56
@muse254 muse254 merged commit a0c8dad into main Jan 7, 2025
4 checks passed
@muse254 muse254 deleted the upd/cache-logic-1-1 branch January 7, 2025 09:59
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

Successfully merging this pull request may close these issues.

1 participant