matrix-org/dendrite

The transaction_id within events is not serialised in many endpoints

Aperta

#3000 aperta il 6 mar 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Go (658 fork)auto 404
O-FrequentT-Defectgood first issuespec-compliance

Metriche repository

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

Descrizione

Similar to https://github.com/matrix-org/synapse/issues/15173 in Synapse

There are many endpoints that return events, and in those representations they should include the transaction_id in the unsigned part of the event.

I wrote a Complement test to highlight that Synapse had this issue by testing using the /rooms/{roomId}/event/{eventId} endpoint, and Dendrite also fails on this test. I haven't tested the other endpoints, nor covered them in the Complement test. https://github.com/matrix-org/complement/pull/621

Guida contributor

Direzione di ricerca
Studia il test Complement esistente su https://github.com/matrix org/complement/pull/621 per capire il comportamento atteso. Traccia il codice di serializzazione degli eventi in Dendrite, concentrandoti sull'endpoint /rooms/{roomId}/event/{eventId} e su altri endpoint che restituiscono eventi. Identifica dove aggiungere transaction id al campo unsigned e implementa la correzione in tutti gli endpoint rilevanti.
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 event model
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75