Skip to content

Commit d929ff9

Browse files
fix: retirement PII leaks by redacting pending secondary email/name data
1 parent 48e5e17 commit d929ff9

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

common/djangoapps/student/views/management.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -893,9 +893,8 @@ def activate_secondary_email(request, key):
893893
# Overwrite the pending email with a neutral placeholder before deletion so
894894
# that any downstream soft-delete mirror does not retain the real address.
895895
# We do NOT use get_retired_email_by_email() here because this is a normal
896-
# user action, not a retirement flow; a hashed retirement-style value would
897-
# produce false-positive retirement signals in downstream systems.
898-
pending_secondary_email_change.new_secondary_email = "completed@delete-pending.com"
896+
# user action, not a retirement flow.
897+
pending_secondary_email_change.new_secondary_email = "redacted@retired.invalid"
899898
pending_secondary_email_change.save(update_fields=['new_secondary_email'])
900899
pending_secondary_email_change.delete()
901900

0 commit comments

Comments
 (0)