Improve error handling for restore caching#297
Improve error handling for restore caching#297dmitry-shibanov wants to merge 2 commits intoactions:mainfrom
Conversation
|
Deposit through the bitcoin wallet for any help |
|
Hello @brcrista. Could you please take a look at this pull request? |
|
@actions/actions-cache, do you have any feedback? For context, the |
|
Hello @brcrista. Actually, as I checked the issue reproduces with node 16 only. On the previous action version it works as expected. I think it's related itself to the change of the default node version and buffer size. I could reproduce the same behaviour with action/cache when I removed this error handling logic and archive size was more than 2147483647. |
|
might this be related to actions/toolkit#1030 ? |
|
Thank you @aparna-ravindra. It works. I've updated the pull request. |
|
I'm going to close the pull request because the issue was resolved in scope of this pull request #308 |
Description:
In scope of this pull request we add error handling to throw error only for ValidationError.
Related issue: #295
Check list: