[1.12.2] How would I send variable over network (using Simpleimpl) from command->server->remote client?

[Nick82285]

The synopsis is that I have a command that sends out data to the server in the form of a CharSequence (I used system.out to verify that this variable is only sent/initialized if I send it to the server first by the way of SimpleNetworkWrapper#sendToServer). I then need to be able to relay that variable to a remote client that's separate from the player who executed the command. Is there a simple way to implement this effectively?

 

Here's my netcode, please don't judge

https://pastebin.com/CrpEvtkY

 

I bet you that you can't guess what my code actually does (Joke)

 

 

Edit 1: Updated code based on Animefan8888/diesieben07. If you want to test it, I believe everything you need is in the paste (If not P.M me). My goal is to create a command that's executed by a command block/player to tell a connected client to create an overlay on the client specified in the command.
https://pastebin.com/irwqLW5q

 

Edit 2: I used the VLCJ (3.12.1), slf4j-api (1.7.25) and the installation directory for VLC media player in C:\Program Files\VideoLAN\VLC

Edited September 6, 2019 by Nick82285
put all classes into pastebin, fixed code

[Animefan8888]

  On 9/6/2019 at 2:08 AM, Nick82285 said:

to a remote client that's separate from the player

Expand  

I'm sorry what does this mean? Like it's not a player connected to the server?

 

  On 9/6/2019 at 2:08 AM, Nick82285 said:

The synopsis is that I have a command that sends out data to the server in the form of a CharSequence

Expand  

I have a problem with your question in particular. Commands are executed on the server. So the sequence goes more like this

Client sends chat message with command->Server receives message and executes command... here is where I'm confused. Now you need to send the string back to the client? Send the packet to the player who issued the command.

[Nick82285]

  On 9/6/2019 at 9:54 AM, diesieben07 said:

Don't post random snippets of your code, it makes it near impossible to understand what's actually going on. 

Expand  

Everythings' in the new pastebin link

[Nick82285]

  On 9/6/2019 at 8:48 PM, diesieben07 said:

this should be done in preInit or init, not in a static initializer

Expand  

Fair (I fixed it), Just following the example in the ForgeDocs for 1.12

  On 9/6/2019 at 8:48 PM, diesieben07 said:

Your IDE should be warning you about this.

Expand  

Fair as well (fixed as well), but doesn't and I don't know why. It's Eclipse 2019 06. That and I'm not the best programmer.

 

  On 9/6/2019 at 8:48 PM, diesieben07 said:

Your code is a giant security problem. You are allowing the server to send arbitrary paths to the client

Expand  

I don't quite see that. This will only be used by command blocks and the commands won't be changed unless there is a rogue admin (potentially preventing a redirect to access system files). The client then checks if the local directory/local video file exists before executing on it.

 

Should I hard code in these directories or sync a json file (with a third party program, however this adds another security concern) to update these directories? It's kind of difficult if I need a fast working environment to do either.

 

Is there any way to get it to work?