Opt-in ISR for pages that are identical for every user behind Cloudflare Access
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- typescript
- Domaine
- backend, performance
Piste de recherche
Start with the ISR lookup and response-copy paths named in the issue: void/dist/runtime/routing.mjs and clientRevalidateResponse() in @void/isr; also read the linked cache-bypass and Zero Trust documentation. Trace how credentials, HTML and X-VoidPages responses are handled on MISS, HIT, STALE and background revalidation. Done means the opt-in follows the stated credential, storage and browser-cache rules while preserving Access checks; the issue leaves naming and configuration scope open.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
Void skips ISR for every request that carries Cookie, Authorization, or (since #335) Cf-Access-Jwt-Assertion (Cache bypass). That is the right default. But it means an app behind Cloudflare Access, including a Void Platform project with Zero Trust protection, can never be served from the ISR cache, even for pages that are the same for everyone Access lets in. The Zero Trust docs already note the cost: "Protected pages render on each request, which can increase latency and request usage." (Caching)
Access decides who may see the app. I'd like a per-path opt-in where the developer states that the response is the same for everyone Access admits, so Void can serve it from the ISR cache once Access has admitted the request. The default would not change.
Use case
A private, annotated collection of statutes behind Access: tens of thousands of pages, a handful of readers, most pages rarely opened. The statute pages are the same for every reader, so we want each page rendered on first request and served from the ISR cache afterwards. Per-user parts (bookmarks, notes, "last read") come from islands that call uncached /api routes, so the cached HTML stays shared. Streaming such parts with defer() would not work, since they would be stored as part of the cached response. output: "static" would render every page on every build, while most pages are never read. Behind Access today, every view renders instead.
What reaches the Worker behind Access
Checked on our production site (workers.dev, protected by Access) with a route that returns only cookie names and header presence: a signed-in browser's request reaches the Worker with the cookies CF_AppSession and CF_Authorization, the headers Cf-Access-Jwt-Assertion and Cf-Access-Authenticated-User-Email, and no Authorization header. Each of these alone is enough to bypass ISR today.
Local reproduction of the current bypass ([email protected], vite preview)
ISR_CACHE was a local KV namespace bound through cloudflare.kv_namespaces, so this is not Cloudflare's edge and not a real Access deployment; Access was simulated by adding its header and cookies by hand. The page is pages/laws/[id].tsx with export const revalidate = 31536000, and its loader returns a render timestamp. Each row shows two requests for the same path:
| Request credentials | 1st | 2nd | Same HTML |
|---|---|---|---|
| none | x-isr-cache: MISS |
HIT |
yes |
Cf-Access-Jwt-Assertion |
no ISR header | no ISR header | no |
Cookie: CF_Authorization=… |
no ISR header | no ISR header | no |
Cookie: CF_AppSession=… |
no ISR header | no ISR header | no |
both Access cookies + Cf-Access-Jwt-Assertion |
no ISR header | no ISR header | no |
Cookie: theme=dark |
no ISR header | no ISR header | no |
Authorization: Bearer … |
no ISR header | no ISR header | no |
A path warmed without credentials was rendered again when requested with both Access cookies and Cf-Access-Jwt-Assertion. Entries are keyed by hostname (127.0.0.1 and localhost each started with MISS). Responses served from the ISR cache carried Cache-Control: public, max-age=0, must-revalidate.
Proposal
The name is a placeholder. The opt-in could sit next to the per-page revalidate export (and, mirroring revalidate, as a routing pattern in void.config.ts):
// pages/laws/[id].server.ts
export const revalidate = 31536000;
export const sharedBehindAccess = true; // placeholder name
What the opt-in means. The whole cached response (layout, embedded loader data and headers included) is the same for every user Access admits, and no authorization beyond Access is needed. The second half matters because a cache hit is served before the app runs: in 1.1.11 the ISR lookup in the Worker's fetch handler comes before app.fetch, so middleware does not run on a hit (void/dist/runtime/routing.mjs). A page that reads the identity, or relies on app middleware to turn some admitted users away, must not opt in.
For a path that opts in:
- Ignored credentials. Void parses the
Cookieheader. If every cookie name isCF_AuthorizationorCF_AppSession(exact names; both are documented on the application hostname in Authorization cookie, and both reach the Worker as shown above) and the only other credential isCf-Access-Jwt-Assertion, the request uses the ISR cache like a public request. Any other cookie, and anyAuthorizationheader, still bypasses the cache. No prefix matching such asCF_*;CF_Bindingis stripped before the origin, andCF_SessionandCF_Devicebelong to the team domain. - Storage rules unchanged. Whether to store is decided on the app's response as today (complete
200only; noSet-Cookie,private,no-store,no-cacheorVaryon request headers;x-revalidate: 0still opts out). - Browser-facing copy is private. For requests that carried Access credentials, the copy sent to the browser gets
Cache-Control: private, no-cacheinstead ofpublic, max-age=0, must-revalidate, onMISS,HIT,STALEand after background revalidation alike. The runtime already builds the stored copy and the client copy separately (clientRevalidateResponse()in@void/isr). - Same boundary for the Pages JSON. The
X-VoidPagesJSON variant follows the same rules as the HTML. - Paths that don't opt in behave exactly as today.
Safety
- Cache hits stay behind the existing checks. On Void Platform, #335 says that dispatch checks the Access JWT again before it serves Worker, static or ISR content; every hit for an opted-in path must keep that order. With direct Cloudflare deploys, Access runs in front of the Worker; this proposal relies on that, does not treat the presence of an Access cookie or header as proof, and does not ask for JWT validation in the Worker.
- No reuse across hostnames. Entries are keyed by hostname, so an entry filled through a protected hostname is never served on another one.
- Identity-read detection is only an aid. Every admitted request carries
Cf-Access-Authenticated-User-Email, so a render can always read the identity. Void could warn in dev, or refuse to store, when a render on an opted-in path reads it, but a hit renders nothing; the developer's statement is what safety rests on. The docs should say so plainly, as #408 does for hand-writtenETags.
Acceptance criteria
- A cache entry filled by admitted user A is served to admitted user B.
- No content is served from the cache to a request that is unauthenticated, carries invalid credentials, or was rejected by Access, even when an entry exists.
- A request with any non-Access cookie or an
Authorizationheader neither reads nor writes the cache. MISS,HIT,STALEand background revalidation apply the same storage rules and the same browser-facing policy.- The
X-VoidPagesJSON variant follows the same boundary as the HTML.
Current workaround and alternatives
We turned ISR off and cache the HTML in our own Cache API middleware. It works, but it loses stale-while-revalidate, per-path TTLs and revalidate({ paths }), and it ignores every credential, not only Access's (in the reproduction it served cached pages to requests with other cookies or Authorization). An Access Bypass policy for those paths would make them public. output: "static" is covered in the use case above.
Open questions
- Naming, and whether the opt-in belongs in a per-page export, in
routing, or both; a single switch may suit an app that sits entirely behind Access. - Cloudflare's edge cookies (
__cf_bm,cf_clearance) on custom domains where Bot Fight Mode or a WAF challenge runs alongside Access.
Environment
[email protected],@void/md, React 19.3.1, Vite 8.3.4, Node.js 24.22.0, pnpm 12.14.0- Production:
void deploy --platform cloudflaretoworkers.dev, protected by Access ("Manage Worker access"). ISR is off there. - Reproduction: local
vite preview(workerd) with a local KV namespace asISR_CACHE.
- Langage dominant
- TypeScript
- Étoiles
- 627
- Forks
- 66
- Merge moyen
- 14 h 38 min
- PR mergées (30 j)
- 179
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de voidzero-dev/void
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
voidzero-dev/void#677 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
voidzero-dev/void#689 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
voidzero-dev/void#686 ·
Les mainteneurs répondent en général sous 1 jour
-
`VoidActionError.message` ignores the server's message, though the docs show it to usersPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
voidzero-dev/void#680 ·
Les mainteneurs répondent en général sous 1 jour
-
`withValidator({ body })` rejects the multipart bodies that `useForm` and `action()` send for filesOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
voidzero-dev/void#679 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de voidzero-dev/void
Issues similaires
-
Add: Yichang TVOuvertecheck:passed streams:add
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
vtselect: data race in Tempo search tags APIs for invalid TraceQL queriesPeut-être pris @Shubham19032004 l’a pris aujourd’hui. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
VictoriaMetrics/VictoriaTraces#304 ·
Les mainteneurs répondent en général sous 1 jour
-
test: claim-conflict drop leaves both facts active and is never re-proposed (untested review cell)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
microsoft/vscode-pull-request-github#9159 ·
Les mainteneurs répondent en général sous 1 jour
-
[Feature]: Usage page should show API-equivalent cost and say when a connected plan shows info onlyOuverteenhancement from-x
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
nightly-labs/openbot#4867 ·
Les mainteneurs répondent en général sous 1 jour