peter/siftlode
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge bug/youtube-error-status: YouTube failures as 422 not 502 (0.11.2)

Browse source
This commit is contained in:
npeter83 2026-06-19 04:27:22 +02:00
commit 93c4c55a75
4 changed files with 14 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

2
VERSION
View file

@ -1 +1 @@
0.11.1 0.11.2

2
backend/app/routes/feed.py
View file

@ -552,7 +552,7 @@ def get_video_detail(
with quota.attribute(user.id, "video_lookup"), YouTubeClient(db, user) as yt: with quota.attribute(user.id, "video_lookup"), YouTubeClient(db, user) as yt:
items = yt.get_videos([video_id]) items = yt.get_videos([video_id])
except YouTubeError as exc: except YouTubeError as exc:
raise HTTPException(status_code=502, detail=f"YouTube lookup failed: {exc}") raise HTTPException(status_code=422, detail=f"YouTube lookup failed: {exc}")
if not items: if not items:
raise HTTPException(status_code=404, detail="Unknown video") raise HTTPException(status_code=404, detail="Unknown video")

8
backend/app/routes/playlists.py
View file

@ -270,7 +270,7 @@ def revert_youtube(
return repull_playlist(db, user, pl) return repull_playlist(db, user, pl)
except YouTubeError: except YouTubeError:
db.rollback() db.rollback()
raise HTTPException(status_code=502, detail="Couldn't refresh from YouTube.") raise HTTPException(status_code=422, detail="Couldn't refresh from YouTube.")
def _pushable(db: Session, user: User, playlist_id: int) -> Playlist: def _pushable(db: Session, user: User, playlist_id: int) -> Playlist:
@ -301,7 +301,7 @@ def push_plan(
with quota.attribute(user.id, "playlist_push"): with quota.attribute(user.id, "playlist_push"):
plan = plan_push(db, user, pl) plan = plan_push(db, user, pl)
except YouTubeError: except YouTubeError:
raise HTTPException(status_code=502, detail="Couldn't read the playlist on YouTube.") raise HTTPException(status_code=422, detail="Couldn't read the playlist on YouTube.")
plan["remaining_today"] = quota.remaining_today(db) plan["remaining_today"] = quota.remaining_today(db)
plan["affordable"] = plan["units_estimate"] <= plan["remaining_today"] plan["affordable"] = plan["units_estimate"] <= plan["remaining_today"]
return plan return plan
@ -332,7 +332,7 @@ def push(
return push_playlist(db, user, pl) return push_playlist(db, user, pl)
except YouTubeError: except YouTubeError:
db.rollback() db.rollback()
raise HTTPException(status_code=502, detail="YouTube playlist sync failed.") raise HTTPException(status_code=422, detail="YouTube playlist sync failed.")
@router.get("/{playlist_id}") @router.get("/{playlist_id}")
@ -422,7 +422,7 @@ def delete_playlist(
with YouTubeClient(db, user) as yt: with YouTubeClient(db, user) as yt:
yt.delete_playlist(pl.yt_playlist_id) yt.delete_playlist(pl.yt_playlist_id)
except YouTubeError: except YouTubeError:
raise HTTPException(status_code=502, detail="YouTube delete failed.") raise HTTPException(status_code=422, detail="YouTube delete failed.")
db.delete(pl) # items cascade db.delete(pl) # items cascade
db.commit() db.commit()
return {"deleted": playlist_id} return {"deleted": playlist_id}

8
frontend/src/lib/releaseNotes.ts
View file

@ -14,6 +14,14 @@ export interface ReleaseEntry {
} }
export const RELEASE_NOTES: ReleaseEntry[] = [ export const RELEASE_NOTES: ReleaseEntry[] = [
{
version: "0.11.2",
date: "2026-06-19",
summary: "Clearer messages when a YouTube action can't be completed.",
fixes: [
"When a YouTube action can't go through — syncing or pushing a playlist, removing a playlist on YouTube, or looking up a video — you now get a clear explanation of why, instead of a misleading “connection lost” notice.",
],
},
{ {
version: "0.11.1", version: "0.11.1",
date: "2026-06-19", date: "2026-06-19",