Make sure job is object before accessing properties on it

SurveyLegend · Aug 17, 2020 · 9bfc969 · 9bfc969
1 parent 17a6a24
commit 9bfc969
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/clean_failed_job_stacks.js b/clean_failed_job_stacks.js
@@ -49,7 +49,7 @@ forEach(args, async (name, index) => {
 
         const jobs = await queue.getFailed()
         const retriedJobCount = await reduce(jobs, async (count, job) => {
-            if (job.attemptsMade <= job.opts.attempts) {
+            if (!job || typeof job !== 'object' || job.attemptsMade <= job.opts.attempts) {
                 return count
             }
             const { id, data, opts } = job

diff --git a/retry_failed_jobs.js b/retry_failed_jobs.js
@@ -49,7 +49,7 @@ forEach(args, async (name, index) => {
 
         const jobs = await queue.getFailed()
         const retriedJobCount = await reduce(jobs, async (count, job) => {
-            if (job.attemptsMade > MAX_FAILED_COUNT) {
+            if (!job || typeof job !== 'object' || job.attemptsMade > MAX_FAILED_COUNT) {
                 return count
             }
             try {