Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Replace react-native-randombytes with get-random-values#8045

Merged
Overtorment merged 3 commits into
masterfrom
cursor/replace-react-native-randombytes-with-get-random-values-379c
Aug 19, 2025
Merged

Replace react-native-randombytes with get-random-values#8045
Overtorment merged 3 commits into
masterfrom
cursor/replace-react-native-randombytes-with-get-random-values-379c

Conversation

@Overtorment

Copy link
Copy Markdown
Member

Replace react-native-randombytes with react-native-get-random-values to use a more standard and maintained random number polyfill.


Open in Cursor Open in Web

@cursor

cursor Bot commented Aug 15, 2025

Copy link
Copy Markdown

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@Overtorment Overtorment marked this pull request as ready for review August 15, 2025 22:04
@Overtorment Overtorment linked an issue Aug 16, 2025 that may be closed by this pull request
Comment thread class/rng.ts
else resolve(data);
});
});
const g: any = globalThis as any;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any as any

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah shit ai slop

@GladosBlueWallet

Copy link
Copy Markdown
Collaborator

Wake the fuck up samurai, we have PRs to merge

image

[all PRs for @junderw] https://github.com/BlueWallet/BlueWallet/pulls/review-requested/junderw

@Overtorment Overtorment merged commit 0abe089 into master Aug 19, 2025
13 of 14 checks passed
@Overtorment Overtorment deleted the cursor/replace-react-native-randombytes-with-get-random-values-379c branch August 19, 2025 09:17
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.

migrate to crypto.getRandomValues() from crypto.randomBytes()

6 participants