Copied support text includes tool commands it says are excluded

Aperta Adatta ai principianti
#2,008 2 commenti 1 reazione 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp
Ambito
desktop, security

Direzione di ricerca

Leggi in GatewayService.cs intorno a OnActivityChanged e in CommandCenterTextHelper.cs intorno a BuildSupportContext per seguire come i titoli delle attività finiscono nel testo di supporto copiato. Aggiungi la regressione descritta nell’issue: un argomento del comando deve essere assente mentre la frase di esclusione rimane. Esegui il test pertinente; il lavoro è completato quando entrambe le asserzioni passano.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

clawsweeper:needs-security-review clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:security issue-rating: 🦞 diamond lobster P2

What happens

A tool event stores command or url in the activity label. GatewayService copies that label into the activity title. Command Center support text includes the title, then says command arguments are excluded.

The tray log line for the same event does go through TokenSanitizer.SanitizeLogMessage, which drops a URL query. The copied support text does not. A command, a path, or a ?token= in the title is therefore copied with the support bundle.

Where

Checked on 7967d509 by reading the code. Support text was not copied from a running app.

Suggested fix

Record only the tool name and phase in the activity title. Keep the "command arguments are excluded" line true.

Environment

Windows 11. OpenClaw Companion source on main at commit 7967d509886fc9efecbb19c8236350a82867f69c (2026-11-04). No packaged installer build was used.

Steps

  1. Receive a gateway tool event with a command or url argument.
  2. Open Command Center and copy the support text.
  3. Read the recent-activity line, then the line that says command arguments are excluded.

GatewayService.OnActivityChanged sets the activity title to sessionKey: activity.Label. CommandCenterTextHelper.BuildSupportContext copies item.Title. The tray log line is sanitized by TokenSanitizer.SanitizeLogMessage before it is written. The copied support text is not.

Repair

Put only the tool name and phase in the activity title used for support text. Add a regression that a command argument is absent from BuildSupportContext while the exclusion sentence remains. Stop there. The tray tooltip is #2007.

Lingua principale
C#
Stelle
2.2k
Fork
340
Merge medio
3g 3h
PR unite (30g)
168

Preparare l'ambiente

  • Nessun Dockerfile né file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di openclaw/openclaw-windows-node

Tutte le issue di openclaw/openclaw-windows-node

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.