Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stacktrace when joining a server with interserver networks enabled #305

Open
Thorinwasher opened this issue Dec 31, 2023 · 0 comments
Open
Labels
issue A problem that – while inconvenient – does not impede core functions of the plugin REWRITE This task/issue only applies to the rewritten version of the plugin (currently in ALPHA)

Comments

@Thorinwasher
Copy link

Bug Description

Stacktrace when joining a server with interserver networks enabled

Reproduction Steps

Start server1, build a portal with the I flag. After you have built the I portal, start server2 and join it.

Desired Behaviour

There should not have been a stacktrace

Observed Behaviour

After a player joins, the following stack trace is shown.

[11:08:25 WARN]: [Stargate] org.sgrewritten.stargate.exception.name.NameConflictException : portal of name 'inter1' already exist in network 'inter'
[11:08:25 WARN]: [Stargate]      at Stargate-1.0.0.14-ALPHA.jar//org.sgrewritten.stargate.network.StargateNetwork.addPortal(StargateNetwork.java:149)
[11:08:25 WARN]: [Stargate]      at Stargate-1.0.0.14-ALPHA.jar//org.sgrewritten.stargate.manager.StargateBungeeManager.portalAddOrRemove(StargateBungeeManager.java:108)
[11:08:25 WARN]: [Stargate]      at Stargate-1.0.0.14-ALPHA.jar//org.sgrewritten.stargate.manager.StargateBungeeManager.updateNetwork(StargateBungeeManager.java:54)
[11:08:25 WARN]: [Stargate]      at Stargate-1.0.0.14-ALPHA.jar//org.sgrewritten.stargate.listener.StargateBungeePluginMessageListener.onPluginMessageReceived(StargateBungeePluginMessageListener.java:101)
[11:08:25 WARN]: [Stargate]      at org.bukkit.plugin.messaging.StandardMessenger.dispatchIncomingMessage(StandardMessenger.java:455)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.network.ServerCommonPacketListenerImpl.a(ServerCommonPacketListenerImpl.java:163)
[11:08:25 WARN]: [Stargate]      at net.minecraft.network.protocol.common.ServerboundCustomPayloadPacket.a(ServerboundCustomPayloadPacket.java:46)
[11:08:25 WARN]: [Stargate]      at net.minecraft.network.protocol.common.ServerboundCustomPayloadPacket.a(ServerboundCustomPayloadPacket.java:12)
[11:08:25 WARN]: [Stargate]      at net.minecraft.network.protocol.PlayerConnectionUtils.lambda$ensureRunningOnSameThread$0(PlayerConnectionUtils.java:53)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.TickTask.run(TickTask.java:18)
[11:08:25 WARN]: [Stargate]      at net.minecraft.util.thread.IAsyncTaskHandler.d(IAsyncTaskHandler.java:153)
[11:08:25 WARN]: [Stargate]      at net.minecraft.util.thread.IAsyncTaskHandlerReentrant.d(IAsyncTaskHandlerReentrant.java:24)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.b(MinecraftServer.java:1324)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.d(MinecraftServer.java:193)
[11:08:25 WARN]: [Stargate]      at net.minecraft.util.thread.IAsyncTaskHandler.x(IAsyncTaskHandler.java:126)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.bg(MinecraftServer.java:1301)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.x(MinecraftServer.java:1294)
[11:08:25 WARN]: [Stargate]      at net.minecraft.util.thread.IAsyncTaskHandler.c(IAsyncTaskHandler.java:136)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.a(MinecraftServer.java:1371)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:1156)
[11:08:25 WARN]: [Stargate]      at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:315)
[11:08:25 WARN]: [Stargate]      at java.base/java.lang.Thread.run(Unknown Source)

Trace Logs

The sg trace command may be run through console or in-game.

Other Information

server1 saves the portal to the database, server2 loads the portal from the database, but also later on receives a portal add message whenever a player joins. This is the cause of the issue.

A valid fix for this would be to hide any exceptions like this. But it might be good to wait with this until a release

@Thorinwasher Thorinwasher added the REWRITE This task/issue only applies to the rewritten version of the plugin (currently in ALPHA) label Dec 31, 2023
@Thorinwasher Thorinwasher added the issue A problem that – while inconvenient – does not impede core functions of the plugin label Jan 6, 2024
Thorinwasher added a commit that referenced this issue Jan 6, 2024
Relates to issue #305 and #313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue A problem that – while inconvenient – does not impede core functions of the plugin REWRITE This task/issue only applies to the rewritten version of the plugin (currently in ALPHA)
Projects
None yet
Development

No branches or pull requests

1 participant