Skip to content

Commit 05271d6

Browse files
authored
impr: log failed worker queue jobs (@fehmer) (monkeytypegame#6295)
!nuf closes monkeytypegame#6294
1 parent 598e167 commit 05271d6

File tree

2 files changed

+18
-4
lines changed

2 files changed

+18
-4
lines changed

backend/src/workers/email-worker.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,15 @@ async function jobHandler(job: Job<EmailTask<EmailType>>): Promise<void> {
3333
Logger.success(`Job: ${type} - completed in ${elapsed}ms`);
3434
}
3535

36-
export default (redisConnection?: IORedis.Redis): Worker =>
37-
new Worker(EmailQueue.queueName, jobHandler, {
36+
export default (redisConnection?: IORedis.Redis): Worker => {
37+
const worker = new Worker(EmailQueue.queueName, jobHandler, {
3838
autorun: false,
3939
connection: redisConnection as ConnectionOptions,
4040
});
41+
worker.on("failed", (job, error) => {
42+
Logger.error(
43+
`Job: ${job.data.type} - failed with error "${error.message}"`
44+
);
45+
});
46+
return worker;
47+
};

backend/src/workers/later-worker.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,8 +208,15 @@ async function jobHandler(job: Job<LaterTask<LaterTaskType>>): Promise<void> {
208208
Logger.success(`Job: ${taskName} - completed in ${elapsed}ms`);
209209
}
210210

211-
export default (redisConnection?: IORedis.Redis): Worker =>
212-
new Worker(LaterQueue.queueName, jobHandler, {
211+
export default (redisConnection?: IORedis.Redis): Worker => {
212+
const worker = new Worker(LaterQueue.queueName, jobHandler, {
213213
autorun: false,
214214
connection: redisConnection as ConnectionOptions,
215215
});
216+
worker.on("failed", (job, error) => {
217+
Logger.error(
218+
`Job: ${job.data.taskName} - failed with error "${error.message}"`
219+
);
220+
});
221+
return worker;
222+
};

0 commit comments

Comments
 (0)