[ 
https://issues.apache.org/jira/browse/SSHD-953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Niklas Rosenstein updated SSHD-953:
-----------------------------------
    Description: 
{code:java}
sshServer.setCommandFactory(new ProcessShellCommandFactory();
{code}

This command factory does not parse quoted arguments correctly. Example:

{code}
"scp -t -r -p '/Volumes/data/input.csv'"
{code}

The last argument is parsed as

{code}
"'/Volumes/git/data/input.csv'"
{code}

when it should be parsed as

{code}
"/Volumes/git/data/input.csv"
{code}

(note the missing single quotes)

The implementation of the parsing uses {{GenericUtils.split()}} rather than a 
function that handles quotes correctly.

{code}
        ShellFactory factory = new 
ProcessShellFactory(GenericUtils.split(command, ' '));
{code}


  was:
{code:java}
sshServer.setCommandFactory(new ProcessShellCommandFactory();
{code}

This command factory does not parse quoted arguments correctly. Example:

{code}
"scp -t -r -p '/Volumes/git/data/input.csv'"
{code}

The last argument is parsed as

{code}
"'/Volumes/git/data/input.csv'"
{code}

when it should be parsed as

{code}
"/Volumes/git/data/input.csv"
{code}

(note the missing single quotes)

The implementation of the parsing uses {{GenericUtils.split()}} rather than a 
function that handles quotes correctly.

{code}
        ShellFactory factory = new 
ProcessShellFactory(GenericUtils.split(command, ' '));
{code}



> ProcessShellCommandFactory does not parse quoted arguments correctly
> --------------------------------------------------------------------
>
>                 Key: SSHD-953
>                 URL: https://issues.apache.org/jira/browse/SSHD-953
>             Project: MINA SSHD
>          Issue Type: Bug
>    Affects Versions: 2.3.0
>            Reporter: Niklas Rosenstein
>            Priority: Major
>
> {code:java}
> sshServer.setCommandFactory(new ProcessShellCommandFactory();
> {code}
> This command factory does not parse quoted arguments correctly. Example:
> {code}
> "scp -t -r -p '/Volumes/data/input.csv'"
> {code}
> The last argument is parsed as
> {code}
> "'/Volumes/git/data/input.csv'"
> {code}
> when it should be parsed as
> {code}
> "/Volumes/git/data/input.csv"
> {code}
> (note the missing single quotes)
> The implementation of the parsing uses {{GenericUtils.split()}} rather than a 
> function that handles quotes correctly.
> {code}
>         ShellFactory factory = new 
> ProcessShellFactory(GenericUtils.split(command, ' '));
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org

Reply via email to