Skip to content
This repository was archived by the owner on Jul 8, 2026. It is now read-only.

fix(security): remove [DISCOVER] log that leaked query + matched agents#4

Merged
amitpaz1 merged 1 commit into
mainfrom
fix/backlog
Jun 20, 2026
Merged

fix(security): remove [DISCOVER] log that leaked query + matched agents#4
amitpaz1 merged 1 commit into
mainfrom
fix/backlog

Conversation

@amitpaz1

Copy link
Copy Markdown
Member

The GET /v1/discover handler logged the raw client query and every matched agent name/score to stdout on each request — noise and an information leak. Removed the single console.log line (no logger util exists in the repo, so removal is the lazy-correct fix). Build + 49/49 tests pass.

🤖 Generated with Claude Code

…matches

src/http-server.ts logged the raw client query string plus every matched agent name
and relevance score to stdout on each GET /v1/discover request — noise + an info leak.
No logger utility exists in the repo, so the line is removed outright.

Verified: pnpm build + pnpm test (49/49) pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@amitpaz1
amitpaz1 merged commit 228fadc into main Jun 20, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant