Skip to content

fix(config): Decrypt sensitive appconfigs when requested#61608

Merged
nickvergessen merged 1 commit into
masterfrom
bugfix/noid/decrypt-sensitive-values
Jun 29, 2026
Merged

fix(config): Decrypt sensitive appconfigs when requested#61608
nickvergessen merged 1 commit into
masterfrom
bugfix/noid/decrypt-sensitive-values

Conversation

@nickvergessen

Copy link
Copy Markdown
Member
occ config:app:set spreed recording_servers --value '{"secret":"abc"}' --sensitive

Before

occ config:list spreed --private
{
    "apps": {
        "spreed": {
…
            "recording_servers": "$AppConfigEncryption$bf2f2eea60075e78a26e608f9eef90b750c8205eddd4e15cc968569af7ae78a3|55aaf290e56e56494cb8cc0b827c121a|61442de1d928f0ad2eeff1c74bd608f6ef18dac87ab7f39075fffdbdc3d10797cf02ce2f2e604bdf775d7add6584721b0d63ba4b0e2696d3fdf94e1659366bb6|3",
…
        }
    }
}

occ config:list spreed
{
    "apps": {
        "spreed": {
…
            "recording_servers": "***REMOVED SENSITIVE VALUE***",
…
        }
    }
}

After

occ config:list spreed --private
{
    "apps": {
        "spreed": {
…
            "recording_servers": "{\"secret\":\"abc\"}",
…
        }
    }
}

occ config:list spreed
{
    "apps": {
        "spreed": {
…
            "recording_servers": "***REMOVED SENSITIVE VALUE***",
…
        }
    }
}

Checklist

AI (if applicable)

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

@nickvergessen nickvergessen added this to the Nextcloud 35 milestone Jun 26, 2026
@nickvergessen nickvergessen self-assigned this Jun 26, 2026
@nickvergessen nickvergessen requested a review from a team as a code owner June 26, 2026 10:47
@nickvergessen nickvergessen added bug 3. to review Waiting for reviews labels Jun 26, 2026
@nickvergessen nickvergessen requested review from ArtificialOwl, come-nc, provokateurin and salmart-dev and removed request for a team June 26, 2026 10:47
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/decrypt-sensitive-values branch from 55b674a to c94bea0 Compare June 26, 2026 11:21
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable34

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable33

@nickvergessen nickvergessen merged commit 6a739c0 into master Jun 29, 2026
194 of 198 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/decrypt-sensitive-values branch June 29, 2026 16:47
@DerDreschner

Copy link
Copy Markdown
Contributor

/backport to stable32

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

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants