matrix-org/dendrite

The transaction_id within events is not serialised in many endpoints

Offen

#3.000 geöffnet am 06.03.2023

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (658 Forks)auto 404
O-FrequentT-Defectgood first issuespec-compliance

Repository-Metriken

Stars
 (5.632 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Studieren Sie den vorhandenen Complement Test unter https://github.com/matrix org/complement/pull/621, um das erwartete Verhalten zu verstehen. Verfolgen Sie den Ereignisserialisierungscode in Dendrite, mit Schwerpunkt auf dem Endpunkt /rooms/{roomId}/event/{eventId} und anderen Endpunkten, die Ereignisse zurückgeben. Identifizieren Sie, wo transaction id zum unsigned Feld hinzugefügt werden soll, und implementieren Sie die Korrektur in allen relevanten Endpunkten.
Tech Stack
go
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoMatrix event model
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75