Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Commit

Permalink
Merge pull request #32 from BonnierNews/reject-with-correlation
Browse files Browse the repository at this point in the history
Reject with correlationId
  • Loading branch information
maxolofsson1 authored Nov 10, 2023
2 parents f3c2a41 + 2e96a4f commit 8c931bf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/message-handler.js
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ export default async function messageHandler(recipeMap, req, res) {
case "resume":
if (Number(resumedCount) >= maxResumeCount) {
logger.error(`Message delayed for max amount ${maxResumeCount} of times`);
await rejectMessage({ ...message, error: `To many resume retries. Retries: ${resumedCount}` }, { key });
await rejectMessage({ ...message, error: `To many resume retries. Retries: ${resumedCount}` }, { key, correlationId, parentCorrelationId });
return res.status(200).send();
}
if (!isValidDelay(result.delayMs)) {
Expand Down Expand Up @@ -168,7 +168,7 @@ export default async function messageHandler(recipeMap, req, res) {
} catch (error) {
if (error.rejected) {
logger.error(`Rejected message with correlationId: ${correlationId}. Error: ${error}. Message: ${message}`);
await rejectMessage({ ...message, error: { message: error.extraMessage } }, { key });
await rejectMessage({ ...message, error: { message: error.extraMessage } }, { key, correlationId, parentCorrelationId });
metrics.rejectedMessages.inc();
return res.status(200).send();
}
Expand Down

0 comments on commit 8c931bf

Please sign in to comment.