LuckPerms

LuckPerms

41.4k Downloads

FakePlayers Permission Group stays on default, despite group Switch [DiscordIntegration]

CDAGaming opened this issue ยท 8 comments

commented

Using DiscordIntegration as an example, even If I set it's UUID's group to, say a group called admin, whenever you try to type a command, it'll still check the default group for permissions, not the correct group.

https://mcuuid.net/?q=828653ca-0185-43d4-b26d-620a7f016be6

(I'm only able to give it perms via it's UUID, by "lp user 828653ca-0185-43d4-b26d-620a7f016be6 parent
setprimarygroup bot)

commented

Related to #362, except DiscordIntegration is a Forge Server-Side Mod...

commented

I don't understand what the problem is?

Can you give an example of the behaviour you're currently seeing, and what you expect to be happening?

commented

I have created a group called 'bot and have assigned it permissions like "minecraft.command.*" or "minecraft.command.stop". The Issue is that despite this and setting the FakeUser to the bot group (via UUID, since It wont let me put in it's name), when i'd try to run !stop within discord to feed /stop into minecraft, the command returns with Insufficient permissions, even though it's set on the bot group.

commented

LuckPerms won't handle permission checks for fake players, unless Sponge/Forge does anything special to bridge the gap.

I don't know how that system works (if it even exists at all) so can't really provide much help.

commented

FakePlayer extends EntityPlayerMP
There should be NO issues checking permissions on a FakePlayer instance.
I see no where in Minecraft Forge, where it distinguish between a normal player and a FakePlayer with regards to commands.

commented

LuckPerms is not a forge mod, so my understanding is that:

if you're using LP on Forge, I'm assuming it's via something like Sponge or Cauldron. Afaik it's the responsibility of those projects to bridge the gap.

Regarding the replies on the other ticket you made:

#834
They say it would be up to you to bridge the gap :P

I didn't say that. I said it's probably something Sponge could (should??) be doing.

commented

As far as I know this isn't a problem LP can solve.

There isn't any official implementation of LP for Forge servers - if you're using LP on Forge, I'm assuming it's via something like Sponge or Cauldron. Afaik it's the responsibility of those projects to bridge the gap.

commented

To clarify:

I suggest opening a ticket on the SpongeForge repository to see if support for FakePlayer permission checks can be added in Sponge.