Extended Information-forwarding support#663
Conversation
| Preconditions.checkNotNull( desc.getName(), "Plugin from %s has no name", file ); | ||
| Preconditions.checkNotNull( desc.getMain(), "Plugin from %s has no main", file ); | ||
|
|
||
| + // Waterfall start: Forwarding rework |
There was a problem hiding this comment.
I err on this, no idea if others have any other thoughts on this, I mean, it makes sense but I don't want this to become a norm in any form of the sense
There was a problem hiding this comment.
If it's any help, I'd happily accept a PR in BungeeGuard that implements this the other way around :)
There was a problem hiding this comment.
Yea, that would probs be nicer, just feels a bit "shameful" like this, and I don't wanna set the means for that oddball slippy slope of "shaming" plugins <3
|
I kinda want this, just solely because I can't be arsed breaking my setup to test waterfall cruddery, there was a linked PR which is closed now, which I have no idea where that derives from, etc, etc; is there any interest in updating this, if Luck has no issues, in good faith, we can probably keep the message as-is |
The PR it's same as here but without that excesive log and higher version support |
This PR brings Waterfall on a forwarding-feature-level with Velocity for better security.
Implemented:
This PR is soon to be accompanied by a PR to update the PaperMC/PaperDocs with information on all of this