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

When clustering is used, solves the problem of not joining the room #1521

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

abdurrahmanekr
Copy link
Contributor

Hello, when using bosh with Openfire cluster, could not join the MUC rooms. This PR for controlling occupantsByNickname and occupantsByFullJID variables.

openfire-cluster-behaves-unexpectedly-on-the-muc

@IgniteRealtime-Bot
Copy link

This pull request has been mentioned on Ignite Realtime Community Forums. There might be relevant details there:

https://discourse.igniterealtime.org/t/openfire-cluster-behaves-unexpectedly-on-the-muc/86520/10

@speedy01
Copy link
Contributor

speedy01 commented Nov 7, 2019

@GregDThomas @guusdk please review! Thanks

@GregDThomas
Copy link
Contributor

I'm not sure I can add anything useful to this.

But that doesn't normally stop me, so:

The Discourse thread seems to indicate that this only happens with BOSH (possibly also WebSocket?) connections, not TCP connections.

Suggest to me that whilst the code fixes the underlying problem of a NPE, the fix is actually masking some issue with the BOSH connectivity.

@akrherz akrherz added this to the 4.6.0 milestone Oct 13, 2020
@akrherz
Copy link
Member

akrherz commented Oct 13, 2020

@abdurrahmanekr , sorry this PR has languished. Are you able to rebase it?

@abdurrahmanekr
Copy link
Contributor Author

Hi @akrherz, when I look at the last changes, I see that the same logic is used. That's why I tried to write the same code while solving the conflict. As mentioned in discouse, I had this problem only when using BOSH (http-bind). We are currently using this code in our system and I have to compile openfire's core accordingly. Because people can't join groups while using bosh (with cluster).

@akrherz akrherz modified the milestones: 4.6.0, 4.7.0 Oct 16, 2020
@akrherz akrherz removed this from the 4.7.0 milestone Jan 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants