`TestRestrictedRoomsLocalJoinInMSC3787Room` asserts the wrong event auth behaviour

Abierto Apto para principiantes
  • ¿Algo no está bien?
#949 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
70/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
go
Área
testing-qa

Línea de trabajo

El comentario incorrecto y la aserción incorrecta están en tests/restricted_rooms_test.go alrededor de las líneas 109-110, dentro de TestRestrictedRoomsLocalJoinInMSC3787Room, en el subtest Join_should_succeed_when_joined_to_allowed_room. Lee las reglas de autorización de la especificación vinculada para la versión de sala v12 y confirma que la comprobación de la firma ocurre antes de la transición de membresía. Luego ejecuta esa prueba para reproducir el 403. Se considera terminado cuando el comentario y la expectativa coinciden con la especificación y la prueba pasa contra un servidor que cumple la especificación.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

https://github.com/matrix-org/complement/blob/e3fcff1998d91f71cae096679b3ec737dda69fdd/tests/restricted_rooms_test.go#L109-L110

The comment here is incorrect: the signature of join_authorised_via_users_server is checked BEFORE the membership transition. This means servers that correctly adhere to the authorisation rules get failed by complement:

  ❌ TestRestrictedRoomsLocalJoinInMSC3787Room/Join_should_succeed_when_joined_to_allowed_room (FAIL; 50ms)
    restricted_rooms_test.go:79: SendEventSynced waiting for event ID $tljunp80q2nmNKDBQTa5pxrkg3sRqMRqKneNHPrlJks
    restricted_rooms_test.go:99: CSAPI.MustDo PUT http://127.0.0.1:32803/_matrix/client/v3/rooms/%21uScyaBRyqLnn:hs1/state/m.room.member/@user-2:hs1 returned non-2xx code: 403 Forbidden - body: {"errcode":"M_FORBIDDEN","error":"fail 5.2: m.room.member event is not signed by server of join_authorised_via_users_server: no signatures found for server "}
Lenguaje dominante
Go
Estrellas
102
Forks
73
Merge medio
1 d 2 h
PR fusionados (30 d)
3

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de matrix-org/complement

Todos los issues de matrix-org/complement

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.