-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: status --all can print credentials hidden in channel base URLs #98633
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:needs-security-reviewClawSweeper marked this issue as needing security-sensitive review.ClawSweeper marked this issue as needing security-sensitive review.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:securitySecurity boundary, credential, authz, sandbox, or sensitive-data risk.Security boundary, credential, authz, sandbox, or sensitive-data risk.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maturity:stableIssue affects a taxonomy feature currently scored M4/M5.Issue affects a taxonomy feature currently scored M4/M5.
Type
Fields
Priority
None yet
Bug type
Behavior bug (incorrect output/state without crash)
Beta release blocker
No
Summary
openclaw status --allcan print channel accountbaseUrlvalues with embedded URL credentials or auth-like query parameters in the pasteable status report.Steps to reproduce
baseUrlthat contains URL userinfo or sensitive query parameters, for example a Mattermost base URL shaped likehttps://user:[email protected]/hooks?token=value&api_key=value.openclaw status --all.Expected behavior
The status report should redact URL userinfo and sensitive query parameter values before printing channel account base URLs.
Actual behavior
Before PR #98599, the status-all channel account notes appended
snapshot.baseUrldirectly. A pasteable support report could therefore include credentials hidden in URL userinfo or auth-like query params.OpenClaw version
Current
mainbefore PR #98599.Operating system
Windows 11 for the local proof. The affected path is platform-independent status report rendering.
Install method
Source checkout / built local CLI.
Model
N/A
Provider / routing chain
N/A
Logs
After PR #98599, a local built CLI proof with a synthetic Mattermost
baseUrlprints:The raw synthetic values
oc-user,oc-pass,oc-token-raw, andoc-api-key-rawwere not present in the output.Screenshots, recordings, and evidence
PR #98599 includes focused regression coverage and a local built CLI proof using:
Impact and severity
Affected: operators who paste
openclaw status --alloutput for debugging while a channel accountbaseUrlcontains embedded credentials or auth-like query params.Severity: Medium. This is a diagnostic data exposure risk in support reports.
Frequency: Reproducible when affected URL-shaped config values are present.
Consequence: Credentials hidden in URLs can be copied into issue reports, chats, or logs.
Additional information
Fix PR: #98599