Skip to content

Commit ae08d98

Browse files
author
David Lee
committed
adding error handling for missing parameters
1 parent 9269978 commit ae08d98

File tree

2 files changed

+28
-3
lines changed

2 files changed

+28
-3
lines changed

app/app.js

Lines changed: 27 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,33 @@ exports.main = function(argv) {
9292
if (typeof program.taskdelay === "undefined" || isNaN(program.taskdelay)) {
9393
program.taskdelay = constants.TASK_DELAY;
9494
}
95+
let errs = [];
96+
try{
97+
if(typeof program.sourceserver === 'undefined'){
98+
errs.push("sourceserver not defined")
99+
}
100+
if(typeof program.sourcekey === 'undefined'){
101+
errs.push("sourcekey not defined")
102+
}
103+
if(typeof program.targetserver === 'undefined'){
104+
errs.push("targetserver not defined")
105+
}
106+
if(typeof program.targetkey === 'undefined'){
107+
errs.push("targetkey not defined")
108+
}
109+
110+
if(errs.length){
111+
throw new Error(`Errors:\n${errs.join('\n')}`);
112+
}
113+
114+
start();
115+
116+
}
117+
catch(err){
118+
console.log(`${err.message}`)
119+
}
120+
95121

96-
start();
97122

98123
/**
99124
* Task to clean the working folder before DCU grab executes
@@ -165,7 +190,7 @@ exports.main = function(argv) {
165190
// DCU TransferAll assets to target server
166191
await transferAll(program);
167192
}
168-
else{
193+
else{
169194
resolve();
170195
}
171196

package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)