-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Removed console ping message #1484
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
Conversation
Why? |
Because I find that it causes spam in the console, and if a significant amount of players are pinging the server it clutters the console up and makes it hard to find the things that are actually important |
I think this was denied in a previous issue |
not sure. but good. idea try code.. new option on cfg put some case else etc in code. i think if optional and look good idea if you hate take some control.. who hell visit your network |
@lockerecca Yeah, this was just a quick little edit. I am working on something more permanent now :) |
maybe make it configurable instead of straight up removing? |
Another possiblity is just using gits power and have a custom fork you rebase on the current bungee version, if this get declined. |
+1 |
without a config option this will not get accepted |
Is there a need for that log line? |
@njb-said As far as I am aware, the only purpose could be debugging except for that it just causes unnecessary console spam |
ff0644c
to
715ec07
Compare
No description provided.