department-of-veterans-affairs / notification-api

Notification API
MIT License
16 stars 9 forks source link

Personalisation Redaction: Sanitize notifications after final delivery #1358

Open kalbfled opened 1 year ago

kalbfled commented 1 year ago

User Story - Business Need

User Story(ies)

As a Notify developer, I want to remove encrypted PII from the database when a redacted message reaches its final status So that everybody remains in compliance with all applicable laws and VA policies.

Additional Info and Resources

See #1357. Those changes should ensure that PII is not visible in any http response, but PII can still live in the database in encrypted form. Once a notification generated using a redacted template reaches a final status, the personalization data should be replaced in the database with "\<redacted>".

https://github.com/department-of-veterans-affairs/notification-api/blob/master/app/models.py#L1406

Engineering Checklist

Acceptance Criteria

QA Considerations

mjones-oddball commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @babyjunior56 @EvanParish @k-macmillan @kalbfled @ldraney @nikolai-efimov

k-macmillan commented 1 month ago

okay to keep