You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "essobedo (via GitHub)" <gi...@apache.org> on 2024/04/19 17:53:37 UTC

[PR] CAMEL-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

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

   Fixes https://issues.apache.org/jira/browse/CAMEL-20692 (Camel 3.22)
   
   ## Motivation
   
   When the logging feature is enabled in the CXF component, the payload is not displayed in the log because the default log limit is set to 0 by unexpected default
   
   ## Modifications:
   
   * Initialize the value of `loggingSizeLimit` to `AbstractLoggingInterceptor.DEFAULT_LIMIT` as expected
   * Align the code related to `loggingSizeLimit` in the CXFRS component to allow to configure `-1` and `0` as limit
   
   


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067107113

   :no_entry_sign: There are (likely) no changes in core core to be tested in this PR


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067041276

   :star2: Thank you for your contribution to the Apache Camel project! :star2: 
   
   :warning: Please note that the changes on this PR may be **tested automatically**. 
   
   If necessary Apache Camel Committers may access logs and test results in the job summaries!


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067107099

   ### Components test results:
   
   | Total | Tested | Failed :x: | Passed :white_check_mark: | 
   | --- | --- | --- |  --- |
   | 1 | 1 | 1 | 0 |


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067257289

   is there anything on main branch that needs fixing/changes ?


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067130271

   :no_entry_sign: There are (likely) no changes in core core to be tested in this PR


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2067130261

   ### Components test results:
   
   | Total | Tested | Failed :x: | Passed :white_check_mark: | 
   | --- | --- | --- |  --- |
   | 1 | 1 | 1 | 0 |


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "essobedo (via GitHub)" <gi...@apache.org>.
essobedo commented on PR #13865:
URL: https://github.com/apache/camel/pull/13865#issuecomment-2076538070

   @davsclaus Hi, sorry I was off, I will propose the PRs


-- 
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-20692: camel-cxf - Set a default value for loggingSizeLimit [camel]

Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus merged PR #13865:
URL: https://github.com/apache/camel/pull/13865


-- 
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