Skip to content

Accept JSON whitespace in attested token and identity fields - #36

Closed
Wondertan wants to merge 1 commit into
mainfrom
fix/json-field-whitespace
Closed

Wondertan wants to merge 1 commit into
mainfrom
fix/json-field-whitespace

Conversation

@Wondertan

@Wondertan Wondertan commented Sep 10, 2026

Copy link
Copy Markdown
Member

GitHub’s real identity response can contain whitespace after JSON colons. Accept that formatting in the shared JSON readers and token bearer framing while retaining duplicate detection across whitespace spellings and reveal seams, contiguous reads, and canonical integer validation.

Validation: all 318 ceremony contract tests pass, including pretty-printed identity, token framing, mixed-spelling duplicates, and disjoint-range rejection. Matching browser/Rust readers are required; existing bearer proof circuits and verification keys are unchanged.

Specification: libid #32. Standalone Bridge fix: libid-server-rs #11, based only on the token-exchange prerequisite (#8).

Preserve original transcript offsets and bearer-only commitments while accepting JSON whitespace around colons and before integer terminators.

Assisted-by: GPT-6
Signed-off-by: Wondertan <hlibwondertan@gmail.com>
@xgreenx

xgreenx commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Closed in favor of #37 for more general coverage.

Tests from here were used in the PR above

@xgreenx xgreenx closed this Sep 10, 2026
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