Dinnerbone / mcstatus

A Python class for checking the status of an enabled Minecraft server
http://dinnerbone.com/minecraft/tools/status/
1.11k stars 146 forks source link

Animated MOTDs result in errors. #31

Closed dmptrluke closed 5 years ago

dmptrluke commented 9 years ago

I know it's not a proper part of the MOTD spec, but a lot of servers now are using "animated" MOTDs. As long as minecraft itself "supports" them, MCStatus should be able to at least get the first frame from an animated MOTD.

dmptrluke commented 9 years ago

Example case: Mineplex

evelynharthbrooke commented 9 years ago

Yeah, I agree with Luke. MCStatus should really be able to handle "animated" MOTDs. Because it really sucks when MCStatus can't handle them. It also breaks the mcstatus command within the CloudBot IRC bot a little bit, due to MCStatus not being able to handle MOTDs that are animated.

connorlurring commented 9 years ago

I think the issue is that the server sends an 'offline' response to the client which is what allows it to edit the message

Ryuutei commented 9 years ago

wtf ? keep it simple.

ctian1 commented 9 years ago

@Ryuutei what do you mean..

Ryuutei commented 9 years ago

Who knows what will happen with future servers, no need to add burden to mcstatus with weird modded stuff.

ctian1 commented 9 years ago

@Ryuutei but the client accepts them, so this should too. Isn't that what this is for?

benaubin commented 9 years ago

@PangeaCake Actually, animated motds are a bug in Minecraft, basically when the server responds with a broken version response, Minecraft polls the server again. In more recent versions of Minecraft (past 1.8.4), animated motds do not work, even in Vanilla Minecraft, I suggest this issue to be closed.

kevinkjt2000 commented 5 years ago

I concur; closing.