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

Use Server List Ping for auto-shutdown instead of query #37

Open
vemacs opened this Issue Oct 14, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@vemacs

vemacs commented Oct 14, 2015

Query is not enabled by default, and may not be supported by alternate server implementations. Why not use the standard 1.7+ server list ping first, and then fallback to query?

Ruby: https://github.com/blockgiven/takumi-server_list_ping
Python: https://github.com/Dinnerbone/mcstatus

@Raekye

This comment has been minimized.

Member

Raekye commented Oct 14, 2015

At the time of writing 1.7 and the 1.7 server list ping were new, and it was fragile supporting old ping protocols (server doesn't respond if you don't follow its protocol, so you set a timeout, then try another protocol version). The query protocol seemed nice because it has remained unchanged

It's been over a year and probably most servers are on 1.7+ now, so I'll look into supporting that. Should be straightforward but probably low priority as I want to focus on scheduling asap

@Raekye Raekye added the enhancement label Oct 15, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment