Skip to content

#165: Redact credentials in passive discovery URLs - #178

Merged
justsml merged 4 commits into
mainfrom
dan/issue-165-url-credential-redaction
Aug 28, 2026
Merged

#165: Redact credentials in passive discovery URLs#178
justsml merged 4 commits into
mainfrom
dan/issue-165-url-credential-redaction

Conversation

@justsml

@justsml justsml commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Stacked on #164 / PR #150.

Closes #165

Scope

  • Unconditionally remove HTTP(S) userinfo and redact sensitive query and fragment values before passive evidence is projected.
  • Apply the invariant through shared evidence redaction with passive normalizer defense in depth.
  • Preserve non-secret host, path, auth categorization, and evidence attribution.
  • Cover response projection, durable report content, and RAG index input, including when general evidence redaction is disabled.

Validation

  • 3 focused integration files / 21 tests pass.
  • Typecheck and targeted lint pass.
  • Diff checks pass.

Merge order

Merge PR #150 first, then retarget this PR to PR #150's base or main.

@justsml

justsml commented Aug 27, 2026

Copy link
Copy Markdown
Owner Author

Council of Dans review: repair/defer. Highest-priority gate: Restack early in the passive slice and prove credentials are absent from response, stored report, Artifact payload, and indexed RAG text. Feature flags are not a substitute for authority, evidence-integrity, or durability fixes; use typed modes only where they provide a real rollout boundary.

…dential-redaction

# Conflicts:
#	tests/integration/stored-passive-auth-surface-api.test.ts
@justsml
justsml changed the base branch from dan/issue-21-artifact-browser to main August 28, 2026 15:02
@justsml
justsml merged commit 58b7d41 into main Aug 28, 2026
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.

Redact credentials embedded in passive-discovery URLs

1 participant