Improve error handling for restore caching

[dmitry-shibanov]

Description:
In scope of this pull request we add error handling to throw error only for ValidationError.

Related issue: #295

Check list:

• Mark if documentation changes are required.
• Mark if tests were added or updated to cover the changes.

[Pompo17102]

Deposit through the bitcoin wallet for any help
bc1q5v34g2dym8c7x5mpfp7wdqp5hse38p99p4dggx

[dmitry-shibanov]

Hello @brcrista. Could you please take a look at this pull request?

[brcrista]

I agree with the change, but how does it relate to #295?

[brcrista]

@actions/actions-cache, do you have any feedback? For context, the setup- actions use the @actions/cache package to cache dependencies directly, which lets you omit actions/cache as a separate step.

[dmitry-shibanov]

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.

• https://github.com/actions/toolkit/blob/39b96406423e444b058d2602e2cea8f889342ee4/packages/cache/src/internal/downloadUtils.ts#L113-L115

[aparna-ravindra]

might this be related to actions/toolkit#1030 ?

[dmitry-shibanov]

Thank you @aparna-ravindra. It works. I've updated the pull request.

[dmitry-shibanov]

I'm going to close the pull request because the issue was resolved in scope of this pull request #308