Merge pull request #470 from crazy-max/fix-outputs

Don't set outputs if empty or nil
docker · Sep 24, 2021 · 291bae5 · 291bae5
2 parents a41d90b + 91520df
commit 291bae5
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/buildx.ts b/src/buildx.ts
@@ -15,7 +15,7 @@ export async function getImageID(): Promise<string | undefined> {
   if (!fs.existsSync(iidFile)) {
     return undefined;
   }
-  return fs.readFileSync(iidFile, {encoding: 'utf-8'});
+  return fs.readFileSync(iidFile, {encoding: 'utf-8'}).trim();
 }
 
 export async function getMetadataFile(): Promise<string> {
@@ -27,7 +27,11 @@ export async function getMetadata(): Promise<string | undefined> {
   if (!fs.existsSync(metadataFile)) {
     return undefined;
   }
-  return fs.readFileSync(metadataFile, {encoding: 'utf-8'});
+  const content = fs.readFileSync(metadataFile, {encoding: 'utf-8'}).trim();
+  if (content === 'null') {
+    return undefined;
+  }
+  return content;
 }
 
 export async function getSecretString(kvp: string): Promise<string> {