department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
284 stars 206 forks source link

Implement solution to prevent PII in logs #97811

Open rmtolmach opened 3 days ago

rmtolmach commented 3 days ago

written by AI

User Story

As a developer, I want to ensure that no PII is logged to Datadog, Sentry, or AWS logs, so that sensitive user information is protected and we maintain compliance with privacy policies.

Issue Description

Currently, we rely on manual reviews to catch instances of potential PII being logged, but this approach is error-prone and insufficient. There are tools like Brakeman that might be able to flag PII-related logging issues, or libraries like Logstop that can sanitize logs automatically.

This issue is to explore and implement a solution that either flags potential PII logging issues during PR reviews or sanitizes logs at runtime to ensure no PII makes it to external logging systems like Datadog, Sentry, or AWS.

Filtering we already have:

Tasks

Acceptance Criteria


Validation

Assignee to add steps to this section. List the actions that need to be taken to confirm this issue is complete. Include any necessary links or context. State the expected outcome.