AuthMe / AuthMeReloaded

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

The inventory packet listener performs IO requests on the main server thread #2307

Open Rollczi opened 3 years ago

Rollczi commented 3 years ago

What behaviour is observed:

freezing the main server thread

What behaviour is expected:

not freezing the main server thread

Steps/models to reproduce:

It is possible that logging in, but the situation does not apply to every login

Plugin list:

AdvancedBan, AsyncWorldEdit, AuthMe, BungeeGuard, Chunky, ClientStats, CombatLogX, ExploitFixer, Gatekeeper, HamsterAPI, HolographicDisplays, LuckPerms, Multiverse-Core, Orebfuscator, PL-Hide-Pro, PlaceholderAPI, PlayerParticles, ProtocolLib, ProtocolSupportPotions, ServerSelectorX, SkullAPI, SSX-Connector, TabList, Vault, ViaBackwards, ViaRewind, ViaRewind-Legacy-Support*, ViaVersion, WorldEdit, WorldGuard

Environment description

Bungeecord network & MySql

AuthMe build number:

AuthMeReloaded v5.6.0-SNAPSHOT (build: 2448)

image image

Error Log:

https://pastebin.com/srJR4Yfw

Configuration:

-

games647 commented 3 years ago

Please post your configuration file

sgdc3 commented 3 years ago

@games647 this a confirmed issue, look at

https://github.com/AuthMe/AuthMeReloaded/blob/master/src/main/java/fr/xephi/authme/listener/protocollib/InventoryPacketAdapter.java#L67

https://github.com/AuthMe/AuthMeReloaded/blob/master/src/main/java/fr/xephi/authme/listener/protocollib/InventoryPacketAdapter.java#L86

https://github.com/AuthMe/AuthMeReloaded/blob/src/main/java/fr/xephi/authme/datasource/AbstractSqlDataSource.java#L30

sgdc3 commented 3 years ago

Also ListenerService#L99 performs a check on the main thread