search

AuthMe / AuthMeReloaded

The best authentication plugin for the Bukkit/Spigot API!
https://www.spigotmc.org/resources/authmereloaded.6269/
GNU General Public License v3.0
628 stars 516 forks source link

Purge deletes player passwords with bypass permission #1716

Open DarkFort opened 5 years ago

DarkFort commented 5 years ago

AuthMe deletes the passwords of players who have authme.bypasspurge permission. I tried all versions of AuthMe, the problem was not solved. Latest LuckPerms

MrMIStakee commented 5 years ago

The same problem!

/authme purge - removes players with the right “authme.bypasspurge”

I am using the latest version of LuckPerms

andrey433 commented 5 years ago

The same problem!

elsol42 commented 5 years ago

I tried to solve this problem for 4 days, I thought it was just my problem. o0omg2ixnxi

Diarg0ne commented 5 years ago

Yes, it doesn't work with Luckperms.

MrLaco commented 5 years ago

Unfortunatelly, I have the same problem too. What can I do to solve it?

ProblemsSender commented 5 years ago

I have the same problem but not when purging the player have bypass.purge, It occur even that player don't have bypass.purge, the password of that player had is purged, but not with item they have

sgdc3 commented 5 years ago

yeah purging is still broken with luckperms, we have to figure out how to fix it properly

ProblemsSender commented 5 years ago

Nah, actually I'm using Pex, it means this bug is pretty old... so it must from AuthMe conflict with itself function, or with Essentials? We should test it to find out what is the problem.

sgdc3 commented 5 years ago

@ProblemsSender i think your is a different issue

ljacqu commented 5 years ago

Relates to #1544

ProblemsSender commented 5 years ago

@sgdc3 Pex is just same as Luckperm as a permissions manager. But, Pex is currently old, my logic said this old plugin would have some bug and that is Pex problem, but however we still met the same bug as luckperm, so I think it's authme conflict with itself, that's all

ProblemsSender commented 5 years ago

But there's a slight chance, that this purge actually try to purge, but however it conflict with minecraft version? I cannot tell since I'm not the developer, it's just logic by the way...

sgdc3 commented 5 years ago

Pex has a different issue, the LuckPerms issue is that it is fully async and it has to load data of offline players! AuthMe purge system expects sync permission lookups, but when we ask LP if the user has the permission the data isn't loaded yet!

sgdc3 commented 5 years ago

@ljacqu the only way to fix that is to rewrite the purge system to support async permission lookups