# NousResearch/hermes-agent · Strict URL userinfo credential redaction fix

## Open now

- Nothing open on GitHub.

These items address a bug where strict URL-credential redaction fails to mask credentials placed in the userinfo username position. The group covers the problem report and associated fixes.

## TL;DR

- 5 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 3 PR(s) closed without merging.
- A fix was merged to main in PR #130548. Verify it addresses your case.
- **Recommended:** read PR #130548 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `resolved` · deciding PR(s): `pr-130548`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.40`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Closed / duplicate issues

- #129983: Strict URL-credential redaction keeps a credential that sits in the userinfo username (dump / debug share)

## Prior & competing PRs

- **#130548: Dump, debug share and other strict-redaction egress no longer leak credentials kept in the URL username (#129983, salvage #129985)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `agent/redact.py`, `tests/agent/test_pre_compress_memory_context.py`, `tests/agent/test_pre_compress_memory_context_handoff.py`, `tests/agent/test_redact.py`, `tests/hermes_cli/test_debug.py`
- **#129985: fix(agent): redact strict URL userinfo credentials**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `c900b1df58`, `#130548`
- **#130027: fix(redact): fully mask strict URL userinfo**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `c900b1df58`, `#129985`, `#130548`
- **#130529: agent: strict URL redaction now masks full userinfo to catch tokens in username**. closed without merge
  - maintainer direction: category `uncategorized`; implication `needs_llm`

## Fix status

- Merged to main in PR #130548 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-10-01`.