matrix-org/dendrite

The transaction_id within events is not serialised in many endpoints

Aberta

#3.000 aberto em 6 de mar. de 2023

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)Go (658 forks)auto 404
O-FrequentT-Defectgood first issuespec-compliance

Métricas do repositório

Stars
 (5.632 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Estude o teste Complement existente em https://github.com/matrix org/complement/pull/621 para entender o comportamento esperado. Rastreie o código de serialização de eventos no Dendrite, focando no endpoint /rooms/{roomId}/event/{eventId} e outros endpoints que retornam eventos. Identifique onde adicionar transaction id ao campo unsigned e implemente a correção em todos os endpoints relevantes.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GoMatrix event model
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75