Giter Site home page Giter Site logo

Comments (15)

eclair4151 avatar eclair4151 commented on July 17, 2024

Hmm interesting. Taking a look now.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

As far as I’m aware, the enable-query is disabled on the server. But I can double check with the server owner.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

Ah, my bad. The enable query is set to true. Just confirmed with the server owner.

Edit: I’ve updated the post to more accuratly describe the issue.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

I’ve updated the imgur link to include a screenshot with a player online.

from mc-status-widget-for-minecraft.

eclair4151 avatar eclair4151 commented on July 17, 2024

So it looks like a bug in the API i am using: https://api.mcsrvstat.us/1/waifu-technologies.moe
for some reason it is returning that enable-query is turned off on the server, but returning the list of names anyway. I have contacted the API owner to see if i can get some more info on whats going on.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

Sounds great, let's hope we can get it resolved :)
It's a small issue anyway, but still something that needs a fix.

from mc-status-widget-for-minecraft.

eclair4151 avatar eclair4151 commented on July 17, 2024

Can you by any chance ask the server owner if in his server.properties list he has
query.port=25565

if not can he add it and if it is set to a different port set it to 25565 and restart the server?
Thanks!

from mc-status-widget-for-minecraft.

DragoonX6 avatar DragoonX6 commented on July 17, 2024

Server owner here, is that TCP or UDP?

from mc-status-widget-for-minecraft.

eclair4151 avatar eclair4151 commented on July 17, 2024

actually both. The server status is pulled over the TCP ping protocol
http://wiki.vg/Protocol

and the user list is pulled over UDP using the query protocol
http://wiki.vg/Query

from mc-status-widget-for-minecraft.

DragoonX6 avatar DragoonX6 commented on July 17, 2024

Oh, in that case just opening up the UDP side of it in my firewall should do the trick.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

That seems to have fixed the issue. An error regarding that would be nice though ;)

from mc-status-widget-for-minecraft.

DragoonX6 avatar DragoonX6 commented on July 17, 2024

It's pretty hard to accurately determine if something isn't listening on the port, if it's null routed, or just not open in the firewall. For example my firewall is configured to just drop the packet, instead of rejecting it.

The easiest way is to just list multiple possible reasons and let users go over them before having to open a bug report.

from mc-status-widget-for-minecraft.

FeikoJoosten avatar FeikoJoosten commented on July 17, 2024

That is indeed an option.
But as he was able to still retrieve the player names, while still giving out an error that enable-query needed to be true, it should be needed to showcase the error regarding setting enable-query to true anyway. Simply only look at the array of player names, that should already be sufficient afaik.

from mc-status-widget-for-minecraft.

eclair4151 avatar eclair4151 commented on July 17, 2024

Yea this is a bit complicated. Feiko is right. it appears UDP is not actually required to get the list of players. The first thing i am going to do is check the list of players and show them in the app even if query is set to false.
The problem is for servers that have UDP turned off but have no players on the server. In that state there is no way to know if enable-query is on and UDP is off(Which is fine), or if enable-query is just turned off.
I think i will update it so that in that case there is no message/error shown unless there are people on the server but we are still unable to get the list of players.

from mc-status-widget-for-minecraft.

eclair4151 avatar eclair4151 commented on July 17, 2024

fixed in commit de0332b

from mc-status-widget-for-minecraft.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.