Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Removing featureflag for expanded audit logs #64415

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

mohammadualam
Copy link
Contributor

This PR removes featureflag which allows for sensitive actions to be logged without needing to enable the feature using the featureflag that was in place.

Test plan

CI checks and new tests

Changelog

  • More auditlogs for sensitive admin actions will be automatically logged.

@mohammadualam mohammadualam self-assigned this Aug 12, 2024
@cla-bot cla-bot bot added the cla-signed label Aug 12, 2024
Copy link
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

Copy link
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

Copy link
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

Copy link
Contributor

Caution

License checking failed, please read: how to deal with third parties licensing.

@mohammadualam mohammadualam requested a review from a team August 12, 2024 19:59
Copy link
Contributor

@shivasurya shivasurya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@evict evict requested a review from a team August 13, 2024 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants