Do not fail action on cache errors

Ensure that we catch and log errors in `beforeSave` and `afterRestore`,
and do not fail the entire workflow in these cases.
This commit is contained in:
Daz DeBoer 2021-10-14 12:19:24 -06:00
parent 0cf00ed767
commit 8ab7c9d8dd
No known key found for this signature in database
GPG key ID: DD6B9F0B06683D5D
5 changed files with 19 additions and 6 deletions

2
dist/main/index.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

2
dist/post/index.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

View file

@ -141,7 +141,13 @@ export abstract class AbstractCache {
`Restored ${this.cacheDescription} from cache key: ${cacheResult}` `Restored ${this.cacheDescription} from cache key: ${cacheResult}`
) )
try {
await this.afterRestore() await this.afterRestore()
} catch (error) {
core.warning(
`Restore ${this.cacheDescription} failed in 'afterRestore': ${error}`
)
}
return return
} }
@ -193,7 +199,14 @@ export abstract class AbstractCache {
return return
} }
try {
await this.beforeSave() await this.beforeSave()
} catch (error) {
core.warning(
`Save ${this.cacheDescription} failed in 'beforeSave': ${error}`
)
return
}
core.info( core.info(
`Caching ${this.cacheDescription} with cache key: ${cacheKey}` `Caching ${this.cacheDescription} with cache key: ${cacheKey}`