ansol/matrix-docker-ansible-deploy
7
0
Fork 0
mirror of https://github.com/spantaleev/matrix-docker-ansible-deploy.git synced 2025-04-23 17:14:05 +00:00
Code Issues Projects Releases Wiki Activity

Only advertise 3478/udp (STUN/TURN) if Coturn actually enables it

Browse Source 
Related to ac26cc1cb0 which disabled
`3478/udp` by default.

Without this patch, homeservers continued to advertise the `3478/udp` port,
even though it was no longer exposed. While clients should handle that
gracefully (by falling back to TCP, etc.), it seems like Element either
doesn't do that or it doesn't support TCP (which this patch won't fix).

In any case, it's better to only announce what is actually supported/exposed.
This commit is contained in:
Slavi Pantaleev 2025-02-22 13:44:14 +02:00
parent 64a402eb65
commit 671a3dc94f
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
roles/custom/matrix-coturn/vars/main.yml
View File

@ -9,6 +9,9 @@ matrix_coturn_turn_uris: |-
+
([
'turn:' + matrix_server_fqn_matrix + '?transport=udp',
] if (matrix_coturn_container_stun_plain_host_bind_port_udp != '' or matrix_coturn_container_network == 'host') else [])
+
([
'turn:' + matrix_server_fqn_matrix + '?transport=tcp',
])
] if (matrix_coturn_container_stun_plain_host_bind_port_tcp != '' or matrix_coturn_container_network == 'host') else [])
}}