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

Consider using react-native-quick-crypto #284

Open
mrousavy opened this issue Oct 6, 2022 · 2 comments
Open

Consider using react-native-quick-crypto #284

mrousavy opened this issue Oct 6, 2022 · 2 comments

Comments

@mrousavy
Copy link

mrousavy commented Oct 6, 2022

Hey!

I noticed this repo uses react-native-crypto, which itself uses crypto-browserify under the hood. crypto-browserify is a JS-based implementation for crypto, which is slower than Margelo's react-native-quick-crypto, since this uses C++ (OpenSSL) under the hood.

If you want to replace react-native-crypto with the faster react-native-quick-crypto, I'm happy to help out or answer questions.

@mrousavy mrousavy changed the title Consider using react-native-crypto Consider using react-native-quick-crypto Oct 6, 2022
@andymatuschak
Copy link
Owner

Thanks; this is helpful. I'd like to eventually move all my dependencies to JSI…

@andymatuschak
Copy link
Owner

(Note to self: it appears that we're no longer actually using crypto in app—we can probably just remove the dependency.)

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

No branches or pull requests

2 participants