davsclaus opened a new pull request, #27001: URL: https://github.com/apache/camel/pull/27001
Fixes [CAMEL-25099](https://issues.apache.org/jira/browse/CAMEL-25099): configuration options of camel-main that were not applied, found in a deep review. ## Fixed 1. **`camel.trace.traceRests` is documented (and in the metadata) to default to `true`, but the field defaulted to `false`.** camel-main then set that on the context, so the backlog tracer did not trace Rest DSL routes, also with `camel run`. `backlog-tracer.adoc` said `false` too; it is updated. 2. **Wildcard component options (`camel.component.seda*.x`) were cleared right after startup**, so components created later (when routes are built, which is the case for most real components) were not configured. `MainSedaWildcardTest` looked up `seda` twice instead of `seda2`, which hid this; it now checks `seda2` and a component added after start. 3. **Options set through the Java API on `metrics()`, `otel()` and `lra()` were never applied, and `withEnabled(false)` still enabled the service.** - This also affects `camel run --metrics`, which enables message history and event notifiers through the Java API. - Java API values that differ from the defaults are now applied; properties still take precedence. - `otel2`, `mdc` and `telemetryDev` have no Java API accessor, so they are not affected. 4. **Cloud properties location.** - The documented `camel.main.cloudPropertiesLocation` key and the Java API setter are now used (previously only the kebab-case key worked). - Override properties are no longer lost when cloud properties are loaded. - Configuring it as an initial or override property no longer fails with a `NullPointerException`. - `withCloudPropertiesLocation` now takes a `String`; the broken `boolean` variant is deprecated. 5. **`streamCachingBufferSize` set the buffer size to `0` when it was not configured.** 6. **`streamCachingStatisticsEnabled` was never applied.** 7. **An explicit `camel.server.useGlobalSslContextParameters=false` (and the same for `camel.management.`) was ignored when global SSL is enabled.** 8. **A custom `ManagementStrategy` bean made the `EventFactory`, `ManagementObjectNameStrategy` and event notifier beans be set on the replaced strategy.** The upgrade guide covers 1, 2, 3, 4, 5 and 7. ## Not changed - The `streamCachingBufferSize` javadoc says the default is 4096; it is 16384. - `producerTemplateCacheSize` and `consumerTemplateCacheSize` are only used by camel-spring-boot. - Some fluent builder methods have misleading names. - The duplicate SSL configuration is fixed in CAMEL-25098 (#27000). ## Tests - **New and updated tests:** `MainConfigurationEdgeCasesTest` (8 tests, new) and `MainSedaWildcardTest` (fixed). All 9 fail without the fix. - **Full suites:** camel-main and all its upstream modules pass (built with `-am`). _Claude Code on behalf of Claus Ibsen_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
