Skip to content

feat(audit_log): expand audit logging - #1271

Open
CarlSchwan wants to merge 2 commits into
masterfrom
feat/admin-audit-logging
Open

CarlSchwan wants to merge 2 commits into
masterfrom
feat/admin-audit-logging

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

On Nextcloud 36 and later, write the following operations to the admin audit log:

  • user_saml.provider.created/updated/deleted, from the admin settings and the saml:config:* commands. Updates list the changed settings, never their values.
  • user_saml.user.provisioned, at login, by a Global Scale master or with saml:user:add.
  • user_saml.login.failed, when the assertion consumer service rejects a login.
  • user_saml.session.revoked, when an identity provider or a Global Scale master ends a session through a single logout.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

On Nextcloud 36 and later, write the following operations to the admin
audit log:

- user_saml.provider.created/updated/deleted, from the admin settings
  and the saml:config:* commands. Updates list the changed settings,
  never their values.
- user_saml.user.provisioned, at login, by a Global Scale master or with
  saml:user:add.
- user_saml.login.failed, when the assertion consumer service rejects
  a login.
- user_saml.session.revoked, when an identity provider or a Global
  Scale master ends a session through a single logout.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Carl Schwan <[email protected]>
@CarlSchwan
CarlSchwan requested a review from blizzz as a code owner October 30, 2026 13:11
Psalm 6.20.4, which CI installs, no longer treats the setUp() method of
PHPUnit tests as initializing their properties, and reports every
property of a test as PropertyNotSetInConstructor. Downgrade the issue
to info for the unit tests, like MissingConstructor.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Carl Schwan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant