bip39-rs
bip39-rs copied to clipboard
Review notes
Hi,
I've reviewed this crate using cargo-crev.
Some notes:
- maybe you should not implement
DebugforSeedandMnemonic.Debugmakes it easy to accidentally log them somewhere and thus leak them. - there are multiple issues that
cargo clippywill point out - nothing serious, but it is a good idea to just do what Clippy says. :)
Your whole comment comes across as a giant coat rack. Too little substance about this project, too much about giving publicity to your tool.
Just sayin'.
@vorot93 I'm not aware of "coat rack" phrase, but if you mean that I link to the review to increase visibility to increase the awareness of it, then it's true. Shamelessly. :)
As this is a bitcoin-related crate, a review by a reasonably reputable person is a valuable thing (IMO). I've spent around 20-30 minutes going through the code and reviewing it and I'm letting the author know that i did and what I think could improve, with a cryptographic proof. Some companies would pay me money for it.
So I think it's a fair game. If somebody minds, feel free to delete/close.