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

automatic parameterized registration of plugin derived permission sets #2427

Closed
PaxJaromeMalues opened this issue May 10, 2022 · 1 comment
Closed
Labels
status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage

Comments

@PaxJaromeMalues
Copy link

PaxJaromeMalues commented May 10, 2022

Major SpongeAPI version

1.12.2 7.4.7 + if possible
otherwise 8+

Is this likely to be a breaking change?

I don't know

What are you requesting?

Currently, the only way to get a (semi) complete list of permissions is to create a dummy user and assign all available permissions to it manually or running lp tree (in case of luckperms) after the server has been up for a long time.

It would be nice if (maybe also for API7.4) a functionality could be introduced, that automatically fetches all non argument based permission nodes and make them available as a list-file, callrequest or dbentry to the server admins.

@PaxJaromeMalues PaxJaromeMalues added the status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage label May 10, 2022
@dualspiral
Copy link
Contributor

PermissionService#getDescriptions() already exists, as does the ability to register them: https://jd.spongepowered.org/spongeapi/8.0.0/org/spongepowered/api/service/permission/PermissionService.html#newDescriptionBuilder(org.spongepowered.plugin.PluginContainer). The caveat is that plugins need to actually register their permissions - but ultimately it is up to the plugin devs to register them. They don't have to.

We cannot magically detect all permissions that plugins will use before they use them - so unless you can encourage all plugins to register all their permissions, I'm afraid you're out of luck.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage
Projects
None yet
Development

No branches or pull requests

2 participants