Skip to content
Snippets Groups Projects

Failsafe access to current user's email

Merged Rajat Jain requested to merge fix/issue-299207 into master
All threads resolved!

What does this MR do?

Fixes #299207 (closed)

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Jan Provaznik

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
  • Rajat Jain resolved all threads

    resolved all threads

  • Rajat Jain added 1 commit

    added 1 commit

    • 7926876d - Failsafe access to users email

    Compare with previous version

  • Rajat Jain requested review from @dchevalier2

    requested review from @dchevalier2

  • Jan Provaznik approved this merge request

    approved this merge request

  • Jan Provaznik removed review request for @jprovaznik

    removed review request for @jprovaznik

  • Désirée Chevalier approved this merge request

    approved this merge request

  • assigned to @jprovaznik and unassigned @dchevalier2

  • Désirée Chevalier removed review request for @dchevalier2

    removed review request for @dchevalier2

  • Rajat Jain resolved all threads

    resolved all threads

  • Rajat Jain added 1 deleted label

    added 1 deleted label

  • Rajat Jain resolved all threads

    resolved all threads

  • Jan Provaznik marked the checklist item Changelog entry as completed

    marked the checklist item Changelog entry as completed

  • Thanks @rajatgitlab, MWPS set 🚀

  • Jan Provaznik enabled an automatic merge when the pipeline for 432a10d9 succeeds

    enabled an automatic merge when the pipeline for 432a10d9 succeeds

  • merged

  • Jan Provaznik mentioned in commit 4476d625

    mentioned in commit 4476d625

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • GitLab Release Tools Bot picked the changes into the branch 13-8-stable-ee-patch-1 with commit ae7505fd

    picked the changes into the branch 13-8-stable-ee-patch-1 with commit ae7505fd

  • Jan Provaznik mentioned in commit ae7505fd

    mentioned in commit ae7505fd

  • Automatically picked into !52527 (merged), will merge into 13-8-stable-ee ready for 13.8.1-ee.

  • GitLab Release Tools Bot removed 1 deleted label

    removed 1 deleted label

  • mentioned in merge request !52527 (merged)

  • Please register or sign in to reply
    Loading