You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "Croway (via GitHub)" <gi...@apache.org> on 2026/07/14 08:55:02 UTC
[PR] CAMEL-24057: check actually-used classes in @ConditionalOnClass for access log configurations [camel-spring-boot]
Croway opened a new pull request, #1850:
URL: https://github.com/apache/camel-spring-boot/pull/1850
## Summary
_Claude Code on behalf of Federico Mariani_
`@ConditionalOnClass(name = "io.undertow.Undertow")` only checks for the top-level Undertow class, but the access log configurations use internal classes from `io.undertow.server.handlers.accesslog.*`. On JBoss EAP, the top-level class is visible (JBoss is built on Undertow) but the modular classloader doesn't export all internal packages, causing `ClassNotFoundException`.
- Updated `ManagementAccessLogConfiguration.UndertowAccessLogCustomizerConfiguration` to also check for `AccessLogHandler` and `JBossLoggingAccessLogReceiver`
- Updated `ManagementAccessLogConfiguration.TomcatAccessLogCustomizerConfiguration` to also check for `AccessLogValve` (defensive consistency)
- Updated `UndertowAccessLogConfiguration` to also check for `AccessLogHandler` and `JBossLoggingAccessLogReceiver`
On main, the Undertow code is commented out pending Spring Boot 4.0 `spring-boot-starter-undertow`, but the annotations are fixed now so the bug won't return when the code is uncommented.
## Test plan
- [x] `core/camel-spring-boot` builds successfully
- [x] `camel-platform-http-starter` builds successfully
- [ ] CI passes
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-24057: check actually-used classes in @ConditionalOnClass for access log configurations [camel-spring-boot]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway merged PR #1850:
URL: https://github.com/apache/camel-spring-boot/pull/1850
--
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: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org