Fail fast on invalid deliverer configuration by endrju19 · Pull Request #119 · softwaremill/okapi · GitHub
Skip to content

Fail fast on invalid deliverer configuration - #119

Merged
rawo merged 2 commits into
mainfrom
fix/deliverer-validation-117-118
Sep 30, 2026
Merged

rawo merged 2 commits into
mainfrom
fix/deliverer-validation-117-118

Conversation

@endrju19

@endrju19 endrju19 commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Verification

  • Core and Spring auto-configuration tests on current main.
  • Spring auto-configuration tests with Spring Boot 3.5.12 and 4.0.4.
  • ktlintCheck for okapi-core and okapi-spring-boot.

This is independent of #116, which concerns claiming entries with unsupported delivery types across instances.

Reject duplicate MessageDeliverer types instead of silently replacing one transport.
Fail default processor startup when no deliverer is registered while preserving
publisher-only and custom processor configurations.

Closes #117
Closes #118
@rawo
rawo self-requested a review September 28, 2026 08:08

@rawo rawo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rawo
rawo merged commit 6c1ffc8 into main Sep 30, 2026
8 checks passed
@rawo
rawo deleted the fix/deliverer-validation-117-118 branch September 30, 2026 08:00
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.

2 participants