Skip to content

Commit

Permalink
Merge pull request #119 from easingthemes/bugfix/#118-default-value-c…
Browse files Browse the repository at this point in the history
…heck

fix: #118 check undefined default values
  • Loading branch information
easingthemes authored Feb 21, 2023
2 parents ae4970d + f0c02fb commit e7b2312
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions src/inputs.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ const githubWorkspace = process.env.GITHUB_WORKSPACE;
const remoteUser = process.env.REMOTE_USER || process.env.INPUT_REMOTE_USER;

const defaultInputs = {
source: '',
target: `/home/${remoteUser}/`,
exclude: '',
args: '-rlgoDzvc -i',
sshCmdArgs: '-o StrictHostKeyChecking=no',
deployKeyName: `deploy_key_${remoteUser}_${Date.now()}`
};

Expand All @@ -21,18 +25,19 @@ const inputs = {
inputNames.forEach((input) => {
const inputName = snakeToCamel(input.toLowerCase());
const inputVal = process.env[input] || process.env[`INPUT_${input}`] || defaultInputs[inputName];
let extendedVal = inputVal;
const validVal = inputVal === undefined ? defaultInputs[inputName] : inputVal;
let extendedVal = validVal;
// eslint-disable-next-line default-case
switch (inputName) {
case 'source':
extendedVal = inputVal.split(' ').map((src) => `${githubWorkspace}/${src}`);
extendedVal = validVal.split(' ').map((src) => `${githubWorkspace}/${src}`);
break;
case 'args':
extendedVal = inputVal.split(' ');
extendedVal = validVal.split(' ');
break;
case 'exclude':
case 'sshCmdArgs':
extendedVal = inputVal.split(',').map((item) => item.trim());
extendedVal = validVal.split(',').map((item) => item.trim());
break;
}

Expand Down

0 comments on commit e7b2312

Please sign in to comment.