Fixing comments

akto-api-security · Jan 2, 2025 · 440c629 · 440c629
1 parent c6d4841
commit 440c629
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/apps/dashboard/src/main/java/com/akto/action/testing/StartTestAction.java b/apps/dashboard/src/main/java/com/akto/action/testing/StartTestAction.java
@@ -37,6 +37,7 @@
 import org.bson.conversions.Bson;
 import org.bson.types.ObjectId;
 
+import java.nio.file.DirectoryStream.Filter;
 import java.time.Instant;
 import java.util.*;
 import java.util.concurrent.Executors;
@@ -513,7 +514,11 @@ public String fetchTestingRunResultSummary() {
 
     private static Bson vulnerableFilter = Filters.and(
         Filters.eq(TestingRunResult.VULNERABLE, true),
-        Filters.in(TestingRunResult.IS_IGNORED_RESULT, Arrays.asList(null, false))
+        Filters.or(
+            Filters.exists(TestingRunResult.IS_IGNORED_RESULT, false),
+            Filters.eq(TestingRunResult.IS_IGNORED_RESULT, false)
+        )
+
     );
 
     private List<Bson> prepareTestRunResultsFilters(ObjectId testingRunResultSummaryId, QueryMode queryMode) {

diff --git a/apps/dashboard/src/main/java/com/akto/action/testing_issues/IssuesAction.java b/apps/dashboard/src/main/java/com/akto/action/testing_issues/IssuesAction.java
@@ -546,7 +546,7 @@ public void run() {
                 Context.accountId.set(accountId);
                 try {
 
-                    Map<String, Integer> countIssuesMap = new HashMap<>();
+                    final Map<String, Integer> countIssuesMap = new HashMap<>();
                     countIssuesMap.put(Severity.HIGH.toString(), 0);
                     countIssuesMap.put(Severity.MEDIUM.toString(), 0);
                     countIssuesMap.put(Severity.LOW.toString(), 0);

diff --git a/libs/dao/src/main/java/com/akto/dao/testing/TestingRunResultDao.java b/libs/dao/src/main/java/com/akto/dao/testing/TestingRunResultDao.java
@@ -198,6 +198,9 @@ public Map<ObjectId,String> mapSummaryIdToTestingResultHexId(Set<String> testing
                                                 .map(ObjectId::new)
                                                 .collect(Collectors.toList());
 
+        // doing only for 1000 results at a time
+        objectIdList = objectIdList.subList(0, 1000);
+
         List<TestingRunResult> runResults = instance.findAll(Filters.in(Constants.ID, objectIdList), Projections.include(TestingRunResult.TEST_RUN_RESULT_SUMMARY_ID));
         for(TestingRunResult runResult: runResults){
             finalMap.put(runResult.getTestRunResultSummaryId(), runResult.getHexId());