Skip to content
Snippets Groups Projects

Audit delivery of confirmation emails

Merged SAM FIGUEROA requested to merge sf/feature/GL377625-audit-user-email-changes into master
All threads resolved!

What does this MR do and why?

This MR adds audit events when an email confirmation is requested by the Emails::Confirm service or an email was changed on the User model.

This will allow us to track when users request email confirmations and when they change their email addresses.

How to set up and validate locally

  1. Setup a streaming destination for audit events (Admin area -> Monitoring -> Audit Events -> Streams)
  2. Change the signed in users's email address.
  3. Add an additional email address for that user
  4. Verify that an audit event is streamed when adding an email address
  5. Verify that an audit event is streamed when changing your email address on your profile

MR acceptance checklist


This description was generated for revision 4aa7ebbf using AI ...and then manually revised.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • requested review from @huzaifaiftikhar1

  • Huzaifa Iftikhar
  • 🤖 GitLab Bot 🤖 changed milestone to %16.4

    changed milestone to %16.4

  • SAM FIGUEROA added 1224 commits

    added 1224 commits

    • 881181ea...7a5e187a - 1222 commits from branch master
    • 0f00940a - Audit delivery of confirmation emails
    • f512a168 - Fix failing specs

    Compare with previous version

  • Huzaifa Iftikhar approved this merge request

    approved this merge request

  • Huzaifa Iftikhar requested review from @rkadam3 and removed review request for @huzaifaiftikhar1

    requested review from @rkadam3 and removed review request for @huzaifaiftikhar1

  • :wave: @huzaifaiftikhar1, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

    For more info, please refer to the following links:

  • Huzaifa Iftikhar
  • SAM FIGUEROA added 1 commit

    added 1 commit

    Compare with previous version

  • SAM FIGUEROA added 78 commits

    added 78 commits

    Compare with previous version

  • Rajendra Kadam approved this merge request

    approved this merge request

  • Rajendra Kadam resolved all threads

    resolved all threads

  • Rajendra Kadam enabled an automatic merge when the pipeline for f089bca5 succeeds

    enabled an automatic merge when the pipeline for f089bca5 succeeds

  • merged

  • Rajendra Kadam mentioned in commit 0dc676cf

    mentioned in commit 0dc676cf

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading