src: handle out-of-size errors, and collisions gracefully (#3)

useblacksmith · Feb 3, 2025 · bafb57e · bafb57e
1 parent fe36458
commit bafb57e
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 1 deletion.
diff --git a/dist/setup/index.js b/dist/setup/index.js
diff --git a/dist/setup/index.js.map b/dist/setup/index.js.map
diff --git a/src/lib/cache/utils.ts b/src/lib/cache/utils.ts
@@ -63,6 +63,21 @@ export function getCacheClient() {
         }
       } else if (reserveCacheResponse?.statusCode === 409) {
         return { success: false }
+      } else if (reserveCacheResponse?.statusCode === 400) {
+        // Handle specific 400 error cases
+        const message = reserveCacheResponse.result?.message || 'Unknown error'
+
+        if (message.includes('Cache size exceeds maximum')) {
+          core.warning(message)
+        } else if (message.includes('Cache entry already exists')) {
+          core.info(message)
+        } else {
+          core.warning(`Unexpected 400 error: ${message}`)
+        }
+
+        // Don't continue if we have a 400 error since in all these cases we know we don't
+        // want to save the cache blob.
+        return { success: false }
       } else {
         const { statusCode, statusText } = reserveCacheResponse
         const data = reserveCacheResponse.result