setup-node icon indicating copy to clipboard operation
setup-node copied to clipboard

Replace uuid with crypto.randomUUID()

Open trivikr opened this issue 5 months ago • 2 comments

Description:

Replace uuid with crypto.randomUUID()

https://github.com/actions/setup-node/blob/89d709d423dc495668cd762a18dd4a070611be3f/src/distributions/base-distribution.ts#L1

Justification:

crypto was added globally as follows:

  • v23.0.0 | No longer experimental.
  • v19.0.0 | No longer behind --experimental-global-webcrypto CLI flag.
  • v17.6.0, v16.15.0 |  Added in: v17.6.0, v16.15.0

Docs: https://nodejs.org/api/globals.html#crypto

Are you willing to submit a PR?

Yes

trivikr avatar Sep 24 '25 00:09 trivikr

Hi @trivikr, Thank you for creating this feature request. We will investigate it and provide feedback as soon as we have some updates.

mahabaleshwars avatar Sep 24 '25 07:09 mahabaleshwars

I've posted a PR at https://github.com/actions/setup-node/pull/1378

trivikr avatar Sep 24 '25 14:09 trivikr