Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Unspecced user_id field present in /messages response #7924

Open
joepie91 opened this issue Jul 21, 2020 · 1 comment
Open

Unspecced user_id field present in /messages response #7924

joepie91 opened this issue Jul 21, 2020 · 1 comment
Labels
A-Spec-Compliance places where synapse does not conform to the spec T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@joepie91
Copy link

When querying the /messages endpoint, the events in the chunk in the response will include a user_id field which duplicates the sender field. The user_id field is not listed in the spec (nor shown in its examples), and so should probably be removed?

Related: #7606?
@clokep clokep added the A-Spec-Compliance places where synapse does not conform to the spec label Jul 21, 2020
@Half-Shot
Copy link
Collaborator

I suspect this might be that synapse is still sending v1 event format for compatibility alongside v2. Related to #6457

@ShadowJonathan ShadowJonathan mentioned this issue Sep 20, 2020
Closed
@reivilibre reivilibre added the T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. label May 23, 2023
@matrixbot matrixbot mentioned this issue Dec 21, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Spec-Compliance places where synapse does not conform to the spec T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@clokep @joepie91 @Half-Shot @reivilibre