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.

Sign up for GitHub

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

Jump to bottom

crypto.pwhash.bcrypt: make silently_truncate_password a member of Params #22792

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

crypto.pwhash.bcrypt: make silently_truncate_password a member of Params #22792

wants to merge 2 commits into from

Conversation

jedisct1
Copy link
Contributor

@jedisct1 jedisct1 commented Feb 6, 2025

This removes the need for having both bcrypt() and bcryptWithTruncation() in the public API.

And whether truncation happens or not becomes even more explicit.

@jedisct1 jedisct1 added breaking Implementing this issue could cause existing code to no longer compile or have different behavior. standard library This issue involves writing Zig code for the standard library. labels Feb 6, 2025
@jedisct1
bcrypt: make silently_truncate_password a member of Params
6a5c695 
This removes the need for having both `bcrypt()` and
`bcryptWithTruncation()` in the public API.

And whether truncation happens or not becomes even more explicit.
@jedisct1
Update crypto benchmark
b338921
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaking Implementing this issue could cause existing code to no longer compile or have different behavior. standard library This issue involves writing Zig code for the standard library.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jedisct1