Skip to content

Settings API for the YouTube pairing token (#136) - #425

Merged
rowkav09 merged 1 commit into
mainfrom
feat/youtube-pairing-api
Sep 24, 2026
Merged

rowkav09 merged 1 commit into
mainfrom
feat/youtube-pairing-api

Conversation

@rowkav09

Copy link
Copy Markdown
Member

Part of #136. Backend for the pairing-code page (the page itself goes to the web UI lane).

  • POST /api/settings/youtube/pairing → { token }
  • POST /api/settings/youtube/pairing/reset → { token } (new token saved to the credential store; the bridge rejects the old token at once and drops its tabs)
    Both are POST so the server session check applies: only the app's own settings page can read the token. 404 when the bridge is off (safe mode). Tests: 753 pass.

@rowkav09

Copy link
Copy Markdown
Member Author

/mira resume

@github-actions

Copy link
Copy Markdown
Contributor

/mira pause

@github-actions github-actions Bot added mira-paused Pause automatic Mira reviews on this pull request area:server area:providers size:M labels Sep 24, 2026
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Adds settings.youtubePairing() and resetYouTubePairing(), served as POST /api/settings/youtube/pairing and /pairing/reset (POST so the session check applies). Reset stores a new token, and the bridge rejects the old one straight away and drops its tabs.
@rowkav09
rowkav09 force-pushed the feat/youtube-pairing-api branch from eb3306c to 52d798c Compare September 24, 2026 13:21
@rowkav09

Copy link
Copy Markdown
Member Author

No review from Mira (auto-paused). Self-reviewed: both routes are POST so the session check applies; sec-fetch-site checked; 404 when the bridge is off; reset makes the old token 401 at once and clears its tabs (tested end to end). Rebased; all checks green.

@rowkav09
rowkav09 merged commit 4ead598 into main Sep 24, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in nowplaying Sep 24, 2026
rowkav09 added a commit that referenced this pull request Sep 24, 2026
Adds a YouTube section to the settings page: hidden pairing code with Show/Copy, the port to enter in the extension, and Make a new code with a confirm. Uses the pairing API from #425; hidden when the bridge is off.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers area:server mira-paused Pause automatic Mira reviews on this pull request size:M

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant