`TestRestrictedRoomsLocalJoinInMSC3787Room` asserts the wrong event auth behaviour

Aberta Para iniciantes
  • Algo errado?
#949 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 2 dias

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
70/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
go
Domínio
testing-qa

Direção de pesquisa

O comentário incorreto e a asserção incorreta estão em tests/restricted_rooms_test.go, por volta das linhas 109-110, dentro de TestRestrictedRoomsLocalJoinInMSC3787Room, no subteste Join_should_succeed_when_joined_to_allowed_room. Leia as regras de autorização da especificação vinculada para a versão de sala v12 para confirmar que a verificação da assinatura acontece antes da transição de associação. Depois, execute esse teste para reproduzir o 403. Está concluído quando o comentário e a expectativa correspondem à especificação e o teste passa em um servidor compatível com a especificação.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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 "}
Linguagem predominante
Go
Estrelas
102
Forks
73
Merge médio
3d 10h
PRs com merge (30d)
6

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de matrix-org/complement

Todas as issues de matrix-org/complement

Issues semelhantes

Mais issues de Go

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.