You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@fineract.apache.org by "Awasum Yannick (Jira)" <ji...@apache.org> on 2019/12/06 18:33:00 UTC
[jira] [Assigned] (FINERACT-696) use SpotBugs and findbugs-slf4j to
detect bad logging anti-patterns
[ https://issues.apache.org/jira/browse/FINERACT-696?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Awasum Yannick reassigned FINERACT-696:
---------------------------------------
Assignee: Awasum Yannick
> use SpotBugs and findbugs-slf4j to detect bad logging anti-patterns
> -------------------------------------------------------------------
>
> Key: FINERACT-696
> URL: https://issues.apache.org/jira/browse/FINERACT-696
> Project: Apache Fineract
> Issue Type: Improvement
> Components: Build
> Reporter: Michael Vorburger
> Assignee: Awasum Yannick
> Priority: Major
> Labels: starter
>
> While code reviewing [https://github.com/apache/fineract/pull/464], I came accross a {{catch}} & {{printStackTrace}}, which is of course wrong (it should instead be, correctly, logged, instead).
> In an ideal world, it would not take human code review to catch this, but just an automated build failure. This is possible using SpotBugs (not FindBugs anymore; SpotBugs is the new FindBugs, it's a fully compatible successor) and [https://github.com/KengoTODA/findbugs-slf4j/] (for this particular case [https://github.com/KengoTODA/findbugs-slf4j/issues/70], but using findbugs-slf4j for Fineract would have value even before that's implemented).
> [~myrle] you mentioned at FOSDEMO that you had hordes of volunteers just waiting to have issues to working on (I'm joking) - so I was wondering if perhaps you knew of anyone who would be willing to work on this one?
--
This message was sent by Atlassian Jira
(v8.3.4#803005)