matrix-org/dendrite

Dendrite does not validate `senders` and `rooms` IDs in filters

Aperta

#2067 aperta il 2 dic 2021

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (658 fork)auto 404
C-Client-APIF-FiltersT-Defectgood first issue

Metriche repository

Star
 (5632 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Background information

  • Dendrite version or git SHA: 83b9cb4d79123ec89e18a46d5202e74a9e9305b8
  • Running in Docker?: yes

This was found while running Complement, so build information from this Dockerfile also applies.

Description

When working on matrix-org/complement@8021d43 (#259), dendrite failed the tests with the following logs;

    client.go:392: POST hs1/_matrix/client/r0/user/@alice:hs1/filter => 200 OK (84.506966ms)
    invalid_test.go:180: Expected 4XX status code, got 200 for testing filter map[room:map[timeline:map[rooms:[not_a_room_id]]]]
    client.go:392: POST hs1/_matrix/client/r0/user/@alice:hs1/filter => 200 OK (57.808076ms)
    invalid_test.go:180: Expected 4XX status code, got 200 for testing filter map[room:map[timeline:map[senders:[not_a_sender_id]]]]

This test comes from sytest, and i am pretty sure that dendrite should at least try to validate these strings to valid User IDs and Room IDs

Guida contributor

Direzione di ricerca
Ispeziona l'endpoint del filtro in Dendrite per trovare dove vengono accettati gli ID delle stanze e dei mittenti, quindi aggiungi la validazione utilizzando funzioni di parsing degli ID Matrix (ad esempio gomatrixserverlib) per rifiutare ID non validi come 'not a room id' e 'not a sender id' restituendo un errore 4XX.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoMatrix
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65