| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130 |
- """Read the single OIDC provider defined by BAMBUDDY_OIDC_* env vars (#2593).
- A declarative deployment (compose, Helm, GitOps) has no way to click through
- the settings UI, so one provider can be configured entirely from the
- environment. This module only reads and defaults; validity is decided by the
- same OIDCProviderCreate schema the API uses, so env config cannot bypass a
- check the UI enforces.
- """
- from __future__ import annotations
- import logging
- import os
- from sqlalchemy import select, update
- from sqlalchemy.ext.asyncio import AsyncSession
- logger = logging.getLogger(__name__)
- # All four or nothing: a provider missing its secret would be written to the
- # database and then fail at authorize time, long after the operator could
- # connect the failure to a typo in their compose file.
- _REQUIRED = (
- "BAMBUDDY_OIDC_NAME",
- "BAMBUDDY_OIDC_ISSUER_URL",
- "BAMBUDDY_OIDC_CLIENT_ID",
- "BAMBUDDY_OIDC_CLIENT_SECRET",
- )
- _TRUTHY = {"true", "1", "yes"}
- def _env_bool(key: str, default: bool) -> bool:
- value = os.environ.get(key)
- return default if value is None else value.strip().lower() in _TRUTHY
- def read_env_oidc_config() -> dict | None:
- """The provider's fields from the environment, or None if it isn't configured.
- An empty required var counts as unset -- `BAMBUDDY_OIDC_CLIENT_SECRET=` in
- a compose file is a forgotten value, not an intentional empty secret.
- """
- if not all(os.environ.get(key) for key in _REQUIRED):
- return None
- return {
- "name": os.environ["BAMBUDDY_OIDC_NAME"],
- "issuer_url": os.environ["BAMBUDDY_OIDC_ISSUER_URL"],
- "client_id": os.environ["BAMBUDDY_OIDC_CLIENT_ID"],
- "client_secret": os.environ["BAMBUDDY_OIDC_CLIENT_SECRET"],
- "scopes": os.environ.get("BAMBUDDY_OIDC_SCOPES", "openid email profile"),
- "is_enabled": _env_bool("BAMBUDDY_OIDC_ENABLED", True),
- "auto_create_users": _env_bool("BAMBUDDY_OIDC_AUTO_CREATE_USERS", False),
- "auto_link_existing_accounts": _env_bool("BAMBUDDY_OIDC_AUTO_LINK_EXISTING", False),
- "email_claim": os.environ.get("BAMBUDDY_OIDC_EMAIL_CLAIM", "email"),
- "require_email_verified": _env_bool("BAMBUDDY_OIDC_REQUIRE_EMAIL_VERIFIED", True),
- "icon_url": os.environ.get("BAMBUDDY_OIDC_ICON_URL"),
- "is_autologin": _env_bool("BAMBUDDY_OIDC_AUTOLOGIN", False),
- }
- # Everything the schema validates and the model stores, except client_secret --
- # that one goes through the property so it is encrypted at rest.
- _APPLIED_FIELDS = (
- "name",
- "issuer_url",
- "client_id",
- "scopes",
- "is_enabled",
- "auto_create_users",
- "auto_link_existing_accounts",
- "email_claim",
- "require_email_verified",
- "icon_url",
- "is_autologin",
- )
- async def apply_env_oidc_provider(db: AsyncSession) -> None:
- """Upsert the env-managed provider, or disable it when the config is gone.
- Never raises: this runs during startup, and a typo in one variable must not
- stop the app from booting. A rejected config is logged and skipped.
- """
- # Imported here rather than at module scope: app.core is imported by the
- # models themselves, so a top-level import would be a cycle.
- from backend.app.models.oidc_provider import OIDCProvider
- from backend.app.schemas.auth import OIDCProviderCreate
- config = read_env_oidc_config()
- existing = (
- await db.execute(select(OIDCProvider).where(OIDCProvider.is_env_managed.is_(True)))
- ).scalar_one_or_none()
- if config is None:
- # Disabled, never deleted: user_oidc_links.provider_id is FK ON DELETE
- # CASCADE, so removing the row would unlink every bound account and the
- # links would not come back when the variables do.
- if existing is not None and existing.is_enabled:
- existing.is_enabled = False
- await db.commit()
- logger.info("BAMBUDDY_OIDC_* is unset -- env-managed provider disabled.")
- return
- try:
- # The same schema the API uses, so env config cannot reach a state the
- # UI would have refused (notably the SEC-1 auto-link check).
- validated = OIDCProviderCreate(**config)
- except Exception as exc: # noqa: BLE001 -- any rejection must be survivable
- logger.error("BAMBUDDY_OIDC_* config rejected, provider not applied: %s", exc)
- return
- if existing is None:
- existing = OIDCProvider(is_env_managed=True)
- db.add(existing)
- for field in _APPLIED_FIELDS:
- setattr(existing, field, getattr(validated, field))
- existing.client_secret = validated.client_secret
- existing.is_env_managed = True
- await db.flush() # the id is needed by the autologin sweep below
- if existing.is_autologin:
- await db.execute(
- update(OIDCProvider)
- .where(OIDCProvider.id != existing.id, OIDCProvider.is_autologin.is_(True))
- .values(is_autologin=False)
- )
- await db.commit()
- logger.info("Env-managed OIDC provider %r applied.", existing.name)
|