Skip to content

Matrix: switch to the mautrix-go library#1543

Closed
nightmared wants to merge 2 commits into
42wim:masterfrom
nightmared:mautrix
Closed

Matrix: switch to the mautrix-go library#1543
nightmared wants to merge 2 commits into
42wim:masterfrom
nightmared:mautrix

Conversation

@nightmared

Copy link
Copy Markdown
Contributor

Replaces https://github.com/matrix-org/gomatrix with https://github.com/tulir/mautrix-go as it appear to be the successor (see #1187 (comment)).

This could be a stepping stone for #137 and #1187.

Note that for reducing rebases this currently depends of #1541, however I will happily change that if #1541 is rejected.

Hopefully one day we will have encryption support with appservices for matrix ;)

@qlty-cloud-legacy qlty-cloud-legacy Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 10239 lines exceeds the maximum allowed for the inline comments feature.

@qlty-cloud-legacy

Copy link
Copy Markdown

Code Climate has analyzed commit 9818f32 and detected 0 issues on this pull request.

View more on Code Climate.

@nightmared

Copy link
Copy Markdown
Contributor Author

FYI I'm currently looking at adding a very basic appservice support for matterbridge using this patch, you can see what it looks like at https://github.com/nightmared/matterbridge/tree/mautrix_appservice.

@nightmared nightmared changed the title WIP: Matrix: switch to the mautrix-go library Matrix: switch to the mautrix-go library Jul 14, 2021
@TheHolyRoger

Copy link
Copy Markdown
Contributor

@nightmared you bloody beast thank you! I can help review and test this, I did briefly try to tackle it before.

Does this support encryption already?

@nightmared

Copy link
Copy Markdown
Contributor Author

I haven't yet tried it against encrypted room, there is probably more work to be done to support it (like some mechanism to store keys across sessions maybe ? I have to admit I don't know much of the way keys are managed in the matrix world, I would have to take a look. Do you know about it maybe ?)

@42wim

42wim commented Jul 20, 2021

Copy link
Copy Markdown
Owner

Hi @nightmared thanks for your work! I'll try to review it this week

@poVoq

poVoq commented Jul 27, 2021

Copy link
Copy Markdown

Would be great to have this merged sorry for nagging

@nightmared

Copy link
Copy Markdown
Contributor Author

I think I'm going to work on #1544 on the next few evenings (that draft PR depends on this PR), which will offer me the occasion to test this a bit further.
In any case, if you have tested this and it worked (and even more if it didn't), please share ;)

@w3bb

w3bb commented Aug 2, 2021

Copy link
Copy Markdown

I haven't yet tried it against encrypted room, there is probably more work to be done to support it (like some mechanism to store keys across sessions maybe ? I have to admit I don't know much of the way keys are managed in the matrix world, I would have to take a look. Do you know about it maybe ?)

A lot of bots just use Pantalaimon.

@nightmared

Copy link
Copy Markdown
Contributor Author

I'm not currently working on this, so I'm gonna close this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants