Skip to content

Add per-provider registration control#2

Open
johnmaguire wants to merge 1 commit intomoghtech:mainfrom
johnmaguire:granular-registration-control
Open

Add per-provider registration control#2
johnmaguire wants to merge 1 commit intomoghtech:mainfrom
johnmaguire:granular-registration-control

Conversation

@johnmaguire
Copy link
Copy Markdown
Contributor

Add granular registration methods to AuthImpl:

  • local_registration_disabled()
  • oidc_registration_disabled()
  • github_registration_disabled()
  • google_registration_disabled()

Each defaults to the existing registration_disabled() for full backwards compatibility. Signup paths now use the provider-specific methods, allowing apps to disable local signups while permitting OIDC (or any other combination).

Includes unit tests for delegation behavior and override logic.

Ref: moghtech/komodo#1087

Add granular registration methods to AuthImpl:
- local_registration_disabled()
- oidc_registration_disabled()
- github_registration_disabled()
- google_registration_disabled()

Each defaults to the existing registration_disabled() for full
backwards compatibility. Signup paths now use the provider-specific
methods, allowing apps to disable local signups while permitting
OIDC (or any other combination).

Includes unit tests for delegation behavior and override logic.

Ref: moghtech/komodo#1087
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.

1 participant