Merge pull request #241 from dgageot/fix-darwin

Fix darwin

Author: trungutt

src/extension/ui/src/utils/Files.ts

@@ -5,18 +5,19 @@
 import { v1 } from "@docker/extension-api-client-types";
 import { ExecResult } from "@docker/extension-api-client-types/dist/v0";
 import { encode } from "js-base64";
-
 import { BUSYBOX } from "../Constants";
 
 export const tryRunImageSync = async (
   client: v1.DockerDesktopClient,
   args: string[],
   ignoreError = false
 ) => {
-  const showError = ignoreError ? () => {} : client.desktopUI.toast.error;
+  const showError = ignoreError ? () => { } : client.desktopUI.toast.error;
   try {
     const result = await client.docker.cli.exec("run", args);
-    if (result.code !== undefined && result.code != 0 && result.stderr) {
+    if (result.stderr && result.stderr.includes("shell operators are not allowed")) {
+      // This shouldn't happen: it means we have our escaping wrong.
+      // And there's no way to check that failure other than by looking at stderr's content...
       showError(result.stderr);
     }
     return result.stdout || "";
@@ -82,7 +83,7 @@ export const writeToPromptsVolume = async (
     BUSYBOX,
     "/bin/sh",
     "-c",
-    `\\"echo ${encode(content)} | base64 -d > ${filename}\\"`,
+    `'echo ${encode(content)} | base64 -d > ${filename}'`,
   ]);
 };
 
@@ -99,6 +100,6 @@ export const writeToMount = async (
     BUSYBOX,
     "/bin/sh",
     "-c",
-    `\\"echo ${encode(content)} | base64 -d > ${filename}\\"`,
+    `'echo ${encode(content)} | base64 -d > ${filename}'`,
   ]);
 };