fvaleri commented on pull request #193: URL: https://github.com/apache/camel-kafka-connector/pull/193#issuecomment-624512292
> It appears the existing logging is at DEBUG logging level. This new option dont mention this fact so I would assume end users think its logged at INFO logging or always. Yes now is at DEBUG level, which is often set when debugging issues. The proposed option logs at INFO level only if the user explicitly set `camel.source.logPayload=true`, so it's somewhat independent from log level (unless you set it to OFF). > Also it may not only be message body that has sensitive data. Headers can contain autorization tokens, usernames etc. So its probably better to have this option for all of it or nothing. It's actually including all of this, maybe the option name is not clear enough. I'm open to suggestions for better wording. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
