fix(MavenSupport): Also use the list of repos for the artifact cache key

Previously, the cache for the remote artifacts only used the artifact
coordinates as the cache key. But if the list of repos to query changes,
there may be another match for the same artifact in another repo. Thus,
also take the list of repos into account for the cache key.

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>

analyzer/src/main/kotlin/managers/utils/MavenSupport.kt

@@ -496,11 +496,6 @@ class MavenSupport(private val workspaceReader: WorkspaceReader) {
         repositories: List<RemoteRepository>,
         useReposFromDependencies: Boolean
     ): RemoteArtifact {
-        remoteArtifactCache.read(artifact.toString())?.let {
-            logger.debug { "Reading remote artifact for '$artifact' from disk cache." }
-            return yamlMapper.readValue(it)
-        }
-
         val allRepositories = if (useReposFromDependencies) {
             val repoSystem = containerLookup<RepositorySystem>()
 
@@ -513,6 +508,13 @@ class MavenSupport(private val workspaceReader: WorkspaceReader) {
             repositories
         }
 
+        val cacheKey = "$artifact@$allRepositories"
+
+        remoteArtifactCache.read(cacheKey)?.let {
+            logger.debug { "Reading remote artifact for '$artifact' from disk cache." }
+            return yamlMapper.readValue(it)
+        }
+
         // Filter out local repositories, as remote artifacts should never point to files on the local disk.
         val remoteRepositories = allRepositories.filterNot {
             // Some (Linux) file URIs do not start with "file://" but look like "file:/opt/android-sdk-linux".
@@ -630,7 +632,7 @@ class MavenSupport(private val workspaceReader: WorkspaceReader) {
 
                 return RemoteArtifact(info.downloadUrl, hash).also {
                     logger.debug { "Writing remote artifact for '$artifact' to disk cache." }
-                    remoteArtifactCache.write(artifact.toString(), yamlMapper.writeValueAsString(it))
+                    remoteArtifactCache.write(cacheKey, yamlMapper.writeValueAsString(it))
                 }
             } else {
                 logger.debug {
@@ -647,7 +649,7 @@ class MavenSupport(private val workspaceReader: WorkspaceReader) {
 
         return RemoteArtifact.EMPTY.also {
             logger.debug { "Writing empty remote artifact for '$artifact' to disk cache." }
-            remoteArtifactCache.write(artifact.toString(), yamlMapper.writeValueAsString(it))
+            remoteArtifactCache.write(cacheKey, yamlMapper.writeValueAsString(it))
         }
     }