Skip to content

fix(config): Censor more app configs in system report - #52424

Merged
AndyScherzinger merged 1 commit into
masterfrom
bugfix/noid/censor-more-app-configs
Apr 25, 2025
Merged

fix(config): Censor more app configs in system report#52424
AndyScherzinger merged 1 commit into
masterfrom
bugfix/noid/censor-more-app-configs

Conversation

@nickvergessen

@nickvergessennickvergessen commented Apr 25, 2025

Copy link
Copy Markdown
Member

Checklist

@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Apr 25, 2025
@nickvergessennickvergessen added this to the Nextcloud 32 milestone Apr 25, 2025
@nickvergessennickvergessen self-assigned this Apr 25, 2025
@nickvergessen
nickvergessen requested a review from a team as a code ownerApril 25, 2025 06:44
@nickvergessen
nickvergessen requested review from ArtificialOwl, icewind1991 and sorbaugh and removed request for a teamApril 25, 2025 06:44
@nickvergessen

nickvergessen commented Apr 25, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable31

@nickvergessen

nickvergessen commented Apr 25, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable30

@nickvergessen

nickvergessen commented Apr 25, 2025

Copy link
Copy Markdown
MemberAuthor

/backport to stable29

susnux
susnux reviewed Apr 25, 2025
View reviewed changes
Comment threadlib/private/AppConfig.php
@nickvergessen
fix(config): Censor more app configs in system report
8524955
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/censor-more-app-configs branch from 012d19d to 8524955CompareApril 25, 2025 09:01
@AndyScherzinger
AndyScherzinger merged commit f154e60 into masterApr 25, 2025
@AndyScherzinger
AndyScherzinger deleted the bugfix/noid/censor-more-app-configs branch April 25, 2025 11:03
This was referenced Apr 25, 2025
Merged
Merged
Merged
@skjnldsvskjnldsv mentioned this pull request Aug 19, 2025
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnuxsusnuxsusnux left review comments
@AndyScherzingerAndyScherzingerAndyScherzinger approved these changes
@st3inyst3inyst3iny approved these changes
@AltahrimAltahrimAltahrim approved these changes
@provokateurinprovokateurinprovokateurin approved these changes
@miaulalalamiaulalalaAwaiting requested review from miaulalala
@DorraJaouadDorraJaouadAwaiting requested review from DorraJaouad
@ArtificialOwlArtificialOwlAwaiting requested review from ArtificialOwlArtificialOwl is a code owner automatically assigned from nextcloud/server-backend
@icewind1991icewind1991Awaiting requested review from icewind1991icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@sorbaughsorbaughAwaiting requested review from sorbaughsorbaugh is a code owner automatically assigned from nextcloud/server-backend

Assignees

@nickvergessennickvergessen

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

7 participants

@nickvergessen@AndyScherzinger@st3iny@susnux@Altahrim@provokateurin@skjnldsv