fix: correctly return update status in KnownExploitedDataSource (#5441)

jeremylong · Feb 11, 2023 · 6cb9f33 · 6cb9f33
1 parent 182721d
commit 6cb9f33
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/core/src/main/java/org/owasp/dependencycheck/data/update/KnownExploitedDataSource.java b/core/src/main/java/org/owasp/dependencycheck/data/update/KnownExploitedDataSource.java
@@ -86,13 +86,14 @@ public boolean update(Engine engine) throws UpdateException {
                     }
                     //all dates in the db are now stored in seconds as opposed to previously milliseconds.
                     dbProperties.save(DatabaseProperties.KEV_LAST_CHECKED, Long.toString(System.currentTimeMillis() / 1000));
+                    return true;
                 }
             } catch (TooManyRequestsException | ResourceNotFoundException | IOException
                     | CorruptedDatastreamException | DatabaseException | SQLException ex) {
                 throw new UpdateException(ex);
             }
         }
-        return true;
+        return false;
     }
 
     @Override