feat: implement getRandomValues and randomUUID - resolves #213 and #214

[markhughes]

This implements https://github.com/crypto-browserify/crypto-browserify/issues/213 and https://github.com/crypto-browserify/crypto-browserify/issues/214

[Gambitboy]

Any update on getting this merged or looked at?

[brunomacf]

Any updates? This is important so we can try replace react-native-crypto by crypto-browserify.

[markhughes]

unsure about the status of crypto-browserify, I think the project is not maintained TBH. Happy to help maintain if current maintainers are inactive?

[3cp]

@markhughes do you want to maintain a fork? I will be one of many who are interested to use.

[Gambitboy]

@jprichardson , @calvinmetcalf , @ForbesLindesay or @fanatid any assistance?

[fanatid]

Hi, two things:

• can you move tape update to another PR?
• is it possible to leave links to node source code for verification that everything is the same?

Thanks!

[eggplantzzz]

Just stopping by to say I would also be eager to see getRandomValues implemented in this library! Thanks @markhughes!

[markhughes]

Sorry I will fix this PR up later this week, I've been sick and in-out of hospital

[shmillar]

Hello @markhughes , any updates on adding randomUUID() to crypto-browserify?

[BenGu3]

Also curious about when (if 😅) this will be merged in

[markhughes]

Hi @fanatid sorry for the delay I've been dealing with some health issues!

can you move tape update to another PR?

Got it out of here, thanks. Not sure how that one snuck in but it was required to run on my computer.

is it possible to leave links to node source code for verification that everything is the same?

Perhaps just the specs?

• https://w3c.github.io/webcrypto/#Crypto-method-getRandomValues
• https://w3c.github.io/webcrypto/#Crypto-method-randomUUID

Edit: which matches

• https://nodejs.org/api/webcrypto.html#cryptogetrandomvaluestypedarray
• https://nodejs.org/api/webcrypto.html#cryptorandomuuid

[markhughes]

Hey @fanatid, @jprichardson, @calvinmetcalf, @ForbesLindesay, @dcousens - you seem like the more recent past contributors in the browserify package. Could we possibly look at this?

[jhenriquez]

Any updates? Is there an alternative package anyone is using?

CC: @markhughes @dcousens

[markhughes]

It just needs to be merged. I'm using it for my own projects, but I don't think browserify is maintained like it used to be

[jhenriquez-ax]

Thank you @markhughes, doesn’t seem that anyone is maintaining the library. We forked yours so we can use it indefinitely.

Hopefully someone will take over maintenance here. This library is a recommended webpack 5 polyfill.

[sgtwilko]

Hiya @ljharb any chance this can be reviewed/merged?

[Vitalii4as]

Can't we use randomUUID from browser Crypto API here?

[ljharb]

Not in browsers that don’t have it.