orpiske opened a new pull request, #28565:
URL: https://github.com/apache/camel/pull/28565

   **Target:** `apache/camel` `main` ← `orpiske/camel` `camel-wp-logging-fix`
   
   **Title:** CAMEL-26347: camel-wordpress - stop adding CXF logging 
interceptors to the WordPress client
   
   ---
   
   <!-- claude-projects-attribution: {"github_login":"orpiske"} -->
   _Requested by **Otavio**_
   
   # Description
   
   Fixes [CAMEL-26347](https://issues.apache.org/jira/browse/CAMEL-26347).
   
   The camel-wordpress service adapters added the CXF `LoggingInInterceptor` 
and `LoggingOutInterceptor` to every
   WordPress client they created. Because of that, every request and response 
the component exchanged with the
   WordPress server was logged at `INFO` level by the `org.apache.cxf.services` 
loggers, with no way to turn it off
   from the endpoint.
   
   This PR:
   
   - removes the two interceptors from `AbstractWordpressServiceAdapter`, so 
the client keeps the interceptors CXF
     sets up for a JAX-RS client
   - removes the `cxf-rt-features-logging` dependency, which is no longer used
   - removes the leftover log4j 1.x `src/main/resources/log4j.properties` from 
the main resources, so it is no longer
     shipped in the `camel-wordpress` JAR
   - adds `WordpressServiceAdapterClientConfigurationTest`, which checks that 
an adapter's client has the same
     interceptors as a plain CXF JAX-RS client and still makes an authenticated 
call
   - adds an upgrade guide entry for 4.24, and for 4.22.3 and 4.18.6 where this 
change is planned to be backported
   
   # Target
   
   - [x] I checked that the commit is targeting the correct branch (Camel 4 
uses the `main` branch)
   
   # Tracking
   - [x] If this is a large change, bug fix, or code improvement, I checked 
there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for 
the change (usually before you start working on it).
   
   # Apache Camel coding standards and style
   
   - [x] I checked that each commit in the pull request has a meaningful 
subject line and body.
   - [ ] I have run `mvn clean install -DskipTests` locally from root folder 
and I have committed all auto-generated changes.
   
   # AI-assisted contributions
   
   - [x] If this PR includes AI-generated code, commits have proper 
co-authorship attribution (e.g., `Co-authored-by` trailers) and the PR 
description identifies the AI tool used.
   
   _Claude Code on behalf of orpiske_
   
   🤖 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]

Reply via email to