Skip to content

Fix frozen v1 session-bound token schema - #34

Merged
chen21019 merged 1 commit into
mainfrom
fix/frozen-token-session-v1
Sep 16, 2026
Merged

chen21019 merged 1 commit into
mainfrom
fix/frozen-token-session-v1

Conversation

@chen21019

Copy link
Copy Markdown

Root cause

The live token authorization overlay exposed clientSessionId, but the frozen v1 base, superadmin, and token schema snapshots did not. /v1/token therefore silently dropped the generation and created a legacy unbound token. The integrated Server runtime gate caught this when a mismatched delete revoked the newly-created token.

Changes

  • publish the fixed-format sensitive create-only field in all frozen v1 token schemas
  • add a deserialization regression test covering every frozen schema that exposes token
  • add a bounded, idempotent maintainer utility for snapshot regeneration
  • enforce the contract in source and packaged-artifact gates
  • bump the immutable Engine release to 0.183.308 and document v1/v2 consumer acceptance

Verification

  • FrozenTokenSessionSchemaTest: 1 passed
  • TokenResourceManagerTest: 5 passed
  • scripts/check-pasturestack-source: PASS with auth_token_frozen_v1_schema=base-superadmin-token
  • maintainer tool second run: patched=0 verified=3

Server publication remains blocked until both /v1/token and /v2-beta/token pass mismatched/matching/idempotent delete checks in the real candidate runtime.

@chen21019
chen21019 requested a review from a team as a code owner September 16, 2026 10:39
Comment thread scripts/PatchFrozenTokenSessionSchema.java Fixed
@chen21019
chen21019 force-pushed the fix/frozen-token-session-v1 branch from 98b3914 to 1839cf9 Compare September 16, 2026 10:48
@chen21019
chen21019 enabled auto-merge (squash) September 16, 2026 10:52
@chen21019
chen21019 disabled auto-merge September 16, 2026 10:52
@chen21019
chen21019 merged commit 299ecf9 into main Sep 16, 2026
5 checks passed
@chen21019
chen21019 deleted the fix/frozen-token-session-v1 branch September 16, 2026 10:52
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