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.

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

Password Registry #3609

Closed
wants to merge 4 commits into from
Closed

Password Registry #3609

wants to merge 4 commits into from

Conversation

@sgolemon
Copy link
Contributor

sgolemon commented Oct 16, 2018

WIP implementation for https://wiki.php.net/rfc/password_registry

f2660ac represents the least user-visible change relative to the current state. Some things I'd do if I weren't concerned with BC are:

  1. Remove all the argon stuff and just make ext/sodium the only implementation.
  2. Switch from integers to strings for the mechanism selectors.
@php-pulls

This comment has been minimized.

Copy link

php-pulls commented Oct 16, 2018

Comment on behalf of petk at php.net:

Labelling

@php-pulls php-pulls added the RFC label Oct 16, 2018
sgolemon added 3 commits Oct 18, 2018
@sgolemon sgolemon closed this Dec 6, 2018
jrfnl added a commit to PHPCompatibility/PHPCompatibility that referenced this pull request Jun 26, 2019
> - Standard
>    array password_algos() - return a complete list of all registered password
>    hashing algorithms. For more details see the RFC:
>    https://wiki.php.net/rfc/password_registry

Refs:
* https://wiki.php.net/rfc/password_registry
* php/php-src#3609
* php/php-src@534df87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.