itzg / mc-monitor

Monitor the status of Minecraft servers and provides Prometheus exporter and Influx line protocol output
MIT License
234 stars 30 forks source link

mc-pinger dependency supports Timeouts #3

Open Herobone opened 3 years ago

Herobone commented 3 years ago

The mc-pinger dependency has been updated and now supports Timeouts and Contexts. Maybe have a look at it, so that the java status command can fail after a certain time. Currently the command will try to connect to the server indefinitely even if the server is offline.

If you don't have time I could also implement the feature and open a pull request. Just let me know if you would like to support this feature

itzg commented 3 years ago

Thanks for adding that over there. Yes, a PR to integrate that in here would be great.