fix(node-player): load cover art via axios with Bearer token
Cover images were loaded via <img src> which doesn't include the Authorization header, resulting in 401 from the Rust API. Now covers are fetched through axios as blobs and displayed via object URLs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,6 +1,6 @@
|
|||||||
import { useEffect, useRef, useState, type MouseEvent as ReactMouseEvent } from 'react'
|
import { useEffect, useRef, useState, type MouseEvent as ReactMouseEvent } from 'react'
|
||||||
import './furumi-player.css'
|
import './furumi-player.css'
|
||||||
import { API_ROOT, searchTracks, preloadStream } from './furumiApi'
|
import { searchTracks, preloadStream, fetchCoverBlob } from './furumiApi'
|
||||||
import { store, useAppDispatch, useAppSelector } from './store'
|
import { store, useAppDispatch, useAppSelector } from './store'
|
||||||
import { fetchArtists } from './store/slices/artistsSlice'
|
import { fetchArtists } from './store/slices/artistsSlice'
|
||||||
import { fetchArtistAlbums } from './store/slices/albumsSlice'
|
import { fetchArtistAlbums } from './store/slices/albumsSlice'
|
||||||
@@ -80,15 +80,16 @@ export function FurumiPlayer() {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
document.title = `${nowPlayingTrack.title} — Furumi`
|
document.title = `${nowPlayingTrack.title} — Furumi`
|
||||||
const coverUrl = `${API_ROOT}/tracks/${nowPlayingTrack.slug}/cover`
|
|
||||||
if ('mediaSession' in navigator) {
|
if ('mediaSession' in navigator) {
|
||||||
try {
|
try {
|
||||||
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
|
const meta = new window.MediaMetadata({
|
||||||
navigator.mediaSession.metadata = new window.MediaMetadata({
|
|
||||||
title: nowPlayingTrack.title,
|
title: nowPlayingTrack.title,
|
||||||
artist: nowPlayingTrack.artist || '',
|
artist: nowPlayingTrack.artist || '',
|
||||||
album: '',
|
album: '',
|
||||||
artwork: [{ src: coverUrl, sizes: '512x512' }],
|
})
|
||||||
|
navigator.mediaSession.metadata = meta
|
||||||
|
fetchCoverBlob(nowPlayingTrack.slug).then((url) => {
|
||||||
|
if (url) meta.artwork = [{ src: url, sizes: '512x512' }]
|
||||||
})
|
})
|
||||||
} catch {
|
} catch {
|
||||||
// ignore
|
// ignore
|
||||||
|
|||||||
@@ -1,15 +1,12 @@
|
|||||||
import { useEffect, useState } from 'react'
|
import { useState } from 'react'
|
||||||
import { API_ROOT } from '../furumiApi'
|
|
||||||
import type { QueueItem } from './QueueList'
|
import type { QueueItem } from './QueueList'
|
||||||
|
import { useCoverUrl } from '../hooks/useCoverUrl'
|
||||||
|
|
||||||
function Cover({ src }: { src: string }) {
|
function Cover({ slug }: { slug: string }) {
|
||||||
const [errored, setErrored] = useState(false)
|
const [errored, setErrored] = useState(false)
|
||||||
|
const src = useCoverUrl(slug)
|
||||||
|
|
||||||
useEffect(() => {
|
if (!src || errored) return <>🎵</>
|
||||||
setErrored(false)
|
|
||||||
}, [src])
|
|
||||||
|
|
||||||
if (errored) return <>🎵</>
|
|
||||||
return <img src={src} alt="" onError={() => setErrored(true)} />
|
return <img src={src} alt="" onError={() => setErrored(true)} />
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -32,12 +29,10 @@ export function NowPlaying({ track }: { track: QueueItem | null }) {
|
|||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
const coverUrl = `${API_ROOT}/tracks/${track.slug}/cover`
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="np-info">
|
<div className="np-info">
|
||||||
<div className="np-cover" id="npCover">
|
<div className="np-cover" id="npCover">
|
||||||
<Cover src={coverUrl} />
|
<Cover slug={track.slug} />
|
||||||
</div>
|
</div>
|
||||||
<div className="np-text">
|
<div className="np-text">
|
||||||
<div className="np-title" id="npTitle">
|
<div className="np-title" id="npTitle">
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
import { useEffect, useRef, useState } from 'react'
|
import { useRef, useState } from 'react'
|
||||||
import { API_ROOT } from '../furumiApi'
|
import { useCoverUrl } from '../hooks/useCoverUrl'
|
||||||
|
|
||||||
export type QueueItem = {
|
export type QueueItem = {
|
||||||
slug: string
|
slug: string
|
||||||
@@ -32,13 +32,11 @@ function fmt(secs: number) {
|
|||||||
return `${m}:${pad(s % 60)}`
|
return `${m}:${pad(s % 60)}`
|
||||||
}
|
}
|
||||||
|
|
||||||
function Cover({ src }: { src: string }) {
|
function Cover({ slug }: { slug: string }) {
|
||||||
const [errored, setErrored] = useState(false)
|
const [errored, setErrored] = useState(false)
|
||||||
useEffect(() => {
|
const src = useCoverUrl(slug)
|
||||||
setErrored(false)
|
|
||||||
}, [src])
|
|
||||||
|
|
||||||
if (errored) return <>🎵</>
|
if (!src || errored) return <>🎵</>
|
||||||
return <img src={src} alt="" onError={() => setErrored(true)} />
|
return <img src={src} alt="" onError={() => setErrored(true)} />
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -77,7 +75,7 @@ export function QueueList({
|
|||||||
if (!t) return null
|
if (!t) return null
|
||||||
|
|
||||||
const isPlaying = origIdx === playingOrigIdx
|
const isPlaying = origIdx === playingOrigIdx
|
||||||
const coverSrc = t.album_slug ? `${API_ROOT}/tracks/${t.slug}/cover` : ''
|
const hasAlbum = !!t.album_slug
|
||||||
const dur = t.duration ? fmt(t.duration) : ''
|
const dur = t.duration ? fmt(t.duration) : ''
|
||||||
const isDragging = draggingPos === pos
|
const isDragging = draggingPos === pos
|
||||||
const isDragOver = dragOverPos === pos
|
const isDragOver = dragOverPos === pos
|
||||||
@@ -118,7 +116,7 @@ export function QueueList({
|
|||||||
>
|
>
|
||||||
<span className="qi-index">{isPlaying ? '' : pos + 1}</span>
|
<span className="qi-index">{isPlaying ? '' : pos + 1}</span>
|
||||||
<div className="qi-cover">
|
<div className="qi-cover">
|
||||||
{coverSrc ? <Cover src={coverSrc} /> : <>🎵</>}
|
{hasAlbum ? <Cover slug={t.slug} /> : <>🎵</>}
|
||||||
</div>
|
</div>
|
||||||
<div className="qi-info">
|
<div className="qi-info">
|
||||||
<div className="qi-title">{t.title}</div>
|
<div className="qi-title">{t.title}</div>
|
||||||
|
|||||||
@@ -52,3 +52,9 @@ export async function preloadStream(trackSlug: string) {
|
|||||||
return await furumiApi.get(`/stream/${trackSlug}`, { responseType: 'blob' }).catch(() => null)
|
return await furumiApi.get(`/stream/${trackSlug}`, { responseType: 'blob' }).catch(() => null)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
export async function fetchCoverBlob(trackSlug: string): Promise<string | null> {
|
||||||
|
const res = await furumiApi.get(`/tracks/${trackSlug}/cover`, { responseType: 'blob' }).catch(() => null)
|
||||||
|
if (!res?.data) return null
|
||||||
|
return URL.createObjectURL(res.data)
|
||||||
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,28 @@
|
|||||||
|
import { useEffect, useState } from 'react'
|
||||||
|
import { fetchCoverBlob } from '../furumiApi'
|
||||||
|
|
||||||
|
export function useCoverUrl(trackSlug: string | undefined): string | null {
|
||||||
|
const [url, setUrl] = useState<string | null>(null)
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (!trackSlug) {
|
||||||
|
setUrl(null)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
let revoke: string | null = null
|
||||||
|
|
||||||
|
fetchCoverBlob(trackSlug).then((blobUrl) => {
|
||||||
|
if (blobUrl) {
|
||||||
|
revoke = blobUrl
|
||||||
|
setUrl(blobUrl)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
return () => {
|
||||||
|
if (revoke) URL.revokeObjectURL(revoke)
|
||||||
|
}
|
||||||
|
}, [trackSlug])
|
||||||
|
|
||||||
|
return url
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user