Curios API (Forge/NeoForge)

Curios API (Forge/NeoForge)

151M Downloads

[Bug]: Possible Incompatibility with LuckPerms

hypherionmc opened this issue · 4 comments

commented

Minecraft Version

1.20.1

What happened?

Not really sure if this is a curios bug, or LuckPerms bug, but reporting to both, just in case.

Basically, when you have a mod installed that uses the LuckPerms getCachedData() api, it will break when the player dies, causing the player to become completely unresponsive, unable to completely respawn

One of my users came across this, and I can confirm the bug.

Original report: hypherionmc/sdlink#136

How do you trigger this bug?

  1. Install Curios and LuckPerms
  2. Install CraterLib, Simple Discord Link (No need to configure either, it's enough to just install them)
  3. Let the player die and respawn. If you respawn, you will be unable to do anything, and if you relog, you will be on the death screen again, with the error linked below printed to the logs.

Loader

Forge

Loader Version

47.3.12

Mod Version

5.11.0+1.20.1

Relevant Log Outputs

https://pastebin.com/4wsRNRYF

commented

I'm having some difficulty trying to debug LuckPerms in a development environment, so unfortunately I couldn't be as thorough as I wanted in my investigation of this issue. However, from what I can tell, this seems to be an issue that needs to be fixed on LuckPerms's end.

The log output doesn't show Curios in the stacktrace so I'm a bit confused what could cause this issue to arise from Curios's end, but the failure to load player data seems to happen purely with LuckPerms's code so I don't believe there's a possible workaround I could do on my end to address this either.

commented

Hey

Thanks for the feedback. About a week after this bug report, we got another almost identical one. This time, removing both curios and my mod did not fix it, so it's definitely a core bug in LuckPerms itself.

The other report was because of Styled Chat. I don't have a lot of hope that it will be fixed on their side (since they drop support for older mc as soon as they port) 😢

commented

Ah, I see. That's unfortunate then, if it's not likely to be fixed. But if that's the case, I'll go ahead and close this issue since it doesn't seem to be caused by Curios. However, feel free to let me know if there's any update on this issue that requires another look at it.