Skip to content

fix(adminaudit): Allow named placeholders via CriticalActionPerformedEvent - #64862

Open
nickvergessen wants to merge 1 commit into
masterfrom
bugfix/noid/allow-named-placeholders
Open

nickvergessen wants to merge 1 commit into
masterfrom
bugfix/noid/allow-named-placeholders

Conversation

@nickvergessen

@nickvergessen nickvergessen commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Assisted-by: ClaudeCode:claude-opus-5.5

Steps

  • Before only %s parameters were working, but we switched to named placeholders for majority of the logs by now
  • Now both %s and {action} placeholders are supported
  • Fixes "useless" logs fix(admin-audit): Log when BFP was bypassed #64557 started to generate

Checklist

AI (if applicable)

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

…Event

Assisted-by: ClaudeCode:claude-opus-5.5
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 36 milestone Sep 28, 2026
@nickvergessen nickvergessen self-assigned this Sep 28, 2026
@nickvergessen
nickvergessen requested a review from a team as a code owner September 28, 2026 07:52
@nickvergessen
nickvergessen requested review from CarlSchwan, icewind1991, leftybournes and salmart-dev and removed request for a team September 28, 2026 07:52
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable35

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable34

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable33

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants