Skip to content

Conversation

stijnmoreels
Copy link
Member

Remove the general exception handling in the message router in favor of adding a missing exception handling in the custom message body parsing logic.

Closes #456

@stijnmoreels stijnmoreels added the area:message-processing All issues related to how we process the messages label Sep 19, 2025
Copy link

netlify bot commented Sep 19, 2025

Deploy Preview for arcus-messaging canceled.

Name Link
🔨 Latest commit b21f57f
🔍 Latest deploy log https://app.netlify.com/projects/arcus-messaging/deploys/68ccf0fdfe066a0008d75c31

@stijnmoreels stijnmoreels marked this pull request as ready for review September 19, 2025 07:09
Copy link
Member

@fgheysels fgheysels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks better indeed :)

@fgheysels fgheysels merged commit da69e61 into arcus-azure:main Sep 20, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:message-processing All issues related to how we process the messages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🕵️ Investigate in places where exceptions are being caught in the Azure Service bus message router
2 participants