Skip to content

Fix outbox failed batch retry loop#475

Merged
zarusz merged 1 commit into
masterfrom
feature/outbox-failed-batch-retry-loop
Jun 18, 2026
Merged

Fix outbox failed batch retry loop#475
zarusz merged 1 commit into
masterfrom
feature/outbox-failed-batch-retry-loop

Conversation

@zarusz

@zarusz zarusz commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fixes [Host.RabbitMq] Outbox retries exhaust quickly when broker is down #459 by stopping failed outbox batches from immediately re-entering the same sending pass.
  • Failed messages still have delivery attempts recorded, but the next retry is governed by the outbox polling/lock timing instead of a tight same-instance loop.
  • Adds core outbox unit coverage and PostgreSQL/Testcontainers integration coverage for the retry behavior.

Tests

  • dotnet test src\Tests\SlimMessageBus.Host.Outbox.Test\SlimMessageBus.Host.Outbox.Test.csproj --no-build -v minimal -m:1
  • dotnet test src\Tests\SlimMessageBus.Host.Outbox.PostgreSql.Test\SlimMessageBus.Host.Outbox.PostgreSql.Test.csproj --no-build -v minimal -m:1

Signed-off-by: Tomasz Maruszak <maruszaktomasz@gmail.com>
@zarusz zarusz force-pushed the feature/outbox-failed-batch-retry-loop branch from 7c78365 to b42d712 Compare June 17, 2026 17:42
@sonarqubecloud

Copy link
Copy Markdown

@zarusz

zarusz commented Jun 18, 2026

Copy link
Copy Markdown
Owner Author

@cord-agencyroot do you want to review this PR? It should fix one of the issues you've reported previously. Thanks.

@zarusz zarusz self-assigned this Jun 18, 2026
@cord-agencyroot

Copy link
Copy Markdown

@zarusz looks good! thanks for fixing this

@zarusz zarusz merged commit 73efc3a into master Jun 18, 2026
67 of 71 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Host.RabbitMq] Outbox retries exhaust quickly when broker is down

2 participants