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

Rewrite Crypto WASM using the original iroha_crypto crate #188

Merged
merged 5 commits into from
Mar 14, 2024

Conversation

0x009922
Copy link
Contributor

@0x009922 0x009922 commented Mar 5, 2024

Closes #186

See changesets and the issue for context.

Additionally, I have significantly extended integration tests around crypto.

@0x009922 0x009922 added enhancement iroha2 package: crypto Related to Crypto packages labels Mar 5, 2024
@0x009922 0x009922 self-assigned this Mar 5, 2024
@0x009922 0x009922 marked this pull request as draft March 5, 2024 09:00
0x009922 added 2 commits March 5, 2024 18:00
`cargo check --target wasm32-unknown-unknown` fails:
`getrandom` crate features are invalid

Signed-off-by: Dmitry Balashov <[email protected]>
@0x009922 0x009922 force-pushed the iroha2-upstream-crypto branch from 728d793 to 098226e Compare March 5, 2024 09:00
@DCNick3
Copy link

DCNick3 commented Mar 7, 2024

I don't know that much about wasm-bindgen, but otherwise crypto-rs LGTM

@0x009922
Copy link
Contributor Author

0x009922 commented Mar 8, 2024

@DCNick3, thanks!

@0x009922
Copy link
Contributor Author

This PR needs to update due to merge of hyperledger-iroha/iroha#4341

Signed-off-by: Dmitry Balashov <[email protected]>
@0x009922 0x009922 marked this pull request as ready for review March 13, 2024 10:32
@0x009922 0x009922 merged commit 787a198 into hyperledger-iroha:iroha2 Mar 14, 2024
2 checks passed
@0x009922 0x009922 deleted the iroha2-upstream-crypto branch March 14, 2024 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants