status_before and onlines_before will be overwritten when the status API returns multiple servers

k5342 / tskserver-notification-bot

Minecraft server notification bot. Supports Twitter and Discord.

0 stars 0 forks source link

status_before and onlines_before will be overwritten when the status API returns multiple servers #8

Open k5342 opened 3 years ago

k5342 commented 3 years ago

Please use when that the status API return only a single server's status. Otherwise it meets this bug.

The status API is here; https://github.com/k5342/minecraft-status-check-http-api

k5342 commented 3 years ago

Workaround has been merged. But currently we assume that server name must be "tskserver". This hard-coded test suffers from portability. Issue #9 will continue to solve this workaround.

k5342 commented 3 years ago

We need to move two variables into ServerInfo.infos to completely avoid them. Reopen this issue to track this problem.