Pocketmine-mp: /whitelist on should kick non-whitelisted online players

Created on 9 Oct 2020  Â·  3Comments  Â·  Source: pmmp/PocketMine-MP

When the server whitelist is enabled, people who got into the server in time should be kicked. It does not seem reasonable to want to enable whitelist without kicking everyone already online not in the whitelist.

This changes behaviour and may be appropriate in major.next.

BC break Debugged

Most helpful comment

We already depend on this current behavior that prevent more players from
joining without kicking everyone, especially when we're whitelisting the
server in the middle of the day to host "last man standing" games/events.

I suppose there should be an optional Boolean toggle /whitelist on [kick=false] to kick existing non-whitelisted players if this were to be
implemented?

On Fri, Oct 9, 2020, 16:35 Chan Kwan Yin notifications@github.com wrote:

When the server whitelist is enabled, people who got into the server in
time should be kicked. It does not seem reasonable to want to enable
whitelist without kicking everyone already online not in the whitelist.

This changes behaviour and may be appropriate in major.next.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pmmp/PocketMine-MP/issues/3868, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHQQFF7PALQRCAQP7VKPWDSJ3DOVANCNFSM4SJY7VLA
.

All 3 comments

We already depend on this current behavior that prevent more players from
joining without kicking everyone, especially when we're whitelisting the
server in the middle of the day to host "last man standing" games/events.

I suppose there should be an optional Boolean toggle /whitelist on [kick=false] to kick existing non-whitelisted players if this were to be
implemented?

On Fri, Oct 9, 2020, 16:35 Chan Kwan Yin notifications@github.com wrote:

When the server whitelist is enabled, people who got into the server in
time should be kicked. It does not seem reasonable to want to enable
whitelist without kicking everyone already online not in the whitelist.

This changes behaviour and may be appropriate in major.next.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pmmp/PocketMine-MP/issues/3868, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHQQFF7PALQRCAQP7VKPWDSJ3DOVANCNFSM4SJY7VLA
.

If we add toggles for this, nobody will use them and it'll still be a circus of users asking why players don't go away when using /whitelist. You should implement your own logic instead of depending on the built-in whitelist.

no

Was this page helpful?
0 / 5 - 0 ratings

Related issues

nmo0ory picture nmo0ory  Â·  3Comments

SuperAdam47 picture SuperAdam47  Â·  3Comments

dktapps picture dktapps  Â·  3Comments

sergeysova picture sergeysova  Â·  3Comments

XDqrkneZ picture XDqrkneZ  Â·  3Comments