Merge pull request #97 from crazy-max/fix-get-buildkit-version

Fix `docker: invalid reference format`

__tests__/buildx.test.ts

@@ -23,7 +23,7 @@ describe('getVersion', () => {
         silent: true
       })
       .then(res => {
-        return !res.stdout.includes(' ') && res.exitCode == 0;
+        return !res.stdout.trim().includes(' ') && res.exitCode == 0;
       });
   }
   (isDaemonRunning() ? it : it.skip)(
@@ -55,7 +55,7 @@ describe('inspect', () => {
         silent: true
       })
       .then(res => {
-        return !res.stdout.includes(' ') && res.exitCode == 0;
+        return !res.stdout.trim().includes(' ') && res.exitCode == 0;
       });
   }
   (isDaemonRunning() ? it : it.skip)(

dist/index.js

@@ -72,7 +72,7 @@ function getVersion() {
             if (res.stderr.length > 0 && res.exitCode != 0) {
                 throw new Error(res.stderr.trim());
             }
-            return parseVersion(res.stdout);
+            return parseVersion(res.stdout.trim());
         });
     });
 }
@@ -227,24 +227,24 @@ function getBuildKitVersion(containerID) {
             .then(bkitimage => {
             if (bkitimage.exitCode == 0 && bkitimage.stdout.length > 0) {
                 return exec
-                    .getExecOutput(`docker`, ['run', '--rm', bkitimage.stdout, '--version'], {
+                    .getExecOutput(`docker`, ['run', '--rm', bkitimage.stdout.trim(), '--version'], {
                     ignoreReturnCode: true,
                     silent: true
                 })
                     .then(bkitversion => {
                     if (bkitversion.exitCode == 0 && bkitversion.stdout.length > 0) {
-                        return `${bkitimage.stdout} => ${bkitversion.stdout}`;
+                        return `${bkitimage.stdout.trim()} => ${bkitversion.stdout.trim()}`;
                     }
                     else if (bkitversion.stderr.length > 0) {
                         core.warning(bkitversion.stderr.trim());
                     }
-                    return bkitversion.stdout;
+                    return bkitversion.stdout.trim();
                 });
             }
             else if (bkitimage.stderr.length > 0) {
                 core.warning(bkitimage.stderr.trim());
             }
-            return bkitimage.stdout;
+            return bkitimage.stdout.trim();
         });
     });
 }

src/buildx.ts

@@ -42,7 +42,7 @@ export async function getVersion(): Promise<string> {
       if (res.stderr.length > 0 && res.exitCode != 0) {
         throw new Error(res.stderr.trim());
       }
-      return parseVersion(res.stdout);
+      return parseVersion(res.stdout.trim());
     });
 }
 
@@ -196,21 +196,21 @@ export async function getBuildKitVersion(containerID: string): Promise<string> {
     .then(bkitimage => {
       if (bkitimage.exitCode == 0 && bkitimage.stdout.length > 0) {
         return exec
-          .getExecOutput(`docker`, ['run', '--rm', bkitimage.stdout, '--version'], {
+          .getExecOutput(`docker`, ['run', '--rm', bkitimage.stdout.trim(), '--version'], {
             ignoreReturnCode: true,
             silent: true
           })
           .then(bkitversion => {
             if (bkitversion.exitCode == 0 && bkitversion.stdout.length > 0) {
-              return `${bkitimage.stdout} => ${bkitversion.stdout}`;
+              return `${bkitimage.stdout.trim()} => ${bkitversion.stdout.trim()}`;
             } else if (bkitversion.stderr.length > 0) {
               core.warning(bkitversion.stderr.trim());
             }
-            return bkitversion.stdout;
+            return bkitversion.stdout.trim();
           });
       } else if (bkitimage.stderr.length > 0) {
         core.warning(bkitimage.stderr.trim());
       }
-      return bkitimage.stdout;
+      return bkitimage.stdout.trim();
     });
 }