Skip to content

Conversation

dario-piotrowicz
Copy link
Contributor

@dario-piotrowicz dario-piotrowicz commented Aug 13, 2025

No description provided.

Copy link

changeset-bot bot commented Aug 13, 2025

🦋 Changeset detected

Latest commit: 6082569

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@opennextjs/cloudflare Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

pkg-pr-new bot commented Aug 13, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@850

commit: 6082569

@dario-piotrowicz dario-piotrowicz changed the title fix: make regional cache hits bypass the tagCache Add option for regional cache to skip tagCache on cache hits Aug 13, 2025
@dario-piotrowicz dario-piotrowicz marked this pull request as ready for review August 14, 2025 10:34
Copy link
Collaborator

@conico974 conico974 left a comment

Choose a reason for hiding this comment

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

LGTM, maybe just add some comment about the need for a custom domain

dario-piotrowicz and others added 5 commits August 15, 2025 12:20
When the `regional-cache` is used and it finds an entry, there's no need
to then interact with the `tagCache`, so the changes here add
`shouldBypassTagCache: true` to the cache response to avoid interactions
with the tagCache (which can slow down response times)
@dario-piotrowicz dario-piotrowicz force-pushed the dario/regional-cache-shouldBypassTagCache branch from f24ab92 to 49cd870 Compare August 15, 2025 11:20
Co-authored-by: James Anderson <[email protected]>
@dario-piotrowicz dario-piotrowicz merged commit ce5c7b4 into main Aug 18, 2025
7 checks passed
@dario-piotrowicz dario-piotrowicz deleted the dario/regional-cache-shouldBypassTagCache branch August 18, 2025 20:39
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.

3 participants