`TestMSC4311FullEventsOnStrippedStateFederation` fails if the homeserver refuses to produce an invite for a non-existing user

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#947 0 commentaires 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
62/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
go
Domaine
testing-qa

Piste de recherche

Le test en échec est TestMSC4311FullEventsOnStrippedStateFederation dans tests/v12_test.go ; l'appel d'invitation autour de la ligne 1597 est l'endroit où CSAPI.Must transforme le 404 en erreur fatale. Lis ce test et l'helper d'invitation qui l'entoure, puis exécute le sous-test avec go test -run TestMSC4311FullEventsOnStrippedStateFederation ./tests/ contre un homeserver. C'est terminé lorsque le test tolère un 404 pour l'invité inexistant sans échouer, tout en vérifiant toujours le comportement de stripped state qu'il est censé valider.

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

Description

From what I can see, Alice tries to invite Bob, but never registers bob: https://github.com/matrix-org/complement/blob/e3fcff1998d91f71cae096679b3ec737dda69fdd/tests/v12_test.go#L1595-L1597

Then, if the server returns an error because it doesn't recognise the invitee, complement fails the entire test (non-200 code):

  ❌ TestMSC4311FullEventsOnStrippedStateFederation/parallel (FAIL; 0s)
  ❌ TestMSC4311FullEventsOnStrippedStateFederation/parallel/`invite_room_state` (FAIL; 380ms)
    client.go:893: [CSAPI] POST hs1/_matrix/client/v3/register => 401 Unauthorized (361.421µs)
    client.go:893: [CSAPI] POST hs1/_matrix/client/v3/register => 200 OK (195.359442ms)
    client.go:893: [CSAPI] POST hs1/_matrix/client/v3/createRoom => 200 OK (98.712927ms)
    client.go:893: [CSAPI] POST hs1/_matrix/client/v3/rooms/!kqDNpAeTQj7OTwOK2OAv7zRIgvRWnaGbKWkNJbThDI8/invite => 404 Not Found (2.093963ms)
    v12_test.go:1597: CSAPI.Must: POST http://127.0.0.1:32953/_matrix/client/v3/rooms/%21kqDNpAeTQj7OTwOK2OAv7zRIgvRWnaGbKWkNJbThDI8/invite returned non-2xx code: 404 Not Found - body: {"errcode":"M_NOT_FOUND","error":"User does not exist"}

The spec does not require that the server produces invites for users that do not exist, and it would be pretty silly to do so (even if most servers nowadays allow this to prevent things like user enumeration)

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.