FastAsyncWorldEdit

FastAsyncWorldEdit

152k Downloads

WG player left NOTIFY region whenever someone disconnects.

mibby opened this issue · 6 comments

commented

Server Implementation

Paper

Server Version

1.19.2

Describe the bug

Whenever someone disconnects, FAWE sends out a WG NOTIFY region message.

To Reproduce

Everything clean and fresh default generated. There are no regions called NOTIFY or any regions which make use of any notify flags.

Expected behaviour

No FAWE/WG notification whenever someone disconnects. While I don't normally hand out OP in my main environment, I do make use of permission wildcards in LuckPerms for staffing groups which seems to trigger the same message occurring on disconnect of anyone.

Screenshots / Videos

link

Error log (if applicable)

No response

Fawe Debugpaste

https://athion.net/ISPaster/paste/view/4ca619191fb84b13bb443cb126ab0e8a

Fawe Version

FastAsyncWorldEdit version 2.4.10-SNAPSHOT-304;1aa213f

Checklist

Anything else?

Maybe caused by this change upstream? EngineHub/WorldGuard@bd1d772

Introduces uninitialize for sessions that is called when the player disconnects or the server is shut down. Currently handlers don't get any notifications when the mentioned events occur and are unable to restore any player state they are holding to.

For example, this can be seen when entering to a region in survival with a game-mode creative flag, disconnecting and waiting for the session to timeout. When connecting back to the server and walking out of the region you can see that the player is not switched back to survival as they normally would be.

commented

Based on EngineHub/WorldGuard#1966 this behaviour seems to be wanted. No problem with FAWE here.

commented

Same issue at me. So irritating.

commented

Are you able to replicate the issue using WorldGuard and "normal" WorldEdit? I can't seem to find any connection between FAWE and the named issue.
btw, the region does not have to be called NOTIFY, the NOTIFY is hardcoded https://github.com/EngineHub/WorldGuard/blob/master/worldguard-core/src/main/java/com/sk89q/worldguard/session/handler/NotifyExitFlag.java#L59

commented

I just downgraded my WG and now the problem is gone.

commented

Quite an odd change because it occurs without any flags set.

We don't want to change the current behavior because it's a breaking change. Some people might have set the flag to false and want to get notification messages.

And changing anything is not needed, just unset the flag.

@Joo200 This is simply not true. I don't use the flags and the notification occurs. The new change made is breaking prior behavior already.

For 7.1.0-Snapshot: that version is not stable and shouldn't be used in a productive environment.

Unfortunately this commit is essential from the snapshot builds to fix commands not working EngineHub/WorldGuard@f1f1e8a which public release doesn't contain. So downgrading is not an option.

commented

Use the 7.0.x branch, not the master branch.
https://builds.enginehub.org/job/worldguard