Add Audit Event when the 2FA of a user is disabled
With #26149 (closed), we are sending out an email to the user when their 2FA is disabled.
Along similar lines, we also need to an add an Audit event when the 2FA of a user is disabled.
Designs
- Show closed items
Relates to
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Manoj M J added Category:Audit Events devopsmanage groupcompliance typefeature labels
added Category:Audit Events devopsmanage groupcompliance typefeature labels
- Manoj M J mentioned in merge request !39572 (merged)
mentioned in merge request !39572 (merged)
FYI @mattgonzales
Collapse replies - Contributor
Thanks so much for the tag @manojmj
- Maintainer
In !39572 (diffs, comment 398506380) was discussed the need of wrapping
disable_two_factor!
inside user's UpdateService, addition of audit_events was mentioned as the main reason, in case we evaluate that audit_events for 2fa change won't be called from UpdateService, we may consider removing this block wrapper. - 🤖 GitLab Bot 🤖 mentioned in issue #240838 (closed)
mentioned in issue #240838 (closed)
- 🤖 GitLab Bot 🤖 mentioned in issue #243451 (closed)
mentioned in issue #243451 (closed)
- 🤖 GitLab Bot 🤖 added sectiondev label
added sectiondev label
- 🤖 GitLab Bot 🤖 mentioned in issue #246442 (closed)
mentioned in issue #246442 (closed)
- 🤖 GitLab Bot 🤖 mentioned in issue #249044 (closed)
mentioned in issue #249044 (closed)
- Sam Kerr added audit_events_list label
added audit_events_list label
- Anton Smith marked #364148 (closed) as a duplicate of this issue
marked #364148 (closed) as a duplicate of this issue
- Anton Smith marked this issue as related to #364148 (closed)
marked this issue as related to #364148 (closed)
- Anton Smith added to epic &476
added to epic &476
- Anton Smith assigned to @anton
assigned to @anton
- Anton Smith mentioned in merge request !89598 (merged)
mentioned in merge request !89598 (merged)
- Sam Kerr mentioned in merge request gitlab-com/www-gitlab-com!105856 (merged)
mentioned in merge request gitlab-com/www-gitlab-com!105856 (merged)
- Sam Kerr mentioned in issue gitlab-org/manage/general-discussion#17496
mentioned in issue gitlab-org/manage/general-discussion#17496
- Anton Smith changed milestone to %15.1
changed milestone to %15.1
- Developer
The respective MR for this issue has been merged and is live in GitLab.com!
I've just finished my testing and it works as expected.
This will be in the
15.1
release. Closing! - Anton Smith closed
closed