Skip to content

fix(cache): fix issues with if-none-match and if-match - #1611

Merged
zkat merged 1 commit into
mainfrom
zkat/fix-cache
Nov 2, 2026
Merged

zkat merged 1 commit into
mainfrom
zkat/fix-cache

Conversation

@zkat

@zkat zkat commented Oct 29, 2026

Copy link
Copy Markdown
Member

Fixes: #1491

@dgryski
dgryski requested a review from TartanLlama October 29, 2026 21:58
@zkat
zkat enabled auto-merge (squash) October 29, 2026 22:02
@TartanLlama

Copy link
Copy Markdown
Contributor

@zkat the new test failure appears to be real, can you have a look?

@zkat

zkat commented Oct 30, 2026

Copy link
Copy Markdown
Member Author

@TartanLlama all fixed! It was an unnecessary detail in the failing test, which actually showed that things are working properly now

@zkat
zkat merged commit d3322b0 into main Nov 2, 2026
29 checks passed
@zkat
zkat deleted the zkat/fix-cache branch November 2, 2026 08:23
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.

Bug: --enable-http-cache causes 304 Not Modified HIT traffic to drop to 0

2 participants