Skip to content

Commit

Permalink
feat: remove cache: no-store as it breaks cloudflare (#886)
Browse files Browse the repository at this point in the history
Fixes supabase/supabase-js#1001

Hopefully the NextJS caching takes in the Authorization header in the
cache key.
  • Loading branch information
hf authored Apr 23, 2024
1 parent 760611b commit 10e9d38
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/lib/fetch.ts
Original file line number Diff line number Diff line change
Expand Up @@ -177,9 +177,6 @@ async function _handleRequest(
try {
result = await fetcher(url, {
...requestParams,
// UNDER NO CIRCUMSTANCE SHOULD THIS OPTION BE REMOVED, YOU MAY BE OPENING UP A SECURITY HOLE IN NEXT.JS APPS
// https://nextjs.org/docs/app/building-your-application/caching#opting-out-1
cache: 'no-store',
})
} catch (e) {
console.error(e)
Expand Down

0 comments on commit 10e9d38

Please sign in to comment.