Strip trailing / from server_url in register_new_matrix_user (#8823)

When server URL provided to register_new_matrix_user includes path
component (e.g. "http://localhost:8008/"), the command fails with
"ERROR! Received 400 Bad Request". Stripping trailing slash from the
server_url command argument makes sure combined endpoint URL remains
valid.

Signed-off-by: Dmitry Borodaenko angdraug@debian.org
pull/8839/head
Dmitry Borodaenko 2020-11-26 02:57:26 -08:00 committed by GitHub
parent 14f81a6d24
commit 7c43447477
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 1 deletions

1
changelog.d/8823.bugfix Normal file
View File

@ -0,0 +1 @@
Fix `register_new_matrix_user` failing with "Bad Request" when trailing slash is included in server URL. Contributed by @angdraug.

View File

@ -37,7 +37,7 @@ def request_registration(
exit=sys.exit,
):
url = "%s/_synapse/admin/v1/register" % (server_location,)
url = "%s/_synapse/admin/v1/register" % (server_location.rstrip("/"),)
# Get the nonce
r = requests.get(url, verify=False)