fix(gradle-inspector): Keep the artifact URL on invalid hash values

Do not return `RemoteArtifact.EMPTY` early on invalid hash values but
maintain at least the artifact URL in any case.

As logging now is not only done for zero size artifacts but any failure,
also raise the log level from `info` to `warn`.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>

plugins/package-managers/gradle-inspector/src/main/kotlin/GradleDependencyHandler.kt

@@ -246,16 +246,18 @@ private fun createRemoteArtifact(
     }
 
     // TODO: How to handle authentication for private repositories here, or rely on Gradle for the download?
-    val checksum = okHttpClient.downloadText("$artifactUrl.$algorithm")
-        .getOrElse { return RemoteArtifact.EMPTY }
-
-    val hash = parseChecksum(checksum, algorithm)
+    val hash = okHttpClient.downloadText("$artifactUrl.$algorithm")
+        .mapCatching { checksum ->
+            parseChecksum(checksum, algorithm).also {
+                require(it.value != HashAlgorithm.SHA1.emptyValue) {
+                    "Ignoring invalid artifact of zero size at $artifactUrl."
+                }
+            }
+        }.getOrElse {
+            logger.warn("Unable to get a valid artifact checksum.", it)
 
-    // Ignore file with zero byte size, because it cannot be a valid archive.
-    if (hash.value == HashAlgorithm.SHA1.emptyValue) {
-        logger.info { "Ignoring zero byte size artifact: $artifactUrl" }
-        return RemoteArtifact.EMPTY
-    }
+            Hash.NONE
+        }
 
     return RemoteArtifact(artifactUrl, hash)
 }