Merge pull request #618 from jsnoble/startJob_validation_fix

added validation on start job as well resolves #616
terascope · Nov 20, 2017 · ebd7563 · ebd7563
2 parents 17aab5c + 346944b
commit ebd7563
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/cluster/services/jobs.js b/lib/cluster/services/jobs.js
@@ -66,7 +66,8 @@ module.exports = function module(context) {
                         }
                         return _ensureAssets(jobConfig);
                     })
-                    .then(assetIdJob => executionService.createExecutionContext(assetIdJob));
+                    .then(parsedAssetJob => _validateJob(parsedAssetJob))
+                    .then(validJob => executionService.createExecutionContext(validJob));
             })
             .catch((err) => {
                 const errMsg = `could not start job, error: ${parseError(err)}`;