Skip to content

refactor(http)!: nest standard hedge options - #372

Merged
thomhurst merged 4 commits into
mainfrom
issue-331-standard-hedge-options
Aug 27, 2026
Merged

refactor(http)!: nest standard hedge options#372
thomhurst merged 4 commits into
mainfrom
issue-331-standard-hedge-options

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

  • reshape StandardHedgeShieldOptions into nested handler, routing, circuit-breaker, concurrency-limit, attempt-timeout, total-timeout, and hedge options
  • bind nested configuration keys and reject obsolete flat aliases with migration guidance
  • preserve callbacks, endpoint snapshots, defaults, public API docs, samples, and integration behavior

Motivation

Align the standard hedge API with StandardHttpShieldOptions so configuration and code use the same composable options model.

Closes #331

Validation

  • dotnet build Kevlar.slnx -c Release
  • dotnet run --project tests/Kevlar.Tests -c Release --no-build --framework net8.0 -- --timeout 5m (1208 passed)
  • dotnet run --project tests/Kevlar.Tests -c Release --no-build --framework net10.0 -- --timeout 5m (1239 passed)
  • dotnet run --project tests/Kevlar.IntegrationTests -c Release --no-build -- --timeout 5m (166 passed)
  • API/docs/changelog verifiers, package creation, snippet compilation (implicit and explicit usings), and Docusaurus production build

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1bf11b71-eb8a-40e7-bbff-1f627f8bf664


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thomhurst
thomhurst force-pushed the issue-331-standard-hedge-options branch 2 times, most recently from bbf9faa to 7bdaad9 Compare August 27, 2026 01:53
@thomhurst
thomhurst force-pushed the issue-331-standard-hedge-options branch from 7bdaad9 to 12f331c Compare August 27, 2026 02:16
@thomhurst
thomhurst merged commit ea81b96 into main Aug 27, 2026
8 checks passed
@thomhurst
thomhurst deleted the issue-331-standard-hedge-options branch August 27, 2026 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(http)!: give StandardHedgeShieldOptions the nested shape of StandardHttpShieldOptions

1 participant