From 132b44a8b630c37a12936fd07accef6fc496878c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=94=D0=B0=D0=BC=D1=8F=D0=BD=20=D0=9C=D0=B8=D0=BD=D0=BA?= =?UTF-8?q?=D0=BE=D0=B2?= Date: Wed, 27 Apr 2022 13:07:47 -0500 Subject: [PATCH] feat: Drops nginx dependency for turnserver config. We used to multiplex the ports in nginx, but we dropped that at some point, so now coturn is on its own listening and nginx dependency is no longer needed. Our turnserver config can be used with nginx | apache2. --- debian/control | 2 +- debian/jitsi-meet-turnserver.postinst | 16 +--------------- debian/jitsi-meet-turnserver.postrm | 16 +--------------- 3 files changed, 3 insertions(+), 31 deletions(-) diff --git a/debian/control b/debian/control index e146ce186..438779887 100644 --- a/debian/control +++ b/debian/control @@ -54,5 +54,5 @@ Package: jitsi-meet-turnserver Architecture: all Breaks: apache2 Pre-Depends: jitsi-meet-web-config -Depends: ${misc:Depends}, nginx (>= 1.13.10) | nginx-full (>= 1.13.10) | nginx-extras (>= 1.13.10), jitsi-meet-prosody, coturn, dnsutils +Depends: ${misc:Depends}, jitsi-meet-prosody, coturn, dnsutils Description: Configures coturn to be used with Jitsi Meet diff --git a/debian/jitsi-meet-turnserver.postinst b/debian/jitsi-meet-turnserver.postinst index aa9fdb703..1293ad67a 100644 --- a/debian/jitsi-meet-turnserver.postinst +++ b/debian/jitsi-meet-turnserver.postinst @@ -33,7 +33,6 @@ case "$1" in JVB_HOSTNAME=$(echo "$RET" | xargs echo -n) TURN_CONFIG="/etc/turnserver.conf" - NGINX_CONFIG="/etc/nginx/sites-available/$JVB_HOSTNAME.conf" JITSI_MEET_CONFIG="/etc/jitsi/meet/$JVB_HOSTNAME-config.js" # if there was a turn config backup it so we can configure @@ -51,19 +50,6 @@ case "$1" in fi fi - # this detect only old installations with no nginx - db_get jitsi-meet/jvb-serve || true - if [ ! -f $NGINX_CONFIG -o "$RET" = "true" ] ; then - # nothing to do - echo "------------------------------------------------" - echo "" - echo "turnserver not configured" - echo "" - echo "------------------------------------------------" - db_stop - exit 0 - fi - if [[ -f $TURN_CONFIG ]] ; then echo "------------------------------------------------" echo "" @@ -117,7 +103,7 @@ denied-peer-ip=240.0.0.0-255.255.255.255" >> $TURN_CONFIG sed -i "s/jitsi-meet.example.com/$JVB_HOSTNAME/g" $TURN_CONFIG sed -i "s/__turnSecret__/$TURN_SECRET/g" $TURN_CONFIG - # SSL for nginx + # SSL settings db_get jitsi-meet/cert-choice CERT_CHOICE="$RET" diff --git a/debian/jitsi-meet-turnserver.postrm b/debian/jitsi-meet-turnserver.postrm index a809526ee..6e1308e58 100644 --- a/debian/jitsi-meet-turnserver.postrm +++ b/debian/jitsi-meet-turnserver.postrm @@ -23,26 +23,12 @@ set -e case "$1" in - remove) - if [ -x "/etc/init.d/nginx" ]; then - invoke-rc.d nginx reload || true - fi - if [ -x "/etc/init.d/apache2" ]; then - invoke-rc.d apache2 reload || true - fi - ;; purge) rm -rf /etc/turnserver.conf - if [ -x "/etc/init.d/nginx" ]; then - invoke-rc.d nginx reload || true - fi - if [ -x "/etc/init.d/apache2" ]; then - invoke-rc.d apache2 reload || true - fi # Clear the debconf variable db_purge ;; - upgrade|failed-upgrade|abort-install|abort-upgrade|disappear) + remove|upgrade|failed-upgrade|abort-install|abort-upgrade|disappear) ;; *)