owasp-mcp-governance-and-risk-project

Security Findings (13)

• HighAS-012

  🔄Tool Drift

  Tool set changed silently at v1.0: 3 tool(s) added, 29 tool(s) removed without a version bump.

  + 3 added

  get_decision_detailspreflight_change_setverify_receipt

  − 29 removed

  approve_gateassess_risk_tieraudit_pipelineclassify_decisioncompose_memory_packsdistill_memory_packevaluate_thresholdgenerate_impact_reportgenerate_reportgia_apply_packgia_dry_run_packgia_list_packsgia_run_patrolgia_scan_environmentload_memory_packmap_compliancemonitor_agentspromote_memory_packrecord_governance_eventrecord_value_metricscore_governanceseal_memory_packsrt_approve_repairsrt_diagnosesrt_generate_postmortemsrt_run_watchdogsystem_statustransfer_memory_packverify_ledger

  Fix: The set of tools exposed by this server changed between scans of the same version — a sign the package was silently updated without a version bump. Audit the changelog and all tool definitions before trusting this server. Pin to a specific commit hash rather than a floating version tag.

• HighAS-003

  🔀Scope Mismatch

  get_decision_details:tool name "get_decision_details" implies read-only operation but declares exec permission

  Fix: Ensure tool names, descriptions, and permission declarations are internally consistent. Use explicit naming conventions that fully reflect actual capabilities.

• LowAS-011

  ℹ️Missing Rate-Limit / Timeout ×3

  tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration

  preflight_change_setverify_receiptget_decision_details

  Fix: Declare explicit rate-limit, timeout, and retry configuration for all network and execution tools. Implement exponential back-off and surface resource state to the calling agent.

• LowAS-002

  ⚠️Excessive Permissions

  verify_receipt:input schema exposes 12 properties (threshold: 10)

  Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.

• InfoAS-002

  ⚠️Excessive Permissions ×3

  declared capabilities: code/command execution, network access

  preflight_change_set

  declared capabilities: code/command execution, environment variables

  verify_receipt

  declared capabilities: code/command execution

  get_decision_details

  Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.

• InfoAS-014

  ℹ️Dependency Inventory Unavailable ×3

  Tool did not expose metadata.dependencies or repo_url, so supply-chain coverage is limited.

  preflight_change_setverify_receiptget_decision_details

  Fix: Review and remediate the identified issue.

• InfoAS-010

  🔑Insecure Secret Handling

  verify_receipt:input parameter "token" accepts a credential (informational; not evidence of insecure handling)

  Fix: Avoid accepting raw credentials as input parameters. Use secret managers (e.g. 1Password CLI, AWS Secrets Manager) and ensure credentials are never logged or stored in agent traces.