Verify `msgtype` is set on `m.room.message` events

Open S7evinK opened this issue 3 years ago • 2 comments

https://spec.matrix.org/v1.3/client-server-api/#mroommessage "Discovered" because I forgot to set the msgtype in Complement, Dendrite was happy, Synapse failed the tests because of the missing msgtype

Sep 15 '22 12:09 S7evinK

If you want to do unnecessary validation, shouldn't you also check that body exists?

Sep 15 '22 12:09 tulir

In which way unnecessary?

Sep 16 '22 05:09 S7evinK

Closing because we don't think it's necessary for Dendrite to enforce this.

Sep 27 '22 13:09 neilalexander