Merge pull request #143 from snyk/feat/SLS-709-last-modified-delay-download

SLS-709 - Add debug logging for last modified date determination

Author: samsnyk

core/src/main/java/io/snyk/plugins/artifactory/scanner/ScannerModule.java

@@ -98,10 +98,16 @@ private void filter(MonitoredArtifact artifact) {
   private @NotNull MonitoredArtifact toMonitoredArtifact(TestResult testResult, @NotNull RepoPath repoPath) {
     Ignores ignores = Ignores.read(new RepositoryArtifactProperties(repoPath, repositories));
     Instant lastModifiedDate = getLastModifiedDate(repoPath);
+
+    LOG.debug("Retrieved last modified date of {} for {}", lastModifiedDate, repoPath);
 
     // Only apply lastModifiedDate to packages from remote repositories.
-    if(lastModifiedDateRemoteOnly() && !isRemoteRepository(repoPath)) {
-      lastModifiedDate = null;
+    if(lastModifiedDateRemoteOnly()) {
+      LOG.debug("Last modified date applied to only remote repositories.");
+      if (!isRemoteRepository(repoPath)) {
+        LOG.debug("Repository provided is not a remote repository, skipping last modified date check.");
+        lastModifiedDate = null;
+      }
     }
     return new MonitoredArtifact(repoPath.toString(), testResult, ignores, lastModifiedDate);
   }
@@ -111,7 +117,10 @@ private Instant getLastModifiedDate(RepoPath repoPath) {
       ItemInfo itemInfo = repositories.getItemInfo(repoPath);
       if (itemInfo != null) {
         Instant lastModified = Instant.ofEpochMilli(itemInfo.getLastModified());
+        LOG.debug("Retrieved last modifed date of {}", lastModified.toString());
         return lastModified;
+      } else {
+        LOG.debug("Unable to retrieve ItemInfo for {}, could not retrieve last modified date", repoPath);
       }
     } catch (Exception e) {
       LOG.debug("Could not retrieve last modified date for {}: {}", repoPath, e);