jeantil opened a new pull request, #2569:
URL: https://github.com/apache/james-project/pull/2569

   This commit drops the unmaintained logback-classic dependecy and upgrades 
logback to the latest version.
   
   Logback-classic has a native JsonEncoder (already in the original version). 
The latest version allows to have the formatted message and to enable/disable 
any top level field\ by configuration.
   
   This allows using logback's own JSON encoder and get rid ot logback-contrib 
which is unmaintained to output json ( the repository is archived and the 
dependencies are quite outdated)
   
    A side effect was that the format of the Json output changes slightly, I 
adjusted the samplec configurations for the crowdsec module.
   
   One of the change is a higher resolution for the timestamp, which required 
raising crowdsec to 1.6.x.
   
   I missed the previous [merge 
request](https://github.com/apache/james-project/pull/2482) which responds to 
[JAMES-4083](https://issues.apache.org/jira/browse/JAMES-4083). I discovered 
this while rebasing on upstream :grimacing: but I feel getting rid of the whole 
abandoned logback contrib is a better solution so I still propose this change 
to you.
   
   


-- 
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: notifications-unsubscr...@james.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For additional commands, e-mail: notifications-h...@james.apache.org

Reply via email to