This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
v2_alpha /register fixes for Application Services #210
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Application services currently do not work with
v2_alpha
/register
endpoints as the logic to determine if they can register was wrong. In addition,v2_alpha
forced ASes to specify a password for their virtual clients, when it should be possible to register without a password.This PR fixes these issues and tidies up the
synapse/rest/client/v2_alpha/register.py
in general. This tidying up mainly involved re-shuffling theon_POST
logic to fork off to shared secret/AS logic ASAP so the bulk of the method remains purely for "normal" user logic. Previously, we were branching all over the method depending onis_application_server
oris_using_shared_secret
. I've also taken the liberty of adding some unit tests for this file.I've also done the following manual tests:
enable_registration: false
enable_registration: false