Skip to content

docs (scrypt): use ScryptParams::recommended() in usage example#50

Merged
newpavlov merged 1 commit intoRustCrypto:masterfrom
cbzehner:cbzehner/scrypt-example
Aug 24, 2020
Merged

docs (scrypt): use ScryptParams::recommended() in usage example#50
newpavlov merged 1 commit intoRustCrypto:masterfrom
cbzehner:cbzehner/scrypt-example

Conversation

@cbzehner
Copy link
Copy Markdown
Contributor

This PR changes the example in scrypt to generate the params using ScryptParams::recommended(), per discussion in #49.

Confirmed it's working by running doc tests.

❯ cargo test --doc
    Finished test [unoptimized + debuginfo] target(s) in 0.02s
   Doc-tests scrypt

running 1 test
test src/lib.rs -  (line 13) ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

@newpavlov
Copy link
Copy Markdown
Member

Thank you!

@newpavlov newpavlov merged commit f09f60d into RustCrypto:master Aug 24, 2020
@cbzehner cbzehner deleted the cbzehner/scrypt-example branch August 25, 2020 00:26
dns2utf8 pushed a commit to dns2utf8/password-hashes that referenced this pull request Jan 24, 2023
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.

2 participants