fix(#834): Do not refresh on window focus for unprotected pages (#858)

YoshimiShima · web-flow · commit de1dca620a88 · 2024-08-11T19:58:20.000+02:00
diff --git a/src/runtime/utils/refreshHandler.ts b/src/runtime/utils/refreshHandler.ts
@@ -73,8 +73,8 @@ export class DefaultRefreshHandler implements RefreshHandler {
     // Listen for when the page is visible, if the user switches tabs
     // and makes our tab visible again, re-fetch the session, but only if
     // this feature is not disabled.
-    if (this.config?.enableOnWindowFocus && document.visibilityState === 'visible') {
-      this.auth?.refresh()
+    if (this.config?.enableOnWindowFocus && document.visibilityState === 'visible' && this.auth?.data.value) {
+      this.auth.refresh()
     }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -73,8 +73,8 @@ export class DefaultRefreshHandler implements RefreshHandler {`
`73`	`73`	`// Listen for when the page is visible, if the user switches tabs`
`74`	`74`	`// and makes our tab visible again, re-fetch the session, but only if`
`75`	`75`	`// this feature is not disabled.`
`76`		`- if (this.config?.enableOnWindowFocus && document.visibilityState === 'visible') {`
`77`		`- this.auth?.refresh()`
	`76`	`+ if (this.config?.enableOnWindowFocus && document.visibilityState === 'visible' && this.auth?.data.value) {`
	`77`	`+ this.auth.refresh()`
`78`	`78`	`}`
`79`	`79`	`}`
`80`	`80`	`}`