Modern IP Forwarding... in vanilla?

That’s right. VanillaCord’s active branch now has velocity modern IP forwarding support, which means you can now patch it into the latest snapshots or go all the way back to 1.13 if you’re into that sort of thing.

…And now for the real reason this thread is in this section. Can anyone look over the code to make sure there’s no super obvious (security) problems?

Also it would be nice if the devs could let us know a little bit more about the intended implementation of some of these fields. In my case, I had the following questions:

  • How strict should Protocol Version checking be?
  • What should we use the Username field for if the Login Start packet has already delivered it? (Would their values ever differ?)

If I guessed wrongly in the code, let me know.