Skip to content

ejabberd: Fix making call connections when using TURN

Sunil Mohan Adapa requested to merge sunilmohan/freedombox:ejabberd-remove-turn-restricted into master

Closes: #2318 (closed).

We currently set 'restricted: false' to both stun and turn server configuration in ejabberd. This works for stun but for turn, ejabberd assumes that authentication is not needed even though it is needed for our coturn setup. Drop the configuration option entirely as the desired values are already default in both stun and turn cases.

Tests:

  • On a fresh setup, install coturn and ejabberd. In ejabberd.yaml, the stun/turn configuration does not have the restricted option.

  • On a container without the patch, install coturn and ejabberd. Configuration has restricted option. Apply the patch, ejabberd setup is updated and restricted option is removed.

  • Test that calls can't be made with TURN with 'restricted: false' set. Changing it to true for TURN configuration allows the calls to be established. Remove the restricted option entirely also works. This was tested by @znoteer in #2318 (closed).

Signed-off-by: Sunil Mohan Adapa sunil@medhas.org

Merge request reports