Skip to content

Conversation

TheMiningTeamYT
Copy link

When using Velocity, I have been frustrated by the lack of fine control over ping passthrough, specifically the lack of support for passing through favicons without passing through the whole ping request. I've also seen people on line asking how to pass through favicons, so I decided to add the ability to control which aspects of server list ping requests are passed through from the back end server, and which aspects aren't, through 5 new boolean configuration options in the config file.
Please don't judge my code too harshly, this is my first time writing Java, but it does compile and Checkstyle is okay with it.
(If you're wondering, yes this is a duplicate pull request -- sorry about that, I was told to make a new pull request from a different branch. However, the code has been updated since I originally opened the old pull request. Not since I closed the previous pull request though.)

@TheMiningTeamYT
Copy link
Author

I cleaned up my commits (specifically I removed the times I merged from upstream from the ping-passthrough-dev branch), so there is actually a reason for this new pull request now. I hope you will reevaluate my pull request in light of this.

@TheMiningTeamYT
Copy link
Author

I've now gone through the trouble of resolving the merge conflict (so I can have an up-to-date version of Velocity build from my fork), merged ping-passthrough-dev, and tested the result. It builds, and, although I haven't thoroughly tested it, it does work.
I believe this is now in a merge-able state. It functions, maintains compatibility with existing configurations, and brings a useful new feature to Velocity: more granular control of ping passthrough. (I made this fork out of necessity in the first place, and I'm sure others will find this feature useful too). Plus, I've already gone through the trouble of resolving the merge conflicts, and running the build tests (when I merged the branch on my fork). All you have to do is click the button.

* preliminary cleanup of plugin message channel handling

* Fix spot

* Appease checkstyle

* Fix tests

* Add more options for ping passthrough configuration. (Merge ping-passthrough-dev)

* Added 'ALLBUTVERSION' option for ping passthrough.

* Trying to get the GitHub build action to run

* Added more configuration options for ping passthrough.

* Updated default velocity.toml

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Update proxy/src/main/java/com/velocitypowered/proxy/connection/util/ServerListPingHandler.java

Co-authored-by: powercas_gamer <[email protected]>

* Add support for the legacy ping passthrough.

---------

Co-authored-by: TheMiningTeamYT <[email protected]>
Co-authored-by: powercas_gamer <[email protected]>

* Use an ImmutableList Builder

* Appease checkstyle gods

* Also validate length before caring to invest time into processing

* Fix MinecraftChannelIdentifier parsing to align with vanilla (PaperMC#1552)

* Removed legacy ping passthrough & added config migration

- Bumped config-version to 2.8
- Updated comments and grammar
- Removed legacy ping passthrough in the config

* Cleaned up code to match code style

---------

Co-authored-by: Shane Freeder <[email protected]>
Co-authored-by: Loganius <[email protected]>
Co-authored-by: TheMiningTeamYT <[email protected]>
Co-authored-by: powercas_gamer <[email protected]>
Co-authored-by: booky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants