Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

use bcrypt.checkpw #2949

Merged
merged 1 commit into from Mar 6, 2018
Merged

Conversation

krombel
Copy link
Contributor

@krombel krombel commented Mar 5, 2018

in bcrypt 3.1.0 checkpw got introduced (nearly 2 years ago)
This makes use of that with enhancements which might get introduced
by that.
The concrete PR for that feature is pyca/bcrypt#76

Signed-Off-by: Matthias Kesler krombel@krombel.de

@matrixbot
Copy link
Member

Can one of the admins verify this patch?

1 similar comment
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

in bcrypt 3.1.0 checkpw got introduced (already 2 years ago)
This makes use of that with enhancements which might get introduced
by that

Signed-Off-by: Matthias Kesler <krombel@krombel.de>
@richvdh
Copy link
Member

richvdh commented Mar 6, 2018

this is the opposite to #619, but it's true that bcrypt reintroduced checkpw a few months later (not quite 2 years ago).

Given that debian stable has 3.1.2, I can't see much argument against this.

@richvdh
Copy link
Member

richvdh commented Mar 6, 2018

@matrixbot: ok to test

@richvdh richvdh merged commit 8ffaacb into matrix-org:develop Mar 6, 2018
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

1 similar comment
@matrixbot
Copy link
Member

Can one of the admins verify this patch?

@krombel krombel deleted the use_bcrypt_checkpw branch March 6, 2018 13:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants