NotificationsYou must be signed in to change notification settings
Fork9.6k
Star30.4k

[Security] Outsource all the BCrypt heavy lifting to a library#7853

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

fabpot merged 2 commits intosymfony:masterfromfabpot:bcrypt

Apr 25, 2013

Merged

[Security] Outsource all the BCrypt heavy lifting to a library#7853

fabpot merged 2 commits intosymfony:masterfromfabpot:bcrypt

Apr 25, 2013

Conversation

Copy link

Member

fabpot commentedApr 25, 2013

Q	A
Bug fix?	no
New feature?	no
BC breaks?	yes
Deprecations?	no
Tests pass?	yes
License	MIT

TheBCrypt bundle is already using the library.

This is a working implementation of#7247

elnurand others added2 commits

April 25, 2013 17:38

Outsource all the BCrypt heavy lifting to a library

b2e553a

[Security] tweaked previous commit

c83546d

fabpot mentioned this pull request

Apr 25, 2013

[2.2][Security] Outsource all the BCrypt heavy lifting to a library#7247

Closed

fabpot added a commit that referenced this pull request

Apr 25, 2013

merged branch fabpot/bcrypt (PR#7853)

ccc52a4

This PR was merged into the master branch.Discussion----------[Security] Outsource all the BCrypt heavy lifting to a library| Q             | A| ------------- | ---| Bug fix?      | no| New feature?  | no| BC breaks?    | yes| Deprecations? | no| Tests pass?   | yes| License       | MITThe [BCrypt bundle](https://github.com/elnur/ElnurBlowfishPasswordEncoderBundle) is already using the library.This is a working implementation of#7247Commits-------c83546d [Security] tweaked previous commitb2e553a Outsource all the BCrypt heavy lifting to a library

fabpot closed this

Apr 25, 2013

fabpot merged commitc83546d intosymfony:master

Apr 25, 2013