Velocity
Velocity copied to clipboard
Use the default Minecraft port instead of custom one
Currently Velocity uses the custom port 25577 in its default config which does not make any sense for 99.9% of deployments and has caused confusion and unnecessary support requests time and time again without any obvious benefit as usually you want the proxy to be reachable instead of your normal servers which wouldn't even be usable anymore as soon as you enable proxy support on the Minecraft server. (I kinda assume this was originally just copied from Bungee's default being 25577)
While a non-standard port can be useful for testing in some situations when already having a Minecraft server running in others it might actually be decremental (e.g. because you need to add another server into your server list instead of just using one localhost with port 25565). For cases where this behaviour is desired a custom port can always be specified in the config anyways.
Existing setups should not get influenced by this change as they already have a config with the value and even in a case where a container image is used which relies on a specific port this change should not cause any issues as the config should either be persisted or be part of the container image if it is setup right. (e.g. like how the Velocity Pterodactyl egg does it)
And if there was discussion/issues/PRs open for that before then I'm sorry but I couldn't find them...
bungee does the same so idk they probably wanted to keep it the same as bungee.
I generally hit my config over to the default port locally, the only reason for being a non-default port that I can see for quick testing, but, you generally need to configure a bunch of other stuff anyways
I'm generally for merging this, but, need to do it in the next version, not here