Skip to content

Commit

Permalink
Merge pull request #48947 from nextcloud/bugfix/noid/mark-more-config…
Browse files Browse the repository at this point in the history
…s-as-sensitive

fix(config): Mark more configs as sensitive
  • Loading branch information
nickvergessen authored Oct 28, 2024
2 parents 988b600 + 1e5cb82 commit 8e6fd4d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
9 changes: 9 additions & 0 deletions lib/private/AppConfig.php
Original file line number Diff line number Diff line change
Expand Up @@ -1430,10 +1430,16 @@ private function getSensitiveKeys(string $app): array {
'/^key_pairs$/',
'/^local_gskey$/',
],
'call_summary_bot' => [
'/^secret_(.*)$/',
],
'external' => [
'/^sites$/',
'/^jwt_token_privkey_(.*)$/',
],
'globalsiteselector' => [
'/^gss\.jwt\.key$/',
],
'integration_discourse' => [
'/^private_key$/',
'/^public_key$/',
Expand Down Expand Up @@ -1534,6 +1540,9 @@ private function getSensitiveKeys(string $app): array {
'user_saml' => [
'/^idp-x509cert$/',
],
'whiteboard' => [
'/^jwt_secret_key$/',
],
];

return $sensitiveValues[$app] ?? [];
Expand Down
1 change: 1 addition & 0 deletions lib/private/SystemConfig.php
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,7 @@ class SystemConfig {
'onlyoffice' => [
'jwt_secret' => true,
],
'PASS' => true,
];

public function __construct(
Expand Down

0 comments on commit 8e6fd4d

Please sign in to comment.