Changed log level of some logging statements

Author: ic0ns

src/main/java/de/rub/nds/tlsscanner/MultiThreadedScanJobExecutor.java

@@ -87,9 +87,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
             try {
                 resultList.add(probeResult.get());
             } catch (InterruptedException | ExecutionException ex) {
-                LOGGER.warn("Encoutered Exception while retrieving probeResult");
-                ex.printStackTrace();
-                LOGGER.warn(ex);
+                LOGGER.error("Encoutered Exception while retrieving probeResult", ex);
             }
         }
 
@@ -131,9 +129,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
             try {
                 resultList.add(probeResult.get());
             } catch (InterruptedException | ExecutionException ex) {
-                LOGGER.warn("Encoutered Exception while retrieving probeResult");
-                ex.printStackTrace();
-                LOGGER.warn(ex);
+                LOGGER.error("Encoutered Exception while retrieving probeResult", ex);
             }
         }
         // mergeData phase 2

src/main/java/de/rub/nds/tlsscanner/SingleThreadedScanJobExecutor.java

@@ -80,7 +80,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
                         pb.step();
                     }
                 } catch (Exception E) {
-                    LOGGER.warn("Could not execute Probe", E);
+                    LOGGER.error("Could not execute Probe", E);
                 }
                 probeTypes.add(probe.getType());
             }
@@ -94,7 +94,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
             try {
                 result.merge(report);
             } catch (Exception E) {
-                LOGGER.warn("Could not merge SiteReport", E);
+                LOGGER.error("Could not merge SiteReport", E);
             }
         }
         //Finished phase one starting phase 2
@@ -117,7 +117,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
                             pb.step();
                         }
                     } catch (Exception E) {
-                        LOGGER.warn("Could not execute Probe", E);
+                        LOGGER.error("Could not execute Probe", E);
                     }
                 } else if (!config.isImplementation()) {
                     ProbeResult result = probe.getNotExecutedResult();
@@ -135,7 +135,7 @@ private SiteReport scan(ScannerConfig config, ScanJob scanJob, ProgressBar pb) {
             try {
                 result.merge(report);
             } catch (Exception E) {
-                LOGGER.warn("Could not merge SiteReport", E);
+                LOGGER.error("Could not merge SiteReport", E);
             }
         }
         //phase 3 - collect statistics

src/main/java/de/rub/nds/tlsscanner/probe/PaddingOracleProbe.java

@@ -118,7 +118,7 @@ public ProbeResult executeTest() {
                         try {
                             lastResult = attacker.isVulnerable();
                         } catch (Exception E) {
-                            LOGGER.warn("PaddingOracle Unstable - you should probably test this manually", E);
+                            LOGGER.error("Encountered an exception while testing for PaddingOracles", E);
                             lastResult = null;
                             hasError = true;
                         }

src/main/java/de/rub/nds/tlsscanner/probe/TokenbindingProbe.java

@@ -122,8 +122,7 @@ private Set<TokenBindingVersion> getSupportedVersions() {
                 }
 
             } catch (WorkflowExecutionException ex) {
-                LOGGER.warn("Could not execute Workflow to determine supported Tokenbinding Versions");
-                LOGGER.debug(ex);
+                LOGGER.error("Could not execute Workflow to determine supported Tokenbinding Versions", ex);
             }
         }
         return supportedVersions;

src/main/java/de/rub/nds/tlsscanner/probe/certificate/CertificateReportGenerator.java

@@ -123,7 +123,7 @@ private static void setPubkey(CertificateReportImplementation report, org.bouncy
                 report.setPublicKey(x509Cert.getPublicKey());
             }
         } catch (CertificateParsingException ex) {
-            LOGGER.warn("Could not parse PublicKey from certificate", ex);
+            LOGGER.error("Could not parse PublicKey from certificate", ex);
         }
     }