search

kamax-matrix / matrix-appservice-email

Matrix <--> E-mail Bridge
GNU Affero General Public License v3.0
39 stars 11 forks source link

Enabling encryption breaks communication #28

Open vmario89 opened 5 years ago

vmario89 commented 5 years ago

HI, i just played around a little bit in Riot and enabled encrytion for a room. After enabling the communication was broken. The log:

Processing /transactions/114
22.883  INFO 963 --- [nio-8091-exec-5] i.k.m.b.e.m.m.MatrixApplicationService   : HS provided valid credentials
22.883  INFO 963 --- [nio-8091-exec-5] i.k.m.b.e.m.m.MatrixApplicationService   : Unknown event type m.room.encryption from @vmario89:matrix.fablabchemnitz.de
27.760  INFO 963 --- [nio-8091-exec-6] i.k.m.b.e.c.ApplicationServiceController : Processing /transactions/115
27.761  INFO 963 --- [nio-8091-exec-6] i.k.m.b.e.m.m.MatrixApplicationService   : HS provided valid credentials
27.761  INFO 963 --- [nio-8091-exec-6] i.k.m.b.e.m.m.MatrixApplicationService   : Unknown event type m.room.encrypted from @vmario89:matrix.fablabchemnitz.de

i was able to reset the communication by leaving the room and reinvite the email user.

I think the encryption enable button should be disabled like mautrix-whatsapp does it too (there the button in Riot is grey/forbidden). Or at least it maybe should be documented that its not a good idea to use.

kind regards, Mario

maxidorius commented 5 years ago

enabled encrytion for a room. After enabling the communication was broken.

Encryption is indeed not supported. Given that E2EE is also not a stable feature in Matrix yet, it's not possible to implement it at the current time in a reliable way. I'll update the main README to outline this.

As for disabling the option to enable encryption, I'll keep that as an enhancement in a separate issue. Thank you for reporting this.