Skip to content

Commit 69694cf

Browse files
author
David Lanouette
committed
fix: explicitly acknowledge UMB messages in receiver
Messages received from ActiveMQ were not being marked as read because the receiver was closing the connection before auto-accept could transmit the acknowledgment to the broker. Add explicit accept() call after parsing each message to ensure proper acknowledgment. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: David Lanouette <dlanouet@redhat.com> rh-pre-commit.version: 2.3.2 rh-pre-commit.check-secrets: ENABLED
1 parent a20106b commit 69694cf

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

src/pubtools/sign/clients/msg_recv_client.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ def on_message(self, event: proton.Event) -> None:
8686
outer_message = json.loads(event.message.body)
8787
headers = event.message.properties
8888
msg_id = outer_message["msg"][self.id_key]
89+
self.accept(event.delivery)
8990

9091
if msg_id in self.recv_ids:
9192
self.recv_ids[msg_id] = True

0 commit comments

Comments
 (0)