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

fix(HTTP Cache): Use cache_key_with_version for etag #76

Merged
merged 1 commit into from
May 7, 2024

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented May 6, 2024

The cache_key is just the page id. We need to include the cache_version (the page published_at date).

@tvdeyen tvdeyen self-assigned this May 6, 2024
@tvdeyen tvdeyen force-pushed the cache-key-with-version branch from ecfcd22 to bbe4fbc Compare May 6, 2024 20:22
@tvdeyen tvdeyen marked this pull request as ready for review May 6, 2024 20:22
@tvdeyen tvdeyen requested a review from a team May 6, 2024 20:36
@tvdeyen tvdeyen force-pushed the cache-key-with-version branch 3 times, most recently from c86c7e5 to cb91975 Compare May 6, 2024 20:52
@tvdeyen tvdeyen removed their assignment May 6, 2024
@tvdeyen tvdeyen enabled auto-merge May 6, 2024 21:02
@tvdeyen tvdeyen disabled auto-merge May 6, 2024 21:02
@tvdeyen tvdeyen marked this pull request as draft May 6, 2024 21:02
@tvdeyen tvdeyen removed the request for review from a team May 6, 2024 21:02
The cache_key is just the page id. We need to include
the cache_version (the page published_at date).
@tvdeyen tvdeyen force-pushed the cache-key-with-version branch from cb91975 to 0b28e43 Compare May 7, 2024 09:49
@tvdeyen tvdeyen marked this pull request as ready for review May 7, 2024 09:50
@tvdeyen tvdeyen requested a review from a team May 7, 2024 09:50
@tvdeyen tvdeyen merged commit 7302dac into main May 7, 2024
16 checks passed
@tvdeyen tvdeyen deleted the cache-key-with-version branch May 7, 2024 09:51
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.

2 participants