Skip to content

feat(log): Add support for \SensitiveParameter attribute - #65412

Open
come-nc wants to merge 5 commits into
masterfrom
feat/add-support-for-sensitiveparameter
Open

come-nc wants to merge 5 commits into
masterfrom
feat/add-support-for-sensitiveparameter

Conversation

@come-nc

@come-nc come-nc commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor
  • Resolves: #

Summary

Parameters flagged with this attribute are now handled like methods flagged as sensitive in Nextcloud: Their value is stacktraces is replace with the same placeholder (instead of '[class => SensitiveParameterValue]'). And their value is tracked and removed from the rest of the stacktrace. Meaning if another function is called on the parameter it gets removed there as well.

TODO

  • Deprecate other means of adding sensitive parameters
  • Use the standard attribute in core

Checklist

AI (if applicable)

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

@come-nc come-nc added this to the Nextcloud 36 milestone Oct 12, 2026
@come-nc come-nc self-assigned this Oct 12, 2026
@come-nc
come-nc marked this pull request as ready for review October 12, 2026 15:27
@come-nc
come-nc requested a review from a team as a code owner October 12, 2026 15:27
@come-nc
come-nc requested review from icewind1991, leftybournes, salmart-dev and sorbaugh and removed request for a team October 12, 2026 15:27
@come-nc come-nc added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 12, 2026
Parameters flagged with this attribute are now handled like methods
 flagged as sensitive in Nextcloud: Their value is stacktraces is
 replace with the same placeholder (instead of '[class =>
 SensitiveParameterValue]'). And their value is tracked and removed from
 the rest of the stacktrace. Meaning if another function is called on
 the parameter it gets removed there as well.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
come-nc force-pushed the feat/add-support-for-sensitiveparameter branch from 03509fd to db486e4 Compare October 12, 2026 15:29

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants