Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove herumi browser specific code #66

Closed
wants to merge 3 commits into from

Conversation

dapplion
Copy link
Contributor

@dapplion dapplion commented Dec 6, 2020

Updated by @matthewkeil:
Came to delete this PR but noticed it sort of ties in with what @jeluard is doing in #178 . Not sure if we want to keep this or close it.... Up for review though so we can clean the PR board.

yarn.lock Outdated Show resolved Hide resolved
@matthewkeil matthewkeil force-pushed the dapplion/bump-herumi-uint8-bug branch from f544c28 to aeaf4a4 Compare April 22, 2024 09:15
@dapplion dapplion requested a review from a team as a code owner April 22, 2024 09:15
@matthewkeil
Copy link
Member

I think this probably should get deleted... What do you think @jeluard and @wemeetagain ?

@jeluard
Copy link
Contributor

jeluard commented Apr 25, 2024

Once bls drops supports for node 18 we will be able to upgrade bls-eth-wasm that doesn't require providing a getRandomValues implementation, making this PR irrelevant.

Until then I think we should be good? Although I am not clear what the use of noble's randomBytes provide us here (as herumi should be mostly for browser usage). Or is it for runtime without napi nor webcrypto, but node:crypto support?

@matthewkeil
Copy link
Member

Agreed @jeluard. Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants