Skip to content

fix(appconfig): allow overwriting a sensitive value that cannot be decrypted - #66116

Open
lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-63807-sensitive-appconfig-overwrite
Open

fix(appconfig): allow overwriting a sensitive value that cannot be decrypted#66116
lazerg wants to merge 1 commit into
nextcloud:masterfrom
lazerg:fix/issue-63807-sensitive-appconfig-overwrite

Conversation

@lazerg

@lazerg lazerg commented Oct 31, 2026

Copy link
Copy Markdown

Summary

When the instance secret changes, a stored sensitive value can no longer be decrypted, and setTypedValue() threw while comparing it to the new value, so the key could never be overwritten. Now an undecryptable stored value just counts as different and gets replaced.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI
@lazerg
fix(appconfig): allow overwriting a sensitive value that cannot be de…
a5933f3 
…crypted

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: lazerg <lazerg2@gmail.com>
@lazerg
lazerg requested a review from a team as a code owner October 31, 2026 08:15
@lazerg
lazerg requested review from Altahrim, icewind1991, leftybournes and salmart-dev and removed request for a team October 31, 2026 08:15

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@lazerg