Registred User can join private groups

[Bilou]

They joining by clicking on join button on group page, there is no joining request for group's owner.

[Bilou]

Maybe this error is the problem :

WebSocket connection to 'ws://xxx.xxx.fr/socket.io/?EIO=3&transport=websocket&sid=G1SwCsUk0KFun1IOAABK' failed: Error during WebSocket handshake: Unexpected response code: 400

I'm running on Apache 2.4 whit proxypass and proxypassreverse.

Edit : I fixed this error and doesn't fixe the problem.

[Bilou]

Hello
I tried on this forum (community.nodebb.org) to join the group "gamers" which is private and it's work whereas i tried "staff" group and my invitation is pending.

Please help me

Bilou.

[Kowlin]

@baris I can confirm that this is a real bug. I've tested this on both 0.9.0 and Master, and I could join a private group with a normal user account. As shown in my GIF below.

Imgur

https://github.com/NodeBB/NodeBB/issues/3861

AsBilou created this issue in NodeBB/NodeBB

closed Registred User can join private groups #3861

[baris]

I have replied on the issue, check the setting in the ACP.

[Bilou]

Private group was check by default but i had to disabled private group -> save -> enabled private group -> save to activate the verification.

[nicoechaniz]

@baris I installed NodeBB on a debian server just yesterday (cloned nodebb 0.9 branch) and I cannot make groups private. Tried unsetting ande re-setting the private option in ACP, still no-go. The group says private in the frontend but people can still join freely.

[pichalite]

@nicoechaniz make sure you have this option enabled in ACP -> Settings -> Group

[baris]

This was caused by old groups not having the private field in the database. I have fixed it on master, it only affects old groups since newly created groups have the private field. You can probably go into the group that has the problem and uncheck private > save > check private > save to fix the problem.

[nicoechaniz]

@pichalite that was it. It's a bit confusing... Maybe the private option for specific groups should be grayed out if the general option for allowing private groups is disabled, to make this clearer.