Skip to content

fix: disable caching of error responses (300-499) on R2 custom domain#21939

Merged
alexghr merged 1 commit into
merge-train/spartanfrom
claudebox/cloudflare-no-cache-404
Mar 24, 2026
Merged

fix: disable caching of error responses (300-499) on R2 custom domain#21939
alexghr merged 1 commit into
merge-train/spartanfrom
claudebox/cloudflare-no-cache-404

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a Cloudflare cache rule to set edge_ttl = 0 (no-cache) for HTTP status codes 300-499 on the aztec-labs-snapshots.com custom domain.
  • By default, Cloudflare caches 404 responses for up to 3 minutes. This means that when a blob is requested before it's been uploaded to R2, the 404 gets cached and subsequent requests continue to get the stale 404 even after the blob becomes available.
  • This was causing ~8 second delays for nodes fetching blobs, as they had to wait for the cached 404 to expire.

References

ClaudeBox log: https://claudebox.work/s/f256b1637f377041?run=1

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 24, 2026
@alexghr alexghr force-pushed the claudebox/cloudflare-no-cache-404 branch from b3c7dcf to 354c6ce Compare March 24, 2026 10:45

@alexghr alexghr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have fixed and applied the TF

@alexghr alexghr marked this pull request as ready for review March 24, 2026 10:46
@alexghr alexghr enabled auto-merge (squash) March 24, 2026 10:46
@alexghr alexghr added ci-skip and removed ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Mar 24, 2026
@alexghr alexghr merged commit cbea6bd into merge-train/spartan Mar 24, 2026
20 of 29 checks passed
@alexghr alexghr deleted the claudebox/cloudflare-no-cache-404 branch March 24, 2026 10:47
github-merge-queue Bot pushed a commit that referenced this pull request Mar 24, 2026
BEGIN_COMMIT_OVERRIDE
chore: deflake n_tps benchmark for LOW_VALUE_TPS=2 (#21578)
fix(sequencer): use wall-clock timestamp for simulation when pipelining
is disabled (#21888)
fix: disable caching of error responses (300-499) on R2 custom domain
(#21939)
fix: add missing default cases in treeId switch statements (#21942)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants