Files
RepoDructor/stores/music.ts
josedario87 b46d15145f
All checks were successful
build-and-deploy / build-and-deploy (push) Successful in 28s
fix: evitar error CORS al expirar sesión de Authentik
Cuando la sesión de Authentik expira, los fetch() a /api/music recibían
un redirect (302) a la página de login. Por defecto, fetch intenta seguir
el redirect pero falla por CORS porque Authentik no tiene el header
Access-Control-Allow-Origin.

La solución es usar redirect: 'error' en todos los fetch() a endpoints
protegidos, lo que convierte los redirects en errores que podemos capturar
y manejar apropiadamente. Esto coincide con la estrategia que ya usa
useAuth.ts.

Cambios:
- stores/music.ts: Agregar redirect: 'error' a fetchTracks() y cacheByName()
- pages/index.vue: Agregar redirect: 'error' a playTrack()
- Mejorar detección de errores de autenticación para incluir 'Failed to fetch'
  y errores de tipo TypeError relacionados con redirects
2025-10-17 03:40:12 -06:00

171 lines
5.0 KiB
TypeScript

import { defineStore } from 'pinia'
import { openDB, type IDBPDatabase } from 'idb'
type Track = {
name: string
duration?: number
}
type DBTrack = {
name: string
blob: Blob
type?: string
duration?: number
}
let dbPromise: Promise<IDBPDatabase> | null = null
async function getDB() {
if (!dbPromise) {
dbPromise = openDB('repodructor-music', 1, {
upgrade(db) {
if (!db.objectStoreNames.contains('tracks')) {
const store = db.createObjectStore('tracks', { keyPath: 'name' })
store.createIndex('name', 'name', { unique: true })
}
}
})
}
return dbPromise
}
export const useMusicStore = defineStore('music', {
state: () => ({
tracks: [] as Track[],
cachedNames: new Set<string>(),
loading: false,
error: null as string | null
}),
getters: {
isCached: (state) => (name: string) => state.cachedNames.has(name)
},
actions: {
async fetchTracks() {
this.loading = true
this.error = null
try {
const response = await $fetch<{ tracks: Track[] }>('/api/music', {
credentials: 'include',
redirect: 'error' // No seguir redirects de Authentik - convertir en error
})
this.tracks = response.tracks || []
} catch (e: any) {
const errorMsg = e?.message || 'Failed to load tracks'
this.error = errorMsg
// Check if it's an auth error (including redirect attempts from Authentik)
if (e?.statusCode === 401 || e?.statusCode === 403 ||
errorMsg.includes('401') || errorMsg.includes('403') ||
errorMsg.includes('Unauthorized') ||
errorMsg.includes('Failed to fetch') ||
(e?.cause?.name === 'TypeError' && e?.cause?.message?.includes('redirect'))) {
console.warn('[Music Store] Authentication error detected:', errorMsg)
// The useAuth composable will be notified via watch in components
}
} finally {
this.loading = false
}
},
async loadCachedNames() {
try {
const db = await getDB()
const tx = db.transaction('tracks', 'readonly')
const store = tx.objectStore('tracks')
const all = await store.getAllKeys()
this.cachedNames = new Set((all as string[]) || [])
} catch (e) {
// ignore
}
},
async loadCachedTracks(): Promise<Track[]> {
try {
const db = await getDB()
const tx = db.transaction('tracks', 'readonly')
const store = tx.objectStore('tracks')
const all = (await store.getAll()) as DBTrack[]
const names = all.map(t => t.name)
this.cachedNames = new Set(names)
const list: Track[] = all.map(t => ({ name: t.name, duration: t.duration }))
return list
} catch (e) {
return []
}
},
async getCachedBlob(name: string): Promise<Blob | null> {
try {
const db = await getDB()
const entry = (await db.get('tracks', name)) as DBTrack | undefined
if (entry && entry.blob) {
this.cachedNames.add(name)
return entry.blob
}
} catch (e) {
// ignore
}
return null
},
async saveTrackBlob(name: string, blob: Blob, duration?: number) {
try {
const db = await getDB()
const payload: DBTrack = { name, blob, type: blob.type, duration }
await db.put('tracks', payload)
this.cachedNames.add(name)
} catch (e) {
// ignore
}
},
async deleteCachedTrack(name: string) {
try {
const db = await getDB()
await db.delete('tracks', name)
this.cachedNames.delete(name)
} catch (e) {
// ignore
}
},
// Fetch from API and store in IndexedDB
async cacheByName(name: string, duration?: number): Promise<boolean> {
try {
const encodedName = encodeURIComponent(name)
const response = await fetch(`/api/music/${encodedName}`, {
credentials: 'include',
redirect: 'error' // No seguir redirects de Authentik - convertir en error
})
if (!response.ok) {
const errorMsg = `HTTP ${response.status}`
// Detect authentication errors
if (response.status === 401 || response.status === 403) {
console.warn('[Music Store] Cache failed - authentication error:', response.status)
this.error = `${errorMsg}: Unauthorized - Please log in`
}
throw new Error(errorMsg)
}
const blob = await response.blob()
await this.saveTrackBlob(name, blob, duration)
return true
} catch (e: any) {
console.error('[Music Store] Cache failed:', e)
// Propagate auth errors (including redirect attempts from Authentik)
if (e?.message?.includes('401') ||
e?.message?.includes('403') ||
e?.message?.includes('Failed to fetch') ||
(e?.name === 'TypeError' && e?.message?.includes('redirect'))) {
this.error = e.message || 'Authentication error'
}
return false
}
}
}
})