Reduce Database Write Clutter in Nmap Process

scb-engine/src/main/java/io/securecodebox/engine/execution/DefaultScanProcessExecution.java

@@ -134,6 +134,13 @@ public void appendFinding(Finding finding) {
         writeToProcess(DefaultFields.PROCESS_FINDINGS, findings);
     }
 
+    @Override
+    public void appendFindings(List<Finding> newFindings) {
+        List<Finding> findings = getJsonFromProcessVariableModifiable(DefaultFields.PROCESS_FINDINGS, Finding.class);
+        findings.addAll(newFindings);
+        writeToProcess(DefaultFields.PROCESS_FINDINGS, findings);
+    }
+
     @Override
     public void clearTargets() {
         writeToProcess(DefaultFields.PROCESS_TARGETS, new LinkedList<>());

scb-scanprocesses/combined-amass-nmap-process/src/main/java/io/securecodebox/scanprocesses/amassnmap/FilterHttpSecurityHeaders.java

@@ -56,7 +56,7 @@ public void execute(DelegateExecution delegateExecution) throws Exception {
         final long tStrategiesApplied = System.currentTimeMillis();
         final int numberOfAdditionalFindings = findings.size() - process.getFindings().size();
         clearFindings(process);
-        findings.forEach(changedFinding -> process.appendFinding(changedFinding));
+        process.appendFindings(findings);
         LOG.debug("http-headers strategies yielded {} additional findings; finding them took {}ms, storing them {}ms", numberOfAdditionalFindings, tStrategiesApplied - tStart, System.currentTimeMillis() - tStrategiesApplied);
     }
 

scb-sdk/src/main/java/io/securecodebox/model/execution/ScanProcessExecution.java

@@ -89,6 +89,9 @@ public interface ScanProcessExecution {
     @JsonIgnore
     void appendFinding(Finding finding);
 
+    @JsonIgnore
+    void appendFindings(List<Finding> newFindings);
+
     void appendTarget(Target target);
 
     List<Target> getTargets();