docs(security): clarify password and authentication token storage - #15476
Open
joshtrichards wants to merge 1 commit into
Open
docs(security): clarify password and authentication token storage#15476joshtrichards wants to merge 1 commit into
joshtrichards wants to merge 1 commit into
Conversation
Assisted-by: Copilot:gpt-5.6-sol Signed-off-by: Josh <josh.t.richards@gmail.com>
joshtrichards
requested review from
CarlSchwan,
ChristophWurst,
blizzz,
come-nc,
juliusknorr and
nickvergessen
August 22, 2026 17:20
Contributor
📖 Documentation Preview📄 1 changed documentation pageLast updated: Sat, 22 Aug 2026 17:26:13 GMT |
tflidd
reviewed
Aug 25, 2026
| are therefore required to decrypt the password stored in that token record. | ||
|
|
||
| The recoverable copy is used by features that require the login credentials, | ||
| such as connecting to external storage, configuring mail accounts, and |
Contributor
There was a problem hiding this comment.
I would put this part first for clarification. I was first thinking about the Nextcloud user password, why it needs to be store, if this is for the server-side encryption etc.
come-nc
approved these changes
Aug 25, 2026
artonge
reviewed
Aug 25, 2026
| ``auth.storeCryptedPassword``. Disabling it does not affect the one-way | ||
| account-password hash used by the built-in user backend, but features that | ||
| require recovery of the login password may no longer work. Password changes | ||
| made directly in an external user backend might also no longer automatically |
Contributor
There was a problem hiding this comment.
might also no longer automatically
There are a lot of incertitude in this PR. This is one example. It would be better to fact check them to be able to provide a more solid documentation. But I know it takes time 🙈
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
☑️ Resolves
N/A - prompted by a question about the ambiguity of the existing documentation on the forum along with my recurring own need to refer to the code and/or notes in this area.
Summary
Clarifies the password and authentication-token storage guidance in the server hardening documentation.
The existing text can be interpreted as saying that Nextcloud stores account passwords using reversible encryption instead of standard password hashing. In practice, these are separate mechanisms:
auth.storeCryptedPasswordis enabled and the login password is available, a separate, reversibly encrypted copy may be stored in the server-side record associated with an authentication token.This change distinguishes those representations and explains why a recoverable copy may exist, how it is protected, and the security consequences of a combined token, configuration, and database compromise.
It also clarifies that:
auth.storeCryptedPassword, subject to the documented functional consequences.Follow-ups/related:
auth.storeCryptedPasswordconfig parameter)TODO
🖼️ Screenshots
✅ Checklist
codespellor similar and addressed any spelling issues