Skip to content

[Sync EN] Clarify Argon2 memory_cost and time_cost units in password docs - #992

Open
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:sync-en/973
Open

[Sync EN] Clarify Argon2 memory_cost and time_cost units in password docs#992
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:sync-en/973

Conversation

@lacatoire

Copy link
Copy Markdown
Member

Fixes #973

Update EN-Revision to 6cc48aed6f887bdabf09c568f854099f9c74e017 for both reference/password/constants.xml and reference/password/functions/password-hash.xml.

Changes mirroring the EN commit:

constants.xml:

  • PASSWORD_BCRYPT_DEFAULT_COST: fill in previously empty description (uses <simpara> as in EN)
  • time_cost option: clarify as "number of iterations (pasadas)" instead of "maximum duration"
  • PASSWORD_ARGON2_DEFAULT_MEMORY_COST: correct unit from "bytes" to "kibibytes (KiB)"; convert <para> to <simpara>
  • PASSWORD_ARGON2_DEFAULT_TIME_COST: clarify as "number of iterations (pasadas)" instead of "time"; convert <para> to <simpara>

password-hash.xml:

  • time_cost option: same clarification as above; convert <para> to <simpara>

…arifications

EN-Revision updated:
- constants.xml: 66aff414 -> 6cc48aed
- password-hash.xml: 5003a6ea -> 6cc48aed

Changes:
- Fill in PASSWORD_BCRYPT_DEFAULT_COST description (simpara x2)
- Clarify time_cost as iteration count (not duration)
- Fix memory_cost unit from bytes to kibibytes (KiB)
- Clarify PASSWORD_ARGON2_DEFAULT_TIME_COST as iteration count
- Convert para to simpara where EN changed structure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sync EN] Clarify Argon2 memory_cost and time_cost units in password_hash() (#5219)

1 participant