The existing code treated the entrypoint string as a PathBuf, when in reality it could be a cmdline string with spaces and arguments.
This PR updates the handling to reflect this, and also modifies the retrieve_local_plugin() behavior to only copy the plugin entrypoint binary if it is a path. For instance, if the entrypoint started as docker ..., we wouldn't copy the docker binary because does not exist as a path.
Also updated the warning in start_plugin() to warn if the binary component of entrypoint cannot be found with which.
The existing code treated the
entrypoint
string as aPathBuf
, when in reality it could be a cmdline string with spaces and arguments.This PR updates the handling to reflect this, and also modifies the
retrieve_local_plugin()
behavior to only copy the plugin entrypoint binary if it is a path. For instance, if the entrypoint started asdocker ...
, we wouldn't copy thedocker
binary because does not exist as a path.Also updated the warning in
start_plugin()
to warn if the binary component ofentrypoint
cannot be found withwhich
.