Merge pull request #97 from thevpc/fix/commandline-formatter-issue

Use double quotes instead of simple ones for arguments escaping in Shell
thevpc · Oct 30, 2021 · 3769736 · 3769736
2 parents 58cd334 + b63c328
commit 3769736
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/core/nuts-runtime/src/main/java/net/thevpc/nuts/runtime/core/shell/ShNutsShellHelper.java b/core/nuts-runtime/src/main/java/net/thevpc/nuts/runtime/core/shell/ShNutsShellHelper.java
@@ -268,7 +268,7 @@ public String escapeArguments(String[] args, NutsCommandLineShellOptions options
 
  public String escapeArgument(String arg, NutsCommandLineShellOptions options) {
  if(arg == null || arg.isEmpty()){
- return "''";
+ return "\"\"";
  }
  if(options==null){
  options=new NutsCommandLineShellOptions();
@@ -373,7 +373,7 @@ public String escapeArgument(String arg, NutsCommandLineShellOptions options) {
  }
  case REQUIRE_QUOTES:{
  StringBuilder sb=new StringBuilder();
- sb.append("'");
+ sb.append("\"");
  for (char c : arg.toCharArray()) {
  switch (c){
  case '\\':
@@ -396,7 +396,7 @@ public String escapeArgument(String arg, NutsCommandLineShellOptions options) {
  }
  }
  }
- sb.append("'");
+ sb.append("\"");
  return sb.toString();
  }
  default:{