Skip to content

Configurable retry for ACL token acquisition - #719

Merged
StuartFerguson merged 1 commit into
mainfrom
codex/merchant-token-retry
Aug 10, 2026
Merged

StuartFerguson merged 1 commit into
mainfrom
codex/merchant-token-retry

Conversation

@StuartFerguson

@StuartFerguson StuartFerguson commented Aug 10, 2026 •

Copy link
Copy Markdown
Member

Adds a configurable retry around ACL token acquisition so transient SecurityService failures no longer fail merchant read paths immediately.

Changes:

  • Retry token acquisition up to the configurable AppSettings:SecurityServiceTokenRetryCount value.
  • Preserve cancellation semantics so a canceled request still stops immediately.
  • Add regression tests for transient exception recovery, retry exhaustion, and cancellation.

Validation:

  • dotnet test TransactionProcessorACL.BusinessLogic.Tests\TransactionProcessorACL.BusinessLogic.Tests.csproj
  • dotnet test TransactionProcessorACL.Tests\TransactionProcessorACL.Tests.csproj

Closes #718.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 7 complexity · 0 duplication

Metric Results
Complexity 7
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@StuartFerguson StuartFerguson changed the title Merchant token retry Configurable retry for ACL token acquisition Aug 10, 2026
@StuartFerguson
StuartFerguson marked this pull request as ready for review August 10, 2026 20:45
@StuartFerguson
StuartFerguson merged commit 8bbeef3 into main Aug 10, 2026
6 checks passed
@github-actions
github-actions Bot deleted the codex/merchant-token-retry branch October 10, 2026 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ACL] Handle token acquisition failures in merchant read paths

1 participant