Skip to content

Conversation

@julien-nc
Copy link
Member

@julien-nc julien-nc commented Aug 14, 2020

Hey! Here is a proof-of-concept that integrates matterbridge into Talk.

@gary-kim has recently implemented NC Talk compatibility to Matterbridge, which makes it possible to bridge it with many other instant messaging systems (centralized and distributed ones).

Matterbridge binary needs can be installed from https://apps.nextcloud.com/apps/talk_matterbridge

There's a new "bridge" section in the settings tab of the right sidebar for room moderators that lets them configure bridge parts.

Server side generates a bridge config file and runs the binary which connects to each parts of the bridge. The process is launched by the web server user and runs in the background. Its PID is stored and used to kill it when a moderator disables the bridge.

TODO:

  • make it work
  • choose where to trigger checks to potentially relaunch dead bridges or kill zombie ones (checks are implemented)
  • improve bridge state display in UI. only the "desired state" is shown for the moment. we could show the log file content.
  • make it safer with more checks when launching the process.
  • write config and log files in appdata (or somewhere else). currently written in /tmp

@julien-nc
Copy link
Member Author

@nickvergessen Thanks a lot for the review.

@julien-nc julien-nc force-pushed the enh/matterbridge-backend branch 2 times, most recently from 0e7df3b to 25e7ee5 Compare August 19, 2020 11:14
@julien-nc julien-nc force-pushed the enh/matterbridge-backend branch from 8070457 to b73968c Compare August 20, 2020 12:18
@nickvergessen nickvergessen force-pushed the enh/matterbridge-backend branch from 324548e to 5312e50 Compare August 21, 2020 09:51
@nickvergessen
Copy link
Member

@nickvergessen
Copy link
Member

How installing looks:

Peek 2020-08-24 13-38

@nickvergessen nickvergessen force-pushed the enh/matterbridge-backend branch from 600b9d0 to 8484d0b Compare August 24, 2020 11:40
Julien Veyssier and others added 25 commits August 25, 2020 08:47
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
@nickvergessen nickvergessen force-pushed the enh/matterbridge-backend branch from 3a1f1d2 to 4700152 Compare August 25, 2020 06:47
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Doesn't break the rest and should work as per Julien

@nickvergessen nickvergessen merged commit ebfdaa4 into master Aug 25, 2020
@nickvergessen nickvergessen deleted the enh/matterbridge-backend branch August 25, 2020 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing enhancement feature: integration 📦 Integration with 3rd party (chat) service

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants