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

Hardcoded prosody port in mod_c2s.lua #1812

Open
jmallach opened this issue May 28, 2024 · 5 comments
Open

Hardcoded prosody port in mod_c2s.lua #1812

jmallach opened this issue May 28, 2024 · 5 comments

Comments

@jmallach
Copy link

jmallach commented May 28, 2024

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?

@saghul
Copy link
Member

saghul commented May 28, 2024

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

@jmallach
Copy link
Author

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

@saghul
Copy link
Member

saghul commented May 28, 2024

@jmallach
Copy link
Author

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

@saghul
Copy link
Member

saghul commented May 28, 2024

Correct!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants