Skip to content

Conversation

@ChanTsune
Copy link
Contributor

For RustCrypto members

Thank you for developing a awesome crate!

I have a question.

Why do the other crates maintained by this repository have published Ident values, but not in pbkdf2?

Looking at the code, it seems that there is an intention to publish it, but it is not available because there is no use declaration.

If you just forgot to publish it, I would appreciate it if you could merge this PR and make it available.

@tarcieri
Copy link
Member

Before exporting these I think we might consider better names that reflect they're Idents.

Perhaps either adding *_IDENT to their names or making them inherent constants like Pbkdf2::SHA256_IDENT.

@ChanTsune
Copy link
Contributor Author

ChanTsune commented Feb 16, 2023

Thanks for your advice!
I added _IDENT as a suffix to each.

@tarcieri
Copy link
Member

Personally I think I'd prefer the inherent constant approach, since it cuts down on the number of exports/imports

@ChanTsune
Copy link
Contributor Author

OK!, so how about making it an intrinsic constant of Algorithm following the other crates as follows?

https://github.com/ChanTsune/password-hashes/blob/ca2ff2abdfa347ce7d6a3e4bd764f988ba9a282c/pbkdf2/src/simple.rs#L82-L94

@tarcieri
Copy link
Member

That looks good, yes

Copy link
Member

@tarcieri tarcieri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@tarcieri tarcieri merged commit 86bdad3 into RustCrypto:master Feb 17, 2023
@ChanTsune ChanTsune deleted the pbkdf2 branch February 17, 2023 23:24
@tarcieri tarcieri mentioned this pull request Mar 5, 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