`register_rejects_if_user_already_exists` does not account for implementations requiring auth before performing validation

Ouverte Adaptée aux débutants
#955 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
72/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
go
Domaine
testing

Piste de recherche

Trouvez TestRegistration/parallel/POST_/register_rejects_if_user_already_exists dans les tests d’inscription et lisez comment il traite la réponse de /register. Mettez à jour le test pour terminer un défi d’authentification interactif lorsqu’il est renvoyé, puis vérifiez que l’inscription rejette l’utilisateur en double. Le travail est terminé lorsque le test couvre à la fois un rejet direct et un défi UIA suivi d’un rejet.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

TestRegistration/parallel/POST_/register_rejects_if_user_already_exists fails if the homeserver returns a UIA challenge instead of outright rejecting the request due to the duplicate user. Continuwuity, for example, still rejects the request as expected after completing UIA, but interactive auth is required before it is willing to progress further:

$ curl --json '{"username": "nex", "password": "hello world"}' 10.11.10.134:8008/_matrix/client/v3/register
{"flows":[{"stages":["m.login.registration_token"]}],"params":{},"session":"redacted"}%                                                                                                    $ curl --json '{"username": "nex", "password": "hello world", "auth": {"type": "m.login.registration_token", "token": "redacted", "session": "redacted"}}' 10.11.10.134:8008/_matrix/client/v3/register
{"errcode":"M_USER_IN_USE","error":"Username is not available."}

There's no spec that defines what order these two things should be checked in, so the test should probably try to complete the challenge if it receives one.

Langage dominant
Go
Étoiles
102
Forks
73
Merge moyen
3 j 10 h
PR mergées (30 j)
6

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de matrix-org/complement

Toutes les issues de matrix-org/complement

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.