Add script for bridge app-service registration

YunoHost-Apps / dendrite_ynh

Matrix homeserver of second-generation, packaged for YunoHost

GNU General Public License v3.0

16 stars 7 forks source link

Add script for bridge app-service registration #108

Open Gredin67 opened 1 year ago

Gredin67 commented 1 year ago

Adapt this script to dendrite https://github.com/YunoHost-Apps/synapse_ynh/blob/master/sources/update_synapse_for_appservice.sh
see also https://github.com/YunoHost-Apps/mautrix_telegram_ynh/issues/30

orhtej2 commented 1 year ago

While possible this actually requires forking bridge itself to puppet the proper server as it's hardcoded to interact with Synapse ATM.

It is possible though, I have run this PoC repo successfully some time ago: https://github.com/orhtej2/mautrix_signal_dendrite_ynh

Gredin67 commented 1 year ago

requires forking bridge itself

we don't need to fork mautrix bridges, just to add a functionality to their yunohost packages. Can you open a PR from your branch against https://github.com/YunoHost-Apps/mautrix_signal_ynh/tree/testing Choosing the matrix server (synapse or dendrite instance) to link the bridge to should be done at install. I don't think it's a good idea to change the matrix server after installation.

orhtej2 commented 1 year ago

we don't need to fork mautrix bridges, just to add a functionality to their yunohost packages.

That's what I meant, should have used better wording :P

My repo is in super experimental stage, merging it as-is will break Synapse compatibility.

Gredin67 commented 1 year ago

I know, just open a PR so we can work on getting synapse and dendrite work together in the mautrix_signal package

Le 10 août 2023 17:35:25 GMT+02:00, orhtej2 @.***> a écrit :

we don't need to fork mautrix bridges, just to add a functionality to their yunohost packages.

That's what I meant, should have used better wording :P

My repo is in super experimental stage, merging it as-is will break Synapse compatibility.

-- Reply to this email directly or view it on GitHub: https://github.com/YunoHost-Apps/dendrite_ynh/issues/108#issuecomment-1673463664 You are receiving this because you authored the thread.

Message ID: @.***>

Gredin67 commented 1 year ago

It is possible though, I have run this PoC repo successfully some time ago: https://github.com/orhtej2/mautrix_signal_dendrite_ynh

@orhtej2 can you please open a PR from your branch against the testing branch of this repo ?

Gredin67 commented 1 year ago

Adapt this script to dendrite https://github.com/YunoHost-Apps/synapse_ynh/blob/master/sources/update_synapse_for_appservice.sh

@Josue-T could you have a look at the appservice registration in dendrite ? Probably just copy-pasting the job you did in synapse.

Gredin67 commented 1 year ago

It is possible though, I have run this PoC repo successfully some time ago: https://github.com/orhtej2/mautrix_signal_dendrite_ynh

@orhtej2 can you please open a PR from your branch against the testing branch of this repo ?

nevermind I did it https://github.com/YunoHost-Apps/mautrix_signal_ynh/pull/85