Fix #78969 Add password_default_algo() function#5104
Fix #78969 Add password_default_algo() function#5104kocsismate wants to merge 1 commit intophp:masterfrom
Conversation
9089c32 to
99f134f
Compare
|
As I asked on the ML, is there any reason why we can't make The |
Yes! I was considering to add Speaking about the |
|
@kocsismate I'm suggesting to change PASSWORD_DEFAULT for 7.4.3, as a fix for an unintentional BC break in 7.4.0. |
|
@nikic I got your point now. I'll get this ready in this PR. |
|
I chose to create a new PR instead. |
I think it makes sense adding a way for users to determine the actual default password hashing algorithm.
I chose
password_default_algo()to return not just the algorithm's name, but the ID too (which are different in case of Bcrypt). I think it is worth to return the default options as well. This way,password_default_algo()will be in line with the data structure returned bypassword_get_info().