Merge pull request #373 from gwynne/patch-1

Fix mishandling of action inputs

Author: 0xTim

action.yml

@@ -14,18 +14,23 @@ inputs:
   folder-to-copy:
     description: "Folder to copy assets from into S3."
     required: false
+    default: ""
   environment-prefix:
     description: "Prefix of the GitHub Deployment Environment. The PR number will be appended to this value."
     required: false
+    default: ""
   index-document:
     description: "Index document for the S3 bucket. Defaults to index.html."
     required: false
+    default: "index.html"
   error-document:
     description: "Error document for the S3 bucket. Defaults to error.html."
     required: false
+    default: "error.html"
   token:
     description: "Access token used to crete and manage deployments. If both this input and the GITHUB_TOKEN environment variable are provided, this input takes precedence."
     required: false
+    default: ""
 runs:
   using: "node24"
   main: "dist/index.js"

dist/index.js

@@ -27979,6 +27979,12 @@ var ExitCode;
  */
 function getInput(name, options) {
     const val = process.env[`INPUT_${name.replace(/ /g, '_').toUpperCase()}`] || '';
+    if (options && options.required && !val) {
+        throw new Error(`Input required and not supplied: ${name}`);
+    }
+    if (options && options.trimWhitespace === false) {
+        return val;
+    }
     return val.trim();
 }
 /**
@@ -62923,12 +62929,12 @@ function isSuccessResponse(object) {
 
 const main = async () => {
     try {
-        const bucketPrefix = getInput('bucket-prefix');
-        const bucketRegion = getInput('bucket-region');
+        const bucketPrefix = getInput('bucket-prefix', { required: true });
+        const bucketRegion = getInput('bucket-region', { required: true });
         const folderToCopy = getInput('folder-to-copy');
         const environmentPrefix = getInput('environment-prefix');
-        const indexDocument = getInput('index-document') ?? 'index.html';
-        const errorDocument = getInput('error-document') ?? 'error.html';
+        const indexDocument = getInput('index-document') || 'index.html';
+        const errorDocument = getInput('error-document') || 'error.html';
         const tokenFromInput = getInput('token');
         if (tokenFromInput) {
             process.env.GITHUB_TOKEN = tokenFromInput;

dist/index.js.map

@@ -7,12 +7,12 @@ import dayjs from 'dayjs'
 
 const main = async () => {
 	try {
-		const bucketPrefix = core.getInput('bucket-prefix')
-		const bucketRegion = core.getInput('bucket-region')
+		const bucketPrefix = core.getInput('bucket-prefix', { required: true })
+		const bucketRegion = core.getInput('bucket-region', { required: true })
 		const folderToCopy = core.getInput('folder-to-copy')
 		const environmentPrefix = core.getInput('environment-prefix')
-		const indexDocument = core.getInput('index-document') ?? 'index.html'
-		const errorDocument = core.getInput('error-document') ?? 'error.html'
+		const indexDocument = core.getInput('index-document') || 'index.html'
+		const errorDocument = core.getInput('error-document') || 'error.html'
 		const tokenFromInput = core.getInput('token')
 
 		if (tokenFromInput) {