Skip to content

Resolve "notifications broken during email address change before email confirmed"

What does this MR do?

This fixes an issue where the notification email address would be set to an unconfirmed email address.

Are there points in the code the reviewer needs to double check?

This logic were not tested at all before this MR, so there are chances that these changes break something...

Why was this MR needed?

Because the notification email address would be set to a unconfirmed email address, which is a potential security (spam) issue.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #22846 (closed)

Edited by Rémy Coutable

Merge request reports