`TestInboundFederationKeys` fails if empty `old_verify_keys` is omitted

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

Piste de recherche

La vérification en échec se trouve dans tests/federation_keys_test.go, autour des lignes 74-101, où MatchResponse avec JSONMapEach exige la clé old_verify_keys. Lisez ce bloc pour voir comment les autres champs optionnels sont gérés, puis rendez old_verify_keys optionnel afin qu'une clé absente réussisse tandis qu'une clé présente et bien formée soit toujours vérifiée. Exécutez TestInboundFederationKeys ; c'est terminé lorsque le test réussit face à un serveur qui omet le champ vide.

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

Description

This field in /_matrix/key/v2/server is optional, and in Mautrix omitted when empty, but Complement returns an error if it is not present at all:

  ❌ TestInboundFederationKeys (FAIL; 4.75s)
2026/11/03 01:08:53 WARNING: complement_fed_3_fed.1_servers.hs1_1 has a named VOLUME /var/lib/postgresql/data - volumes can lead to unpredictable behaviour due to test pollution. Remove the VOLUME in the Dockerfile to suppress this message.
    federation_keys_test.go:30: Deploy times: 9.103914ms blueprints, 3.450035018s containers
    federation_keys_test.go:45: MatchResponse JSONMapEach: missing key 'old_verify_keys' - https://127.0.0.1:32789/_matrix/key/v2/server => {"server_name":"hs1","signatures":{"hs1":{"ed25519:307d59fc32ee2771e7292fa1c4519e77ce6d6069d0bb78400d1208e48c278abd":"EBzKuwvrcSo2RzONAKAeN76zwdf4OTJxrLFQDeFmOKLq5MFQXFXKwU3cwFH2h19uANQu5wDbxpvy0YNfUUBlCg"}},"valid_until_ts":1793754536056,"verify_keys":{"ed25519:307d59fc32ee2771e7292fa1c4519e77ce6d6069d0bb78400d1208e48c278abd":{"key":"eN5abljPWdwVk+w5MIrl7Ld/KqWVH5+Ms6vnKWxcKBM"}}}

https://github.com/matrix-org/complement/blob/e3fcff1998d91f71cae096679b3ec737dda69fdd/tests/federation_keys_test.go#L74-L101

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.