Skip to content

fix: [PM-19904] Audit Fix#1067

Draft
m2ux wants to merge 5 commits intomainfrom
fix/PM-19904-db-connection-details-leaked-in-logs
Draft

fix: [PM-19904] Audit Fix#1067
m2ux wants to merge 5 commits intomainfrom
fix/PM-19904-db-connection-details-leaked-in-logs

Conversation

@m2ux
Copy link
Copy Markdown
Contributor

@m2ux m2ux commented Mar 24, 2026

Summary

Audit remediation for PM-19904.

🎫 Ticket 📐 Engineering


Motivation

See linked ticket for details.


Changes

Implementation (coming next):

  • Redact infrastructure details from error-level log output
  • Add debug-level logging for authorized troubleshooting
  • Update existing test, add new redaction verification test
  • Change file

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason: [reason]
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • No new todos introduced

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other
  • N/A

🗹 TODO before merging

  • Ready for review

@m2ux m2ux self-assigned this Mar 24, 2026
@m2ux m2ux changed the title fix: [PM-19904] Redact database connection details from error logs fix: [PM-19904] Audit Fix Mar 24, 2026
@m2ux m2ux force-pushed the fix/PM-19904-db-connection-details-leaked-in-logs branch from 400cb68 to 23f290c Compare March 24, 2026 17:42
@m2ux m2ux force-pushed the fix/PM-19904-db-connection-details-leaked-in-logs branch from 8681d9e to ea3723e Compare March 30, 2026 10:52
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.

2 participants