Skip to content

Add bridge critical error action#858

Merged
boblangley merged 1 commit into
masterfrom
bridge-critical-error-action
Jun 5, 2026
Merged

Add bridge critical error action#858
boblangley merged 1 commit into
masterfrom
bridge-critical-error-action

Conversation

@boblangley

@boblangley boblangley commented Jun 5, 2026

Copy link
Copy Markdown
Member

Summary

Adds a public BridgeConfiguration.DefineCriticalErrorAction(...) API so bridge hosts can configure a response to critical errors using the same delegate shape as regular NServiceBus endpoints.

The configured action is stored in the finalized bridge configuration and applied to the raw endpoint configuration for both bridge proxy endpoints and send-only dispatcher endpoints. This keeps the action wired through transport HostSettings.CriticalErrorAction, matching the convention used by NServiceBus Core.

Testing

  • Added bridge configuration tests for null handler validation and finalized configuration storage.
  • Added endpoint proxy factory tests that use a fake transport to raise critical errors through HostSettings.CriticalErrorAction for both dispatcher and proxy raw endpoints.
  • Updated public API approval for the additive API change.

@boblangley boblangley self-assigned this Jun 5, 2026
@boblangley boblangley marked this pull request as ready for review June 5, 2026 17:58
@boblangley boblangley requested a review from andreasohlund June 5, 2026 18:06
@boblangley boblangley merged commit 6d9c81f into master Jun 5, 2026
16 checks passed
@boblangley boblangley deleted the bridge-critical-error-action branch June 5, 2026 20:10
@boblangley boblangley added this to the 5.1.0 milestone Jun 5, 2026
@boblangley boblangley added the Feature New feature or request label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants