Revert "fix: implementar manejo correcto de modo offline"
All checks were successful
build-and-deploy / build-and-deploy (push) Successful in 25s
All checks were successful
build-and-deploy / build-and-deploy (push) Successful in 25s
This reverts commit eedff715d4.
This commit is contained in:
@@ -5,7 +5,6 @@ const authChecked = ref(true) // Iniciamos como chequeado
|
||||
const isAuthenticated = ref(true) // Asumimos autenticado al inicio (si la página cargó, es porque pasamos Authentik)
|
||||
const isCheckingAuth = ref(false)
|
||||
const lastCheckTime = ref(0)
|
||||
const isOffline = ref(false) // Estado de conexión
|
||||
|
||||
// Estado de información del usuario
|
||||
const userInfo = ref(null)
|
||||
@@ -15,34 +14,7 @@ let visibilityChangeListener: (() => void) | null = null
|
||||
let focusListener: (() => void) | null = null
|
||||
|
||||
export const useAuth = () => {
|
||||
// Detectar cambios en el estado de conexión
|
||||
if (import.meta.client) {
|
||||
const updateOnlineStatus = () => {
|
||||
const wasOffline = isOffline.value
|
||||
isOffline.value = !navigator.onLine
|
||||
|
||||
if (wasOffline && !isOffline.value) {
|
||||
console.log('[Auth] Connection restored - back online')
|
||||
} else if (!wasOffline && isOffline.value) {
|
||||
console.log('[Auth] Connection lost - now offline')
|
||||
}
|
||||
}
|
||||
|
||||
// Inicializar estado
|
||||
updateOnlineStatus()
|
||||
|
||||
// Escuchar eventos de conexión
|
||||
window.addEventListener('online', updateOnlineStatus)
|
||||
window.addEventListener('offline', updateOnlineStatus)
|
||||
}
|
||||
|
||||
const checkAuth = async (force = false) => {
|
||||
// Si estamos offline, no hacer requests
|
||||
if (isOffline.value && !force) {
|
||||
console.log('[Auth] Skipping check - offline')
|
||||
return
|
||||
}
|
||||
|
||||
// Si ya sabemos que NO estamos autenticados, no hacer más requests
|
||||
// (evita loops innecesarios)
|
||||
if (!isAuthenticated.value && !force) {
|
||||
@@ -82,22 +54,16 @@ export const useAuth = () => {
|
||||
} catch (error: any) {
|
||||
console.warn('[Auth] Check failed:', error.message || error)
|
||||
|
||||
// Verificar si estamos offline AHORA (pudo cambiar durante el request)
|
||||
if (!navigator.onLine) {
|
||||
console.log('[Auth] Offline detected - maintaining current auth state')
|
||||
isOffline.value = true
|
||||
// NO cambiar isAuthenticated cuando estamos offline
|
||||
// Mantener el último estado conocido
|
||||
} else {
|
||||
// Online pero el fetch falló = problema de autenticación
|
||||
const wasAuthenticated = isAuthenticated.value
|
||||
console.log('[Auth] Online but fetch failed - marking as unauthenticated')
|
||||
isAuthenticated.value = false
|
||||
authChecked.value = true
|
||||
const wasAuthenticated = isAuthenticated.value
|
||||
|
||||
if (wasAuthenticated) {
|
||||
console.log('[Auth] Status changed: authenticated → unauthenticated')
|
||||
}
|
||||
// Si el fetch falla completamente (CORS, redirect, etc), asumir no autenticado
|
||||
// Esto pasa cuando Authentik redirige antes de que el endpoint pueda responder
|
||||
console.log('[Auth] Fetch failed - marking as unauthenticated')
|
||||
isAuthenticated.value = false
|
||||
authChecked.value = true
|
||||
|
||||
if (wasAuthenticated) {
|
||||
console.log('[Auth] Status changed: authenticated → unauthenticated')
|
||||
}
|
||||
|
||||
lastCheckTime.value = now
|
||||
@@ -163,14 +129,6 @@ export const useAuth = () => {
|
||||
|
||||
const markUnauthenticated = async () => {
|
||||
// Helper para marcar como no autenticado (útil cuando detectamos 401/403)
|
||||
// IMPORTANTE: NO llamar esto si estamos offline, solo si hay un error real de auth
|
||||
|
||||
// Si estamos offline, NO marcar como no autenticado
|
||||
if (isOffline.value || !navigator.onLine) {
|
||||
console.log('[Auth] Skipping markUnauthenticated - offline mode')
|
||||
return
|
||||
}
|
||||
|
||||
console.log('[Auth] Marking as unauthenticated')
|
||||
isAuthenticated.value = false
|
||||
authChecked.value = true
|
||||
@@ -240,8 +198,6 @@ export const useAuth = () => {
|
||||
}
|
||||
|
||||
const authStatus = computed(() => {
|
||||
// Si estamos offline, mostrar estado offline
|
||||
if (isOffline.value) return 'offline'
|
||||
if (!authChecked.value) return 'unknown'
|
||||
return isAuthenticated.value ? 'authenticated' : 'unauthenticated'
|
||||
})
|
||||
@@ -250,7 +206,6 @@ export const useAuth = () => {
|
||||
isAuthenticated,
|
||||
authChecked,
|
||||
isCheckingAuth,
|
||||
isOffline,
|
||||
authStatus,
|
||||
userInfo,
|
||||
checkAuth,
|
||||
|
||||
Reference in New Issue
Block a user