-
-
Notifications
You must be signed in to change notification settings - Fork 11
Description
Describe the bug
When enable-status in server.properties is false, Failed to connect is displayed.
To Reproduce
- Set up Velocity & Catalyst.
- Set
enable-statusinserver.propertiesto false. - Failed to connect.
Expected behavior
The player will be successfully connected to the backend server.
Desktop (please complete the following information):
- Velocity Version: 1.1.5
- Catalyst Version: 0.3.2
- Additional Plugins installed on velocity: Anvil 0.3.0
Additional context
This issue does not occur with Velocity alone, so you may want to mark it as a bug. If this is not fixed, it might be a good idea to provide a FAQ.
This issue is a bit tricky for the following reasons:
- No errors will be output to the console at all.
- Console of the backend server will log as if the player connected for a moment and then immediately disconnected.
Almost certainly, the following checks are causing this issue.
https://github.com/AnvilPowered/Catalyst/blob/088793a6de30fe200169ef365a1a8a319c127492/catalyst-velocity/src/main/java/org/anvilpowered/catalyst/velocity/listener/VelocityListener.java#L119-L120
However, simply removing this check does not seem like a good idea. I don't understand Java and Velocity, so I can't come up with a good solution.
