Skip to content

docs(auth): clarify recoverable password storage - #63806

Open
joshtrichards wants to merge 2 commits into
masterfrom
jtr/docs-config-auth-crypted-length
Open

docs(auth): clarify recoverable password storage#63806
joshtrichards wants to merge 2 commits into
masterfrom
jtr/docs-config-auth-crypted-length

Conversation

@joshtrichards

@joshtrichards joshtrichards commented Aug 30, 2026

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

Clarify the documentation for auth.storeCryptedPassword:

  • clearly distinguish the recoverable password copy stored with authentication tokens from the one-way password hash used for account authentication;
  • replace the approximate password-length threshold of 300 characters with the actual thresholds, measured in bytes;
  • document that passwords longer than 214 bytes require a larger RSA key, increasing token-generation overhead;
  • explain that disabling the setting avoids password-length-related RSA key-size increases; and
  • clarify that Nextcloud's 469-byte account-password maximum remains enforced whether this option is enabled or disabled.

The 214-byte threshold reflects the corrected RSA-OAEP key-size selection in #63804. It is the actual maximum plaintext size for the existing 2048-bit RSA-OAEP configuration, rather than the approximate 300-character guidance. Before #63804, the implementation switched to a larger key only above 250 bytes, leaving passwords from 215 through 250 bytes assigned to a key too small to encrypt them. (In other words, 214 was already the true cryptographic maximum, but it was not the implemented key-selection threshold before #63804).

TODO

  • Backport to v35
  • Backport to v34-v33?

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Explain when recoverable login passwords are stored in authentication-token records and the consequences.

Document the actual RSA key-size performance threshold (214 and neither "300" nor 250) and clarify that the 469-byte account-password limit applies regardless of this option.

Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant