Skip to content

fix: redact reflector stderr on failure#1

Merged
brannn merged 1 commit into
mainfrom
fix/redact-reflector-stderr
May 21, 2026
Merged

fix: redact reflector stderr on failure#1
brannn merged 1 commit into
mainfrom
fix/redact-reflector-stderr

Conversation

@brannn
Copy link
Copy Markdown
Contributor

@brannn brannn commented May 21, 2026

Summary

  • redact reflector subprocess stderr before including it in returned errors
  • keep redacted stderr context available for debugging
  • add a regression test for reflector failure stderr redaction

Test Plan

  • go test ./internal/reflector -count=1
  • go test ./...
  • git diff --check

@brannn brannn merged commit fcd6f57 into main May 21, 2026
1 check passed
@brannn brannn deleted the fix/redact-reflector-stderr branch May 21, 2026 06:49
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