Skip to content
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

Nag waning for using System.out/err.print. #30

Closed
7368697661 opened this issue Aug 4, 2021 · 1 comment
Closed

Nag waning for using System.out/err.print. #30

7368697661 opened this issue Aug 4, 2021 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@7368697661
Copy link

Hey!

Loving the plugin so far (seriously, the portals are a new server staple for me, couldn't play without them), however, is there any way to quiet the log/console spam from the following warning? It sends every time a player sends a chat message. Not the end of the world, but somewhat obnoxious due to cluttering the console/log.

[Server] WARN Nag author(s): '[oddlama]' of 'vane-admin' about their usage of System.out/err.print. Please use your plugin's logger instead (JavaPlugin#getLogger).

Thanks!

@oddlama
Copy link
Owner

oddlama commented Aug 4, 2021

I'm glad you like it!

Thanks for bringing this to my attention. At first I wasn't sure what warning you mean, but it seems that PaperMC has added this just 3 days ago with a patch on the 2nd of August.

If I understand PaperMC's patch correctly, you should be able to silence the warning for the time being by appending -Dio.papermc.paper.suppress.sout.nags=true to your parameters when starting paper, like this:

java -Dio.papermc.paper.suppress.sout.nags=true -jar paper.jar

I'd argue it's a good warning, but in this case it's rather unfortunate. Let me elaborate:

Usually the logger should be used to ensure that plugin messages look like this: "[pluginname] The message", which is good.
I suspect you are getting the message every time a player says something and it is logged to the console, because this is the only place I specifically used System.out.println so that chat messages can have the "[chat]" tag instead of "[vane-admin]" which would be awkward.

Of course this should be fixed properly on my side, so I guess I'll have to come up with something. But as I'm busy the next two months I won't have time to do so now. I'll leave this issue open until I get around to fix it, and I hope you can use the workaround above until then.

@oddlama oddlama added bug Something isn't working good first issue Good for newcomers labels Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants