Skip to content

docs: document OIDC IdP email verification requirement - #209

Merged
boris-w merged 1 commit into
mainfrom
docs/oidc-idp-email-verification
Aug 10, 2026
Merged

docs: document OIDC IdP email verification requirement#209
boris-w merged 1 commit into
mainfrom
docs/oidc-idp-email-verification

Conversation

@boris-w

@boris-w boris-w commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Teable links OIDC logins to existing accounts by matching email, so the connected identity provider must guarantee email ownership is verified. This PR documents that deployment prerequisite instead of leaving it implicit:

  • Add an Identity Provider Security Requirements section to en/deploy/oidc.mdx and zh/deploy/oidc.mdx:
    • Explains the auto-link-by-email behavior and why the IdP must verify emails.
    • <Warning> callout describing the account-takeover path when an IdP allows arbitrary unverified emails, especially with local password login enabled on the same instance.
    • Recommends trusted enterprise IdPs with email verification enabled, and PASSWORD_LOGIN_DISABLED=true for SSO-only instances.
  • Add a corresponding item to the Important Notes checklist on both pages.

This records the documented deployment requirement / explicit risk-acceptance stance for the related draft GHSA (OIDC email-binding account linking).

🤖 Generated with Claude Code

Teable links OIDC logins to existing accounts by email, so the
connected IdP must verify email ownership. Add an Identity Provider
Security Requirements section (en/zh) spelling out the account-takeover
risk of unverified-email IdPs and recommending trusted enterprise IdPs
and PASSWORD_LOGIN_DISABLED for SSO-only instances.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@CLAassistant

CLAassistant commented Aug 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@boris-w
boris-w merged commit 3d2eb18 into main Aug 10, 2026
2 checks passed
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