search

GSA / notifications-admin

The UI of Notify.gov
https://notify.gov
Other
11 stars 2 forks source link

API-1328 - Using a custom formatter to scrub PII from all log records. #2063

Closed xlorepdarkhelm closed 3 weeks ago

xlorepdarkhelm commented 1 month ago

A note to PR reviewers: it may be helpful to review our code review documentation to know what to keep in mind while reviewing pull requests.

Description

The scrub() method was restructured to be a function, and then applied to a new custom log formatter.

See: https://github.com/GSA/notifications-api/issues/1328

Security Considerations

Scrubbing PII from log entries is very good for security purposes.