Skip to content

Add log sanitization for user data - #92

Open
TensorDevLJ wants to merge 1 commit into
redhat-data-and-ai:mainfrom
TensorDevLJ:feat/log-sanitization-issue-31
Open

Add log sanitization for user data#92
TensorDevLJ wants to merge 1 commit into
redhat-data-and-ai:mainfrom
TensorDevLJ:feat/log-sanitization-issue-31

Conversation

@TensorDevLJ

Copy link
Copy Markdown

Summary

  • Adds log_sanitizer.py to redact PII, tokens, and sensitive headers before logging
  • Applies sanitization in api.py, stream.py, history.py
  • Adds LOG_SANITIZATION_* settings

Closes #31

Test plan

  • uv run pytest -q (88 tests pass)

Introduces log_sanitizer utility and applies redaction across request
logging, stream/history routes, and user metadata logs. Closes redhat-data-and-ai#31.
@codecov-commenter

Copy link
Copy Markdown

@NP-compete

Copy link
Copy Markdown
Member

The branch needs an update.

@NP-compete

Copy link
Copy Markdown
Member

This PR targets main, but main is pending a large merge from deep-agent (#104). Please rebase onto deep-agent instead, once that merge lands, this PR will likely conflict or need rework against the new codebase structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add log sanitization for user data

3 participants