davsclaus opened a new pull request, #24783: URL: https://github.com/apache/camel/pull/24783
## Summary - Fix shared `DefaultRestOpenapiProcessorStrategy` instance causing cross-talk between multiple `rest().openApi()` consumers - Each endpoint now gets its own strategy instance (unless the user explicitly supplies a custom one at the component level) - Fixes `missingOperation`/`mockIncludePattern` overwrite, silent fail-fast loss, and `uris` list mixing across consumers ## Root cause `RestOpenApiComponent.doInit()` created ONE `DefaultRestOpenapiProcessorStrategy` and `createEndpoint()` shared it across all endpoints. Each consumer's `afterPropertiesConfigured()` then called `setMissingOperation()` / `setMockIncludePattern()` on the shared object — last writer wins. Combined with `ServiceHelper.initService` idempotency, the second consumer's null `missingOperation` was never resolved, causing `validateOpenApi()` to silently skip all fail/ignore/mock branches. ## Changes - **`RestOpenApiComponent.createEndpoint()`**: Create a fresh `DefaultRestOpenapiProcessorStrategy` per endpoint when no user-supplied custom strategy exists. Share only an explicitly user-supplied strategy. - **`RestOpenApiComponent.doInit()`**: Remove auto-creation of shared default strategy (no longer needed). _Claude Code on behalf of davsclaus_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.6 <[email protected]> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
