mcp-memory-service Vulnerable to System Information Disclosure via Health Endpoint

Attack Scenario

1. Attacker scans the local network for services on port 8000
2. Finds mcp-memory-service with HTTP enabled and anonymous access
3. Calls GET /api/health/detailed (no credentials needed)
4. Receives OS version, Python version, full database path (revealing username), system resources
5. Uses this information to:
   • Target known CVEs for the specific OS/Python version
   • Identify the database file location for potential direct access
   • Profile the system for further attacks

PoC

# Show the system info that would be exposed
import platform, psutil

system_info = {
    "platform": platform.system(),
    "platform_version": platform.version(),
    "python_version": platform.python_version(),
    "cpu_count": psutil.cpu_count(),
    "memory_total_gb": round(psutil.virtual_memory().total / (1024**3), 2),
}
print(system_info)  # All of this is returned to unauthenticated users

Impact

• OS fingerprinting: Exact OS and kernel version enables targeted exploit selection
• Path disclosure: Database path reveals username, home directory structure, and file locations
• Resource enumeration: CPU, memory, and disk info reveal infrastructure scale
• Reconnaissance enablement: Combined information significantly reduces attacker effort for follow-up attacks

Remediation

1. Remove system details from default health endpoint - return only status, version, uptime:

@router.get("/health/detailed")
async def detailed_health_check(
    storage: MemoryStorage = Depends(get_storage),
    user: AuthenticationResult = Depends(require_write_access)  # Require admin/write access
):
    # Only return storage stats, not system info
    ...

2. Do not expose database_path - this leaks the filesystem structure:

# Remove or redact
# storage_info["database_path"] = storage.db_path  # REMOVE THIS

3. Add auth to basic /health or limit it to status-only (no version):

@router.get("/health")
async def health_check():
    return {"status": "healthy"}  # No version, no uptime

Alternatively, Bind to 127.0.0.1 by default instead of 0.0.0.0, preventing network-based reconnaissance entirely:

# In config.py — change default from '0.0.0.0' to '127.0.0.1'
HTTP_HOST = os.getenv('MCP_HTTP_HOST', '127.0.0.1')

Users who need network access can explicitly set MCP_HTTP_HOST=0.0.0.0, making the exposure a conscious opt-in rather than a default.