findbugs-slf4j
findbugs-slf4j copied to clipboard
KengoTODA
A SpotBugs/FindBugs plugin to verify usage of SLF4J
This is a suggestion for a new validation: When reviewing code, I regularly see folks doing things such as: LOG.trace("bla bla: {}", dpid.toString()); where that `toString()` is wrong of course,...
Under Java 9, both findbugs and spotbugs seem to fail when running the findbugs-slf4j checks on our usage of `Thread.UncaughtExceptionHandler` with the following: ```java private static final UncaughtExceptionHandler uncaughtExceptionHandler =...
In a large project, it can be useful to enforce that everyone is actually using slf4j and not (directly) log4j (v1/v2) or JUL etc. A new detector checking that any...
Proposal for a new check which flags this up as wrong: ``` } catch (SomeException e) { LOG.error("..."); } ``` but this is, typically, what you want people to do...
We should have a new check here which [detects this kind of non-sense I'm seeing suprisingly often](http://blog2.vorburger.ch/2018/03/slf4j-logging-api-miss-use-detection.html): LOG.error("bla bla bla {}", e.getMessage()) as well as `e.getStackTrace()` and `e.getLocalizedMessage()` (anything else?),...
Separating out from original #69 : We could have a new check which [detects this kind of non-sense I'm seeing suprisingly often](http://blog2.vorburger.ch/2018/03/slf4j-logging-api-miss-use-detection.html): LOG.error("bla bla bla problem {}", e.getStackTrace()) @KengoTODA would...
A new check named something like `RFE: SLF4J_LOGGER_NAMING` which, by default, would validate that a Logger field is named `LOG` (if it's static) and `logger` (?) if it's not, but...
Since slf4j automatically adds square brackets to collections, arrays etc passed into the message within square brackets would result in two sets of square brackets. Example: ``` final List serviceClasses...
Bumps [mockito-core](https://github.com/mockito/mockito) from 4.5.1 to 4.8.1. Release notes Sourced from mockito-core's releases. v4.8.1 Changelog generated by Shipkit Changelog Gradle Plugin 4.8.1 2022-10-17 - 6 commit(s) by andrepaschoal, dependabot[bot] Possible fix...
![dependabot[bot] avatar](https://avatars.githubusercontent.com/in/29110?v=4 "dependabot[bot] avatar"){kind=avatar}
Bumps [spotless-maven-plugin](https://github.com/diffplug/spotless) from 2.22.5 to 2.27.2. Commits 23784ba Published maven/2.27.2 b32df21 Allow replacement to be null for Replace and ReplaceRegex of plugin maven (#1... 4989245 Update changelog. aa48b8d Allow replacement...