Velocity icon indicating copy to clipboard operation
Velocity copied to clipboard

Expose player info forwarding in the API ( Fix #1063 )

Open kyngs opened this issue 2 years ago • 5 comments

Fix #1063

kyngs avatar Aug 21 '23 13:08 kyngs

Hang on, gotta make checkstyle happy with the import order

kyngs avatar Aug 21 '23 14:08 kyngs

This requires approval by @astei as he made a very deliberate effort in ensuring this does not get exposed to the API.

May I ask the reason?

kyngs avatar Aug 21 '23 16:08 kyngs

The intention is to avoid exposing implementation details to plugins, especially given that things like Configuration are not API

I do wonder the use-case for this sorta thing, the forwarding mode should be effectively transparent to plugins, and have no real impacts elsewhere

electronicboy avatar Aug 21 '23 16:08 electronicboy

I do wonder the use-case for this sorta thing, the forwarding mode should be effectively transparent to plugins, and have no real impacts elsewhere

My practical use case is setting up nano limbo on-demand from the proxy. To do so, I need to forward the forwarding configuration to it.

kyngs avatar Aug 21 '23 16:08 kyngs

Any update?

kyngs avatar Sep 14 '23 09:09 kyngs