Skip to content

feature: kick player on /logout command#231

Open
epilepticseizureee wants to merge 1 commit intoElytrium:masterfrom
epilepticseizureee:master
Open

feature: kick player on /logout command#231
epilepticseizureee wants to merge 1 commit intoElytrium:masterfrom
epilepticseizureee:master

Conversation

@epilepticseizureee
Copy link
Copy Markdown

No description provided.

@cxvqo
Copy link
Copy Markdown

cxvqo commented Apr 24, 2026

Destroy Session !== Logout (The purpose seems to likely mainly be for disabling autologin, which can be useful if playing from a shared IP address). This should probably be under a config option set to false by default if you really need it.

@epilepticseizureee
Copy link
Copy Markdown
Author

Destroy Session !== Logout (The purpose seems to likely mainly be for disabling autologin, which can be useful if playing from a shared IP address). This should probably be under a config option set to false by default if you really need it.

Stop spouting bullshit—the function is correct.

@cxvqo
Copy link
Copy Markdown

cxvqo commented Apr 24, 2026

I do not believe it to be correct that's why I'm commenting a suggestion, do you take it as a personal threat?
Sending the message when the player gets kicked right after that looks to be very useless, you will just see the message in the kick, why send a useless chat message?

@epilepticseizureee
Copy link
Copy Markdown
Author

I do not believe it to be correct that's why I'm commenting a suggestion, do you take it as a personal threat?

Sending the message when the player gets kicked right after that looks to be very useless, you will just see the message in the kick, why send a useless chat message?

Well, I felt like doing it—what difference does it make to you? Shut your fucking mouth with your comments and stop stinking up the place.

@cxvqo
Copy link
Copy Markdown

cxvqo commented Apr 24, 2026

Your comments are much more "stinky" to this place.
It makes a difference because I am using this plugin and you have made a pull request to merge this feature into it.
I do not think the way you're doing it would suit some people (I'm sure some people prefer how it is right now). You're not the only user of this project, you know that right?

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.

2 participants