bip39-rs icon indicating copy to clipboard operation
bip39-rs copied to clipboard

Review notes

Open dpc opened this issue 6 years ago • 2 comments

Hi,

I've reviewed this crate using cargo-crev.

Some notes:

  • maybe you should not implement Debug for Seed and Mnemonic. Debug makes it easy to accidentally log them somewhere and thus leak them.
  • there are multiple issues that cargo clippy will point out - nothing serious, but it is a good idea to just do what Clippy says. :)

dpc avatar Jul 19 '19 22:07 dpc

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 avatar Jul 19 '19 22:07 vorot93

@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.

dpc avatar Jul 19 '19 22:07 dpc