Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reject message #200

Open
andreieuganox opened this issue Oct 12, 2023 · 3 comments
Open

Reject message #200

andreieuganox opened this issue Oct 12, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@andreieuganox
Copy link

Describe the bug
It is not the bug per se..but there is currently no way to send the message explicitly to the DLQ. For any reason, consumers must be able to notify the broker that this message can not be processed, sending the message to DLQ. The application decides the message can never be processed (for example, the context subject has no permission to perform this operation), meaning retry will not work, even if you do that 1000 times.

To Reproduce
I can use message.ack to acknowledge the message that considers it to be executed.

Expected behavior
Any event/queue consumer must be able to explicitly reject the message, letting the broker know that this message can not be processed and send it to DLQ. Currently, this can be achieved only based on schema - that is not the best practice.

Additional context
Add any other context about the problem here.

@andreieuganox andreieuganox added the bug Something isn't working label Oct 12, 2023
@yanivbh1
Copy link
Contributor

Hi @andreieuganox,
Your required behavior is called NACK, meaning "Do not acknowledge", and force the broker to flag this event/message as a dead-letter.
@idanasulinmemphis, it would be great to add for the next release.

@andreieuganox
Copy link
Author

@yanivbh1 thank you for picking it...again) Seems we will have some tickets to go through with you, sorry for that upfront)

Exactly as you mentioned, NACK is one of the must have, IMHO.

Apart from that, it seems to be impossible to pick anything up from DLQ (I can be wrong here, need to read docs more)

@yanivbh1
Copy link
Contributor

@andreieuganox ,
in the coming release, consumption for DLQ would be enabled in the coming release. Two weeks from now.
Till then, dead-letter messages can be transmitted via the GUI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants