Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify matrix webhook session #5

Open
enter286 opened this issue Nov 7, 2024 · 4 comments
Open

Verify matrix webhook session #5

enter286 opened this issue Nov 7, 2024 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@enter286
Copy link

enter286 commented Nov 7, 2024

Is it possible to verify the matrix session from this webhook? As all messages sent from the webhook currently have a warning that the session isn't verified.

image

I tried verifying from another session in my browser, but that does send a message to this client, but I don't see a way to receive it?

@enter286
Copy link
Author

enter286 commented Nov 7, 2024

Tried to manually enter the device_id and access_token in the credentials.json as described here: #1 (comment)

The error message under the red exclamation mark changed from: Encrypted by a device not verified by its owner to Encrypted by an unknown or deleted device.

@immanuelfodor
Copy link
Owner

It's possible to do manual or emoji verification with the Nio library that this project uses, but it's not (yet) implemented in the E2EEClient.py, so you can't do it today. The linked comment is about another topic with SSO login, it's unrelated to yours.

Why I haven't implemented verification?

  • I'm the only user
  • The whole thing runs in a private network with no external access
  • Only Element is reluctant to complain about trust, better clients, such as SchildiChat don't complain
  • Even larger projects, such as Mautrix bridges don't implement it, so if it seems "good enough" to them, then it's good enough to me 😀

But I'm happy to accept a PR if you (or anybody else) add verification to the project! 🙏

@immanuelfodor immanuelfodor added enhancement New feature or request good first issue Good for newcomers labels Nov 7, 2024
@enter286
Copy link
Author

enter286 commented Nov 7, 2024

For some reason I had verification working without issues on a previous server installation. Just moved it to a new server this week and for some reason can't get it to appear as verified anymore.

image

Really appreciate your response. Just wanted to make sure there wasn't an easy fix I'm overlooking.

@immanuelfodor
Copy link
Owner

Yeah, nothing to worry about, the bot messages are still end-to-end encrypted, it's just a "cosmetic annoyance" by the Element clients.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants