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

DietPi-Software | Nextcloud Talk: New coturn systemd unit on Buster #2440

Closed
MichaIng opened this issue Jan 25, 2019 · 1 comment
Closed
Labels
Buster Solution available 🥂 Definite solution has been done
Milestone

Comments

@MichaIng
Copy link
Owner

On Buster, the coturn APT package now includes a systemd unit, superseding the init.d service.

  • It runs as user turnserver now. This means we need to adjust permissions to allow TLS with LetsEncrypt certificate. Goes hand in hand with https://github.com/Fourdee/DietPi/issues/2363 to implement TLS support for various software titles which do not use a dedicated webserver.
  • It does (currently) not use /etc/default/coturn, thus command arguments (currently logging related) need to be added via systemd unit drop-in config.
    • Would be great, if possible to add arguments to the main ExecStart instead of completely replacing it. $(sed ...) --no-stdout-log ... perhaps possible, but very hacky 🤔.
@MichaIng MichaIng added this to the Planned for implementation milestone Jan 25, 2019
@MichaIng MichaIng modified the milestones: Planned for implementation, v6.23 Mar 9, 2019
MichaIng added a commit that referenced this issue Apr 26, 2019
+ DietPi-Software | Nextcloud Talk: Do not apply TLS settings to coTURN, which has no security benefit and complicates key handling on Buster, where coTURN runs as limited user: #2592, #2440
+ DietPi-Software | Nextcloud Talk: Fix wrong/outdated coTURN settings handling due to some changes with newer versions
@MichaIng
Copy link
Owner Author

Ui, I already enabled logging settings for systemd unit some time ago, great!

PR up to not enable TLS on coTURN anymore, since there is no security advantage and Nextcloud Talk does not use the required TURNS protocol anyway: PR up: #2740

MichaIng added a commit that referenced this issue Apr 26, 2019
+ DietPi-Software | Nextcloud Talk: Do not apply TLS settings to coTURN, which has no security benefit and complicates key handling on Buster, where coTURN runs as limited user: #2592, #2440
+ DietPi-Software | Nextcloud Talk: Fix wrong/outdated coTURN settings handling due to some changes with newer versions (Buster)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Buster Solution available 🥂 Definite solution has been done
Projects
None yet
Development

No branches or pull requests

1 participant