Skip to content

fix(auth): honor JWT media playback contract - #877

Draft
Serph91P wants to merge 10 commits into
developfrom
fix/828-jwt-library-session-contract
Draft

Serph91P wants to merge 10 commits into
developfrom
fix/828-jwt-library-session-contract

Conversation

@Serph91P

@Serph91P Serph91P commented Sep 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • preserve the reviewed JWT/legacy library and keepalive contract
  • let interactive JWT and legacy-session identities reach authenticated thumbnails, streams, chapters, share-token generation, and filename streaming without accepting API-key-only access
  • normalize VTT chapter responses to numeric id/title/start/end values used by the player service contract
  • load the active VideoPlayer auto-chapter path through videoApi.getChapters() and retain legacy prop/WebVTT handling
  • include the verified dependency-only fast-uri 3.1.8 and PyJWT/oauthlib audit remediations from PR fix(frontend): add system back navigation #876

Verification

  • uv run --locked --group test pytest -q (605 passed, 6 skipped)
  • focused JWT/media, range, and dependency compatibility contracts (28 passed)
  • frontend unit suite (204 passed, 4 skipped), including the real VideoPlayer chapter adapter/service test
  • production build, build-output, precache-integrity, and output-budget checks
  • npx playwright test tests/e2e/ux06-player.spec.ts (18 passed, 2 skipped)

Historical review 273 approved only the earlier narrow library/keepalive commit 7c7400c; it does not approve this expanded media-auth and active-player head. This PR remains a draft pending exact-head CI and a new same-card review. No merge, release, deployment, production, or public-instance action was performed.

@Serph91P
Serph91P marked this pull request as ready for review September 29, 2026 10:47
@Serph91P
Serph91P marked this pull request as draft September 30, 2026 00:40
@Serph91P Serph91P changed the title fix(auth): honor JWT contract for library and keepalive fix(auth): honor JWT media playback contract Sep 30, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant