Skip to content

Fix Zoho OIDC nonce compatibility - #1468

Merged
lovasoa merged 1 commit into
mainfrom
codex/zoho-oidc-nonce
Sep 17, 2026
Merged

lovasoa merged 1 commit into
mainfrom
codex/zoho-oidc-nonce

Conversation

@lovasoa

@lovasoa lovasoa commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Zoho Accounts rejects OIDC nonce values containing = before authentication. SQLPage used an Argon2 PHC hash as the nonce, and PHC hashes contain =.

Encode the PHC value as unpadded base64url for the authorization request, then decode it before the existing Argon2 verification. This preserves strict nonce validation while limiting the transmitted nonce to URL-safe characters.

Fixes #1466.

Validation:

  • cargo test --test mod oidc
  • cargo clippy --all-targets --all-features -- -D warnings

Draft pending a manual Zoho end-to-end sign-in and green GitHub Actions.

@lovasoa lovasoa mentioned this pull request Sep 17, 2026
@lovasoa
lovasoa force-pushed the codex/zoho-oidc-nonce branch from d319fd5 to 8a58814 Compare September 17, 2026 20:40
@lovasoa
lovasoa marked this pull request as ready for review September 17, 2026 20:48
@lovasoa
lovasoa merged commit f4bec6d into main Sep 17, 2026
51 checks passed
@lovasoa
lovasoa deleted the codex/zoho-oidc-nonce branch September 17, 2026 20:48
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.

OIDC ON ZOHO

1 participant