Skip to content

Add --redaction option to log forwarding integrations#117

Open
mharacewiat wants to merge 1 commit into
upsun:mainfrom
mharacewiat:feat/sre-1663-log-redaction
Open

Add --redaction option to log forwarding integrations#117
mharacewiat wants to merge 1 commit into
upsun:mainfrom
mharacewiat:feat/sre-1663-log-redaction

Conversation

@mharacewiat

Copy link
Copy Markdown
Contributor

Enables opt-in PII redaction for log lines forwarded via log forwarding integrations. When enabled, agilis strips sensitive data (e.g. email addresses) from log lines before they leave the host — satisfying the PCI-DSS requirement to never transmit PII off-machine unredacted.

Defaults to false (opt-in).

Ref SRE-1663.

@mharacewiat mharacewiat force-pushed the feat/sre-1663-log-redaction branch from 35e88d8 to 4319d05 Compare June 26, 2026 08:49
@mharacewiat mharacewiat marked this pull request as ready for review June 26, 2026 08:52
Copilot AI review requested due to automatic review settings June 26, 2026 08:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an opt-in boolean configuration field for log-forwarding integrations so users can enable PII redaction via the CLI (intended to prevent sensitive data from being forwarded off-host unredacted).

Changes:

  • Introduces a redaction BooleanField on integrations, gated to log-forwarding integration types.
  • Defaults the new setting to false (opt-in) and exposes it as a CLI option through the existing ConsoleForm wiring.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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