Check for invalid retention-days input

yacaovsnc · yacaovsnc · commit 4560c23b396d · 2020-09-18T16:04:35.000-04:00
diff --git a/dist/index.js b/dist/index.js
@@ -6405,6 +6405,9 @@ function getInputs() {
     const retentionDaysStr = core.getInput(constants_1.Inputs.RetentionDays);
     if (retentionDaysStr) {
         inputs.retentionDays = parseInt(retentionDaysStr);
+        if (isNaN(inputs.retentionDays)) {
+            core.setFailed('Invalid retention-days');
+        }
     }
     return inputs;
 }
diff --git a/src/input-helper.ts b/src/input-helper.ts
@@ -31,6 +31,9 @@ export function getInputs(): UploadInputs {
   const retentionDaysStr = core.getInput(Inputs.RetentionDays)
   if (retentionDaysStr) {
     inputs.retentionDays = parseInt(retentionDaysStr)
+    if (isNaN(inputs.retentionDays)) {
+      core.setFailed('Invalid retention-days')
+    }
   }
 
   return inputs

Original file line number	Diff line number	Diff line change
`@@ -6405,6 +6405,9 @@ function getInputs() {`
`6405`	`6405`	`const retentionDaysStr = core.getInput(constants_1.Inputs.RetentionDays);`
`6406`	`6406`	`if (retentionDaysStr) {`
`6407`	`6407`	`inputs.retentionDays = parseInt(retentionDaysStr);`
	`6408`	`+ if (isNaN(inputs.retentionDays)) {`
	`6409`	`+ core.setFailed('Invalid retention-days');`
	`6410`	`+ }`
`6408`	`6411`	`}`
`6409`	`6412`	`return inputs;`
`6410`	`6413`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,9 @@ export function getInputs(): UploadInputs {`
`31`	`31`	`const retentionDaysStr = core.getInput(Inputs.RetentionDays)`
`32`	`32`	`if (retentionDaysStr) {`
`33`	`33`	`inputs.retentionDays = parseInt(retentionDaysStr)`
	`34`	`+ if (isNaN(inputs.retentionDays)) {`
	`35`	`+ core.setFailed('Invalid retention-days')`
	`36`	`+ }`
`34`	`37`	`}`
`35`	`38`
`36`	`39`	`return inputs`