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

Reconnecting to XMPP server if MucClient failed to join the MUCs. #181

Closed
wants to merge 1 commit into from
Closed

Conversation

veratiz
Copy link

@veratiz veratiz commented Sep 29, 2022

@jitsi-jenkins
Copy link

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

@bgrozev
Copy link
Member

bgrozev commented Sep 29, 2022

Why is this necessary? The XMPP connection was successful and the retry code will attempt to re-join the MUCs (without disconnecting)

@veratiz
Copy link
Author

veratiz commented Sep 29, 2022

If xmppConnection successfully connected and logged in, but failed when join, re-join will never happen.
For example, when Prosody is rebooted and jicofo not yet reconnected, Jibri throws this exception:
2021-11-22 14:59:16.276 WARNING: [41] org.jitsi.xmpp.mucclient.MucClient.log() Failed to join the MUCs. org.jivesoftware.smack.XMPPException$XMPPErrorException: XMPP error reply received from jibribrewery@local/jibri-instanse-321207090: XMPPError: not-allowed - cancel [Room creation is restricted]

@damencho
Copy link
Member

2021-11-22 14:59:16.276 WARNING: [41] org.jitsi.xmpp.mucclient.MucClient.log() Failed to join the MUCs. org.jivesoftware.smack.XMPPException$XMPPErrorException: XMPP error reply received from jibribrewery@local/jibri-instanse-321207090: XMPPError: not-allowed - cancel [Room creation is restricted]

This is jibri trying to create the brewery room before jicofo. There is no way for synchronizing when jicofo and jibri will do the attempt. To mitigate that problem you can put the jibri user to be admin and it will not see the error and will be allowed to create the room.

@veratiz veratiz closed this by deleting the head repository Jun 10, 2024
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 this pull request may close these issues.

4 participants