Skip to content

Added logic to prevent new users from registering an id account#811

Merged
MrKevJoy merged 12 commits intomainfrom
prevent-new-registrations
Apr 7, 2026
Merged

Added logic to prevent new users from registering an id account#811
MrKevJoy merged 12 commits intomainfrom
prevent-new-registrations

Conversation

@MrKevJoy
Copy link
Copy Markdown
Contributor

@MrKevJoy MrKevJoy commented Jan 16, 2026

Context

https://trello.com/c/xDWeziy2/2241-prevent-new-aytq-users-creating-dfe-id-accounts-and-direct-them-to-one-login

  • Skip landing page for certain clients
  • Prevent new account sign ups for certain clients, configured from appsettings/keyvault.
  • added new client prevent-registration-client
  • added client to appsettings
  • updated StartOAuthJourney to include clientid as an optional parameter.

Changes proposed in this pull request

Include a summary of the change.

Guidance to review

Include any useful information needed to review this change.
Include any dependencies that are required for this change.

Checklist

  • Attach to Trello card
  • Rebased master
  • Cleaned commit history
  • Tested by running locally
  • Reminder created to manually clean any removed app settings post deployment

Comment thread dotnet-authserver/tests/TeacherIdentity.AuthServer.EndToEndTests/Register.cs Outdated
@MrKevJoy MrKevJoy force-pushed the prevent-new-registrations branch 4 times, most recently from 7d0958a to f466419 Compare January 19, 2026 12:45
@MrKevJoy MrKevJoy requested a review from gunndabad January 19, 2026 12:55
@MrKevJoy MrKevJoy force-pushed the prevent-new-registrations branch from 92e6430 to e52654a Compare March 31, 2026 11:44
MrKevJoy added 2 commits April 7, 2026 08:59
Merge registration-token branch into prevent-new-registrations.
@MrKevJoy MrKevJoy enabled auto-merge (squash) April 7, 2026 08:11
@MrKevJoy MrKevJoy merged commit 9ccc6e0 into main Apr 7, 2026
8 checks passed
@MrKevJoy MrKevJoy deleted the prevent-new-registrations branch April 7, 2026 08:13
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