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

Plugin performed restricted action. #52

Open
SilverCory opened this issue Jul 21, 2014 · 8 comments
Open

Plugin performed restricted action. #52

SilverCory opened this issue Jul 21, 2014 · 8 comments

Comments

@SilverCory
Copy link
Contributor

I use the metrics in my plugin. And I was wondering if you could patch the Bungee plugin.

Bungeecord requests that instead of doing "new Thread()", you should use the Bungee scheduler.

Here is a stacktrace:
13:58:16 [WARNING] Plugin performed restricted action, please inform them to use proper API methods: Illegal thread group access java.security.AccessControlException: Plugin violation: Illegal thread group access at net.md_5.bungee.BungeeSecurityManager.checkRestricted(BungeeSecurityManager.java:39) at net.md_5.bungee.BungeeSecurityManager.checkAccess(BungeeSecurityManager.java:62) at java.lang.ThreadGroup.checkAccess(ThreadGroup.java:315) at java.lang.Thread.init(Thread.java:389) at java.lang.Thread.init(Thread.java:349) at java.lang.Thread.<init>(Thread.java:547) at eu.ac3_servers.dev.bvotifier.bungee.Metrics.start(Metrics.java:205) at eu.ac3_servers.dev.bvotifier.bungee.BVotifier.onEnable(BVotifier.java:69) at net.md_5.bungee.api.plugin.PluginManager.enablePlugins(PluginManager.java:207) at net.md_5.bungee.BungeeCord.start(BungeeCord.java:230) at net.md_5.bungee.Bootstrap.main(Bootstrap.java:65)

Link to the issue: https://bitbucket.org/acecheesecr14/bvotifier/issue/1/javasecurityaccesscontrolexception-plugin

@turt2live
Copy link

I would argue that the Bungee commit that implemented this is a dumb change and should be resolved by them for their idiotic waste of time.

@SilverCory
Copy link
Contributor Author

Okay, well it's there and I doubt it's going to be removed for one java class.

Other plugins adapt, if this plugin doesn't then it's a feature I can't use and mcstats will be useless to me!

@XxCoolGamesxX
Copy link

More info of the error can look on the Minecrell/ServerListPlus#23 of the Bungee Plugin Metrics.

@stephan-gh
Copy link

@XxCoolGamesxX Well I use my own Plugin-Metrics implementation on BungeeCord so you can't use the fix of my issue for this one.
The BungeeCord version here should be rewritten to use the BungeeCord scheduler (similar to the Bukkit version) instead of using threads directly. (Additionally #40 should be probably merged..)

For anyone interested, here is my own Plugin-Metrics implementation for BungeeCord: Minecrell/BungeeMetrics - it works a bit different to the versions here but won't warn about the thread creations.

@XxCoolGamesxX
Copy link

@Minecrell Thanks for the info :P It help the devs. :P

@SilverCory
Copy link
Contributor Author

@Minecrell brilliant. You beast ;P

@stephan-gh
Copy link

@acecheese You should keep it open until it is fixed anyway. ;)

@SilverCory
Copy link
Contributor Author

@Minecrell Point!

@SilverCory SilverCory reopened this Jul 28, 2014
@RafVDL RafVDL mentioned this issue Jul 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants