Skip to content

Commit fa6aa32

Browse files
fix: redacting user retirement data in lms
1 parent ed4c3c0 commit fa6aa32

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • openedx/core/djangoapps/user_api/accounts

openedx/core/djangoapps/user_api/accounts/views.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1046,8 +1046,8 @@ def cleanup(self, request):
10461046
if len(usernames) != len(retirements):
10471047
raise UserRetirementStatus.DoesNotExist("Not all usernames exist in the COMPLETE state.")
10481048

1049-
# Redact PII fields instead of deleting records
1050-
# This ensures Fivetran syncs redacted data to Snowflake instead of creating soft deletes with PII
1049+
# Redact PII fields instead of deleting records to prevent ETL tools
1050+
# from creating soft deletes with visible PII in downstream data warehouses
10511051
for retirement in retirements:
10521052
redacted_value = f"jenkins-{jenkins_run_id}-{retirement.id}"
10531053
retirement.original_username = redacted_value

0 commit comments

Comments
 (0)