Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Ask Homeserver for username availability upon registration #2421

Merged
merged 6 commits into from
Feb 29, 2024

Conversation

sandhose
Copy link
Member

@sandhose sandhose commented Feb 28, 2024

Fixes #2376

Whenever MAS register a user, from the registration form, the upstream OAuth 2.0 registration or the GraphQL API, it now ask the homeserver whether the username is available.
This makes sure a user doesn't accidentally register with a username used by an internal account or by an application service.

This check can be bypassed on the GraphQL API by administrators using the new skipHomeserverCheck flag.

This can be reviewed commit-by-commit.

Copy link

cloudflare-workers-and-pages bot commented Feb 28, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3e458a1
Status: ✅  Deploy successful!
Preview URL: https://6fb15905.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-check-username-avai.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose force-pushed the quenting/check-username-availability branch from 2cee764 to 0794f2d Compare February 28, 2024 17:17
@sandhose sandhose marked this pull request as ready for review February 28, 2024 18:07
@sandhose sandhose added A-GraphQL Changes to the GraphQL API A-Registration-Flow Related to the user registration flow A-Homeserver-Integration Integration with the homeserver labels Feb 29, 2024
@sandhose sandhose force-pushed the quenting/check-username-availability branch from 3e458a1 to 508382a Compare February 29, 2024 09:36
@sandhose sandhose merged commit 183f7ba into main Feb 29, 2024
16 checks passed
@sandhose sandhose deleted the quenting/check-username-availability branch March 19, 2024 10:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-GraphQL Changes to the GraphQL API A-Homeserver-Integration Integration with the homeserver A-Registration-Flow Related to the user registration flow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check if username is valid on registration through the homeserver
1 participant