Skip to content

🛰️🔑 ↝ [SSC-35]: Serve signed-in game reads through the edge Worker on staging - #260

Merged
Gizmotronn merged 10 commits into
mainfrom
ssc-edge-api-staging
Sep 30, 2026
Merged

Gizmotronn merged 10 commits into
mainfrom
ssc-edge-api-staging

Conversation

@Gizmotronn

Copy link
Copy Markdown
Member

Adds the staging-only NEXT_PUBLIC_EDGE_API flag, waits for Clerk before routing reads to the Worker, checks the Clerk azp claim only when present (foreign azp still 403), and chains the staging playtest after the staging deploy so it runs on push.

Verified: deploy + playtest green on 918155d (Worker accepts the signed-in Clerk token; garden flow completes; endpoint closed afterwards).

🤖 Generated with Claude Code

Gizmotronn and others added 10 commits September 30, 2026 16:45
Production builds stay unchanged; only the staging deploy sets NEXT_PUBLIC_EDGE_API.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…from the edge Worker

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…the edge Worker

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…rough the edge Worker

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…et propagates

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…en on staging

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…he staging session

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…minted sessions reach the edge Worker

A token naming a foreign origin is still rejected (403); tokens without azp, such as sessions minted from a sign-in ticket, are accepted once issuer and signature verify, matching Clerk's own verifier.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@Gizmotronn
Gizmotronn merged commit 21f0a1d into main Sep 30, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
production — 918155d3 Deployed Sep 30, 2026 by Gizmotronn via playtest / playtest #16
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