Skip to content

Fix admin Google sign-in after original proof issuance - #134

Merged
rafiki270 merged 1 commit into
mainfrom
codex/admin-google-signin-fix
Oct 25, 2026
Merged

rafiki270 merged 1 commit into
mainfrom
codex/admin-google-signin-fix

Conversation

@rafiki270

@rafiki270 rafiki270 commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Google sign-in completed successfully, but /internal/admin/session then returned 401. Admin cookie storage shortened the refresh family after its immutable authentication proof had been issued, so live proof validation rejected the session.

Establish the admin family deadline before issuing either original or inherited proof, using JWT whole-second precision. Cookie storage now only writes the private handle. Debug redemption keeps the source ancestry and deadline; ordinary product refresh lifetimes and proof validation remain unchanged. No UI source or database schema changes.

Validation

  • Reproduced the failure through the production Google browser journey and confirmed INVALID_SESSION_ASSURANCE in Cloud Run logs.
  • Added an HTTP integration regression covering Google callback → PKCE exchange → admin session → debug redemption → source logout and descendant revocation. It failed with 401 before the fix and passes afterward.
  • On Windows Maxis: 16 tests across six focused files passed, including existing admin/debug and product Google flows plus timestamp-boundary tests.
  • API typecheck, build, changed-file ESLint and git diff --check passed.
  • All PR CI checks passed, including the full DB-less and PostgreSQL API suites and dependency audit.
  • Production deployment passed. Cloud Run revision uoa-auth-00349-fxk serves 100% of traffic. On 2026-10-25 at 17:43 UTC, the real Google browser journey reached /admin/dashboard, and the session remained valid after a reload.
  • Native apps were not built because their source and build inputs are unchanged.

Copilot AI balanced review requested due to automatic review settings October 25, 2026 17:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@rafiki270
rafiki270 merged commit f744cc1 into main Oct 25, 2026
2 checks passed
@rafiki270
rafiki270 deleted the codex/admin-google-signin-fix branch October 25, 2026 17:39
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.

2 participants