60 lines
2.2 KiB
Plaintext
60 lines
2.2 KiB
Plaintext
---
|
|
summary: Federation 'send_join' is insufficiently pedantic
|
|
---
|
|
created: 2015-10-27 17:07:22.0
|
|
creator: leonerd
|
|
description: |-
|
|
In the process of writing a wire-level federation test in sytest to check the room join dance of synapse, I have ended up with a "minimal viable case" test (https://github.com/matrix-org/sytest/blob/paul/federation/tests/50federation/30room-join.pl).
|
|
|
|
This test appears to be too small in places, and is missing things that we feel synapse ought to require. This bug exists to track those things it should be checking, that this test demonstrates it currently does not.
|
|
|
|
synapse does not notice any of the following mistakes:
|
|
|
|
* missing {{m.room.member}} event for the room creator.
|
|
* missing {{m.room.power_levels}} event.
|
|
* {{m.room.member}} event of joining user has {{depth}} of 1.
|
|
* all {{auth_events}} have a depth of 0.
|
|
* joining server does not hash or sign the event submitted to {{send_join}}.
|
|
* the {{state}} key returned by {{send_join}} can be an empty object, or an empty list.
|
|
id: '12045'
|
|
key: SYN-507
|
|
number: '507'
|
|
priority: '3'
|
|
project: '10000'
|
|
reporter: leonerd
|
|
status: '10100'
|
|
type: '1'
|
|
updated: 2016-11-07 18:28:31.0
|
|
votes: '0'
|
|
watches: '2'
|
|
workflowId: '12148'
|
|
---
|
|
actions:
|
|
- author: leonerd
|
|
body: |-
|
|
Actually, this seems not quite true.
|
|
|
|
While {{send_join}} appears to be happy, and causes the joining HS to return a 200 OK to the requesting client along with the room ID, any subsequent queries by the user's client on their HS seem to reply with 403 "user is not in the room".
|
|
created: 2015-12-10 15:54:22.0
|
|
id: '12461'
|
|
issue: '12045'
|
|
type: comment
|
|
updateauthor: leonerd
|
|
updated: 2015-12-10 15:54:22.0
|
|
- author: leonerd
|
|
body: Ugh. I retract my previous comment. It was working fine if I actually use the room ID, not the room alias.
|
|
created: 2015-12-10 16:17:30.0
|
|
id: '12466'
|
|
issue: '12045'
|
|
type: comment
|
|
updateauthor: leonerd
|
|
updated: 2015-12-10 16:17:30.0
|
|
- author: richvdh
|
|
body: 'Migrated to github: https://github.com/matrix-org/synapse/issues/1407'
|
|
created: 2016-11-07 18:28:31.0
|
|
id: '13716'
|
|
issue: '12045'
|
|
type: comment
|
|
updateauthor: richvdh
|
|
updated: 2016-11-07 18:28:31.0
|