Skip to content

[SECURITY] Scope OAuthConfigAnalyzer away from blind home-dir discovery #290

@hello-args

Description

@hello-args

Summary

OAuthConfigAnalyzer auto-discovers all local MCP configs during mcts scan, independent of inventory privacy flags.

Problem

src/mcts/analyzers/oauth_config.py:149 calls discover_config_paths() without user scoping. Inventory privacy controls (#87 / PR #279) apply only to mcts inventory, not scan-time analyzer discovery.

A user running mcts scan ./repo may still trigger reads of all well-known home-directory MCP configs via this analyzer.

Expected Behavior

Either:

  • Scope discovery to scan target / explicit --config when provided, or
  • Document that OAuth config analysis always performs local config discovery and treat as accepted scan behavior.

Evidence

Impact

Privacy-conscious users may not expect scan of a repo path to read unrelated home MCP configs.

References

Acceptance Criteria

  • Design decision documented or discovery scoped
  • Tests for scoped vs full discovery
  • Cross-linked from docs/scanning/inventory.md or security-checks if documented-only

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions