Skip to content

fix(platform): redact spaced secret assignments - #83

Open
XnLemon wants to merge 105 commits into
mainfrom
codex/phase2-redactor-spaced-secrets
Open

fix(platform): redact spaced secret assignments#83
XnLemon wants to merge 105 commits into
mainfrom
codex/phase2-redactor-spaced-secrets

Conversation

@XnLemon

@XnLemon XnLemon commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Objective

Close another Phase2 sensitive-output gap by redacting common secret assignments that include spaces around = or : separators.

Changes

  • Extended default platform redaction patterns for api_key, token, secret, password, passwd, and cookie assignments with optional whitespace around separators.
  • Preserved compatibility with existing no-space key=value and key:value forms.
  • Added regression coverage for spaced api_key, password, token, secret, and cookie values.

Validation

  • go test ./platform
  • go vet ./platform
  • git diff --check
  • open-code-review subagent/manual review: 0 High / 0 Medium / 0 Low

Risks / Follow-up

  • This increment only broadens platform redaction for spaced secret assignments; remaining Phase2 closure items will continue in follow-up fork-only PRs.

XnLemon added 30 commits July 8, 2026 11:47
@XnLemon
XnLemon force-pushed the codex/phase2-approval-redaction-audit branch from 2078eb6 to a8819d3 Compare July 11, 2026 13:38
@XnLemon
XnLemon force-pushed the codex/phase2-redactor-spaced-secrets branch from 170433e to fb85850 Compare July 11, 2026 13:38
@XnLemon
XnLemon changed the base branch from codex/phase2-approval-redaction-audit to main July 13, 2026 02:23
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