docker-jitsi-meet icon indicating copy to clipboard operation
docker-jitsi-meet copied to clipboard

Hardcoded prosody port in mod_c2s.lua

Open jmallach opened this issue 1 year ago • 5 comments

After changing the default XMPP_PORT to '5223', prosody was still trying to use 5222 in addition to 5223:

jitsi-prosody-1   jitsi/prosody:stable-9457-2   "/init"   prosody   16 minutes ago   Up 8 minutes    5222-5223/tcp, 5269/tcp, 5280/tcp, 5347/tcp

After inspecting the container, it looks like the jitsi prosody container has a hardcoded value for the default port, which does not use XMPP_PORT at all:

root@1b30451b8a46:/# rgrep 5222 /lib/prosody/
/lib/prosody/modules/mod_c2s.lua:       default_port = 5222;

In this setup, there's a Jitsi stack already running in parallel, using port 5222 for prosody, which is why I need to change it.

This results in jvb and jicofo being very unhappy:

jvb-1      | 2024-05-28T08:40:08.847079060Z org.jivesoftware.smack.SmackException$EndpointConnectionException: The following addresses failed: 'RFC 6120 A/AAAA Endpoint + [xmpp.meet.jitsi:5223] (xmpp.meet.jitsi/172.31.0.2:5223)' failed because: java.net.ConnectException: Connection refused
jicofo-1   | 2024-05-28T08:40:08.852750162Z org.jivesoftware.smack.SmackException$EndpointConnectionException: The following addresses failed: 'RFC 6120 A/AAAA Endpoint + [xmpp.meet.jitsi:5223] (xmpp.meet.jitsi/172.31.0.2:5223)' failed because: java.net.ConnectException: Connection refused

Am I doing anything wrong, or is it really a hardcoded value for 5222/tcp?

jmallach avatar May 28 '24 08:05 jmallach

Ops, that might need to be templated, do you want to send a PR?

saghul avatar May 28 '24 08:05 saghul

I would have, but where on earth does the prosody container come from? I have not been able to find it. :sweat_smile: Gràcies!

jmallach avatar May 28 '24 08:05 jmallach

Here it is: https://github.com/jitsi/docker-jitsi-meet/tree/master/prosody

saghul avatar May 28 '24 08:05 saghul

Ah, sorry, I knew about this after all: so I just need to template under rootfs/lib/prosody, I assume.

jmallach avatar May 28 '24 09:05 jmallach

Correct!

saghul avatar May 28 '24 11:05 saghul

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Mar 15 '25 02:03 github-actions[bot]