search

YunoHost-Apps / synapse_ynh

Matrix server (synapse) package for YunoHost
https://matrix.org/
GNU General Public License v3.0
79 stars 42 forks source link

coturn-synapse not starting after update #327

Closed svenbrier closed 1 year ago

svenbrier commented 1 year ago

Hi, I just updated my instance and since then my coturn-synapse service is not starting anymore. Do you know what could cause this behaviour.

Log: https://paste.yunohost.org/uhuboyoxoy

ghost commented 1 year ago

I have the same problem

dr41nU commented 1 year ago

It seems to be related to the new sandboxing options of the systemd service file

Systemd kills the program as there is a sandbox violation.

This workaround can be applied:

# systemctl edit coturn-synapse

Add the following lines:

[Service]
SystemCallFilter=
SystemCallFilter=~@clock @debug @module @mount @obsolete @reboot  @swap

It will remove @setuid filter

Then reload the unit file:

# systemctl daemon-reload

svenbrier commented 1 year ago

Thank you @dr41nU! The service is now running again. 😄

Thatoo commented 1 year ago

I get this error : Editing "/etc/systemd/system/coturn-synapse.service.d/override.conf" canceled: temporary file is empty.

svenbrier commented 1 year ago

You can't edit this file directly. Follow the steps of @dr41nU s workaround. Hint: You must place the lines between the comments, not at the end of the file.

tutle commented 1 year ago

Works back again , thanks a lot !!!

tio-trom commented 1 year ago

Same issue here. @dr41nU tips fixed it for me too. I have many yunohost services installed on the same server so my question is, will this "hack" impact them?

Thatoo commented 1 year ago

You can't edit this file directly. Follow the steps of @dr41nU s workaround. Hint: You must place the lines between the comments, not at the end of the file.

Thanks for the hint. I was missing this information. I thought we had to add at the end, sorry...

It works.

codedbreeze commented 1 year ago

I had the same issue. @dr41nU's tips helped. Thanks.

Thatoo commented 1 year ago

Will we have to think of undoing this before the next update that will contain this PR?

jbastruz commented 1 year ago

Hi every one, I had the same issue as @dr41nU, I just did the modification and it fixed my point, but since then, it seem to generate some instability on my Yunohost instance, I have many difficulties to join a call (error messages, i Have to wait 1/2 real minutes before the call start, and then my yunohost crashes during the next minutes... am i the only one with this issue ?

Thatoo commented 1 year ago

@Josue-T when Testing will be merged into master, for those who did the trick, do we have to "undo" it before updating?