Not compatible with LimboAPI

[ghost]

Expected Behavior

LimboAPI doesn't work on Velocity CTD.

Actual Behavior

LimboAPI should work on Velocity CTD.

Steps to Reproduce

1. Install LimboAPI.
2. It won't start.

Plugin List

LimboAPI

Velocity Version

3fe97fc

Additional Information

Elytrium/LimboAPI#163

[R00tB33rMan]

Expected Behavior

LimboAPI doesn't work on Velocity CTD.

Actual Behavior

LimboAPI should work on Velocity CTD.

Steps to Reproduce

1. Install LimboAPI.

2. It won't start.

Plugin List

LimboAPI

Velocity Version

3fe97fc

Additional Information

Elytrium/LimboAPI#163

I've provided a comment on the LimboAPI repository. Once Velocity merges this already-completed PR (based on extensive testing), LimboAPI will no longer be compatible with upstream Velocity. I'll leave this open for the time being.

[ghost]

Velocity merging this might take months, while this will be broken @R00tB33rMan

[ghost]

To follow up, I want to say that any upstream changes in Velocity lead to Limbo plugins being updated very quickly, such as here or here. This won't be an issue when upstream merges this, this is an issue regarding upstream backwards compatibility right now, even though packet registration is not an API.

[Justiks1337]

this broke my plugins, that depend on velocity, this very importart for my server smp and four black pigs. Author pls fix that

[R00tB33rMan]

I'll look into reverting this change tomorrow (the 24th) despite its effectiveness. Hopefully, Velocity intends to merge this change sometime soon.

[R00tB33rMan]

Expected Behavior

LimboAPI doesn't work on Velocity CTD.

Actual Behavior

LimboAPI should work on Velocity CTD.

Steps to Reproduce

1. Install LimboAPI.

2. It won't start.

Plugin List

LimboAPI

Velocity Version

3fe97fc

Additional Information

Elytrium/LimboAPI#163

Issue should be resolved in: b0794ca

[Justiks1337]

Thanks, this has been fixed for both my plugins and LimboAPI!

[R00tB33rMan]

Awesome! Thanks for the report!