diff --git a/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/DependencyCheckMojo.java b/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/DependencyCheckMojo.java index 0090fb135..c54429d4e 100644 --- a/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/DependencyCheckMojo.java +++ b/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/DependencyCheckMojo.java @@ -69,6 +69,8 @@ import org.owasp.dependencycheck.utils.Settings; requiresOnline = true) public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageReport { + private final Logger logger = Logger.getLogger(DependencyCheckMojo.class.getName()); + /** * The properties file location. */ @@ -200,7 +202,7 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR @Parameter(property = "connectionTimeout", defaultValue = "", required = false) private String connectionTimeout = null; /** - * The Connection Timeout. + * The path to the suppression file. */ @SuppressWarnings({"CanBeFinal", "FieldCanBeLocal"}) @Parameter(property = "suppressionFile", defaultValue = "", required = false) @@ -393,7 +395,7 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR cve.open(); prop = cve.getDatabaseProperties(); } catch (DatabaseException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, "Unable to retrieve DB Properties", ex); + logger.log(Level.FINE, "Unable to retrieve DB Properties", ex); } finally { if (cve != null) { cve.close(); @@ -403,13 +405,13 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR try { r.generateReports(outDirectory.getCanonicalPath(), format); } catch (IOException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.SEVERE, + logger.log(Level.SEVERE, "Unexpected exception occurred during analysis; please see the verbose error log for more details."); - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, null, ex); + logger.log(Level.FINE, null, ex); } catch (Throwable ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.SEVERE, + logger.log(Level.SEVERE, "Unexpected exception occurred during analysis; please see the verbose error log for more details."); - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, null, ex); + logger.log(Level.FINE, null, ex); } } @@ -816,14 +818,14 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR mojoProperties = this.getClass().getClassLoader().getResourceAsStream(PROPERTIES_FILE); Settings.mergeProperties(mojoProperties); } catch (IOException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.WARNING, "Unable to load the dependency-check ant task.properties file."); - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, null, ex); + logger.log(Level.WARNING, "Unable to load the dependency-check ant task.properties file."); + logger.log(Level.FINE, null, ex); } finally { if (mojoProperties != null) { try { mojoProperties.close(); } catch (IOException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINEST, null, ex); + logger.log(Level.FINEST, null, ex); } } } @@ -943,9 +945,9 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR checkForFailure(engine.getDependencies()); } } catch (DatabaseException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.SEVERE, + logger.log(Level.SEVERE, "Unable to connect to the dependency-check database; analysis has stopped"); - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, "", ex); + logger.log(Level.FINE, "", ex); } finally { Settings.cleanup(); if (engine != null) { @@ -984,9 +986,9 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR generateMavenSiteReport(engine, sink); } } catch (DatabaseException ex) { - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.SEVERE, + logger.log(Level.SEVERE, "Unable to connect to the dependency-check database; analysis has stopped"); - Logger.getLogger(DependencyCheckMojo.class.getName()).log(Level.FINE, "", ex); + logger.log(Level.FINE, "", ex); } finally { Settings.cleanup(); if (engine != null) { @@ -1010,8 +1012,7 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR } else if ("VULN".equalsIgnoreCase(this.format)) { return "dependency-check-vulnerability"; } else { - Logger.getLogger(DependencyCheckMojo.class - .getName()).log(Level.WARNING, "Unknown report format used during site generatation."); + logger.log(Level.WARNING, "Unknown report format used during site generatation."); return "dependency-check-report"; } } @@ -1149,9 +1150,7 @@ public class DependencyCheckMojo extends AbstractMojo implements MavenMultiPageR final String msg = String.format("%n%n" + "One or more dependencies were identified with known vulnerabilities:%n%n%s" + "%n%nSee the dependency-check report for more details.%n%n", summary.toString()); - Logger - .getLogger(DependencyCheckMojo.class - .getName()).log(Level.WARNING, msg); + logger.log(Level.WARNING, msg); } } }