fix: run Commands when set by "cmds add" cli (#655) #1106
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Hello, I came across the exact issue #655 in that when adding cmds via
filebrowser
cli instead of in web UI, filebrowser fails to run the command due to leading whitespace (eg. input command is "echo hello
" but filebrowser will run "echo hello
")To reproduce
Try to upload 1 file
hello world
is printed to stdoutno such file or directory
error due to running " echo hello world" instead of "echo hello world"Root cause
[]string{""}
instead of[]string{}
due to strings.Split() expected behaviorlen(Shell)
always >= 1command = append(s.Shell, raw)
which is ["", "echo hello world"] which leads tocmd := exec.Command("", "echo", "hello", "world")
here=> runs exec command
" echo hello world"
instead of "echo hello world"Testing
🚨 I have followed all the below:
❤️ Thank you!