🥅 Ignore unreadable files when hasing (and print warning)

burrunan · Aug 11, 2020 · 01d4c8f · 01d4c8f
1 parent 6d3e893
commit 01d4c8f
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 5 deletions.
diff --git a/layered-cache/src/main/kotlin/com/github/burrunan/gradle/hashFiles.kt b/layered-cache/src/main/kotlin/com/github/burrunan/gradle/hashFiles.kt
@@ -15,6 +15,7 @@
  */
 package com.github.burrunan.gradle
 
+import actions.core.warning
 import crypto.createHash
 import actions.glob.create
 import com.github.burrunan.wrappers.nodejs.pipe
@@ -55,11 +56,17 @@ suspend fun hashFiles(vararg paths: String, algorithm: String = "sha1"): HashRes
         numFiles += 1
         totalBytes += statSync.size.toInt()
         // Add filename
-        hash.update(key, "utf8")
 
-        fs.createReadStream(name).use {
-            it.pipe(hash, end = false)
+        try {
+            fs.createReadStream(name).use {
+                it.pipe(hash, end = false)
+            }
+        } catch (e: Throwable) {
+            warning("Unable to hash $name, will ignore the file: ${e.stackTraceToString()}")
+            continue
         }
+
+        hash.update(key, "utf8")
     }
     hash.end()
     HashResult(

diff --git a/layered-cache/src/main/kotlin/com/github/burrunan/gradle/hashing/HashDetails.kt b/layered-cache/src/main/kotlin/com/github/burrunan/gradle/hashing/HashDetails.kt
@@ -15,6 +15,7 @@
  */
 package com.github.burrunan.gradle.hashing
 
+import actions.core.warning
 import crypto.createHash
 import fs.createReadStream
 import fs2.promises.stat
@@ -92,8 +93,13 @@ suspend fun hashFilesDetailed(vararg paths: String, algorithm: String = "sha1"):
                 key.substringAfterLast('/')
             else -> {
                 val hash = createHash(algorithm)
-                createReadStream(name).use {
-                    it.pipe(hash)
+                try {
+                    createReadStream(name).use {
+                        it.pipe(hash)
+                    }
+                } catch (e: Throwable) {
+                    warning("Unable to hash $name, will ignore the file: ${e.stackTraceToString()}")
+                    continue
                 }
                 hash.digest().toString("hex")
             }