Skip to content

Update filesystem ALLOWED_PATHS in nightly stress test workflow#690

Merged
lpcox merged 5 commits intomainfrom
copilot/add-allowed-paths-configuration
Feb 5, 2026
Merged

Update filesystem ALLOWED_PATHS in nightly stress test workflow#690
lpcox merged 5 commits intomainfrom
copilot/add-allowed-paths-configuration

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

The filesystem MCP server requires ALLOWED_PATHS environment variable to whitelist accessible paths. This PR updates the nightly stress test workflow configuration to expand the allowed paths.

Changes

Updated .github/workflows/nightly-mcp-stress-test.md:

  • Changed ALLOWED_PATHS: "/workspace" to ALLOWED_PATHS: "/tmp,/workspace"
  • Allows the filesystem MCP server to access both /tmp and the mounted /workspace directory during stress testing

Configuration

filesystem:
  type: stdio
  container: "mcp/filesystem"
  env:
    ALLOWED_PATHS: "/tmp,/workspace"
  mounts:
    - "/tmp/mcp-test-fs:/workspace:rw"

The configuration provides access to:

  • /tmp - Container's temporary directory for filesystem operations
  • /workspace - Mounted directory (mapped from host /tmp/mcp-test-fs)

Security

  • Uses restrictive path whitelist (only /tmp and /workspace)
  • Does not expose sensitive directories
  • Follows principle of least privilege for testing purposes

Testing

  • ✅ Build successful
  • ✅ Unit tests passed
  • ✅ Configuration syntax validated
Original prompt

This section details on the original issue you should resolve

<issue_title>[mcp-stress-test] Filesystem MCP server requires ALLOWED_PATHS configuration</issue_title>
<issue_description>## MCP Server Configuration Required: filesystem

The nightly stress test detected that the Filesystem MCP server requires configuration to function properly.

Test Details

  • Test Session: stress-test-20260205-030515
  • Test Date: 2026-02-05T03:05:15Z
  • Server Container: mcp/filesystem

Required Configuration

Environment Variable: ALLOWED_PATHS

Configuration Type: Path whitelist (security requirement)

Issue:
The filesystem server requires explicit allowed paths configuration for security reasons. Without this, the server cannot operate to prevent unauthorized file access.

Suggested Configuration

Add the following environment variable to the MCP Gateway configuration for the filesystem server:

{
  "filesystem": {
    "type": "stdio",
    "container": "mcp/filesystem",
    "env": {
      "ALLOWED_PATHS": "/tmp,/home/runner/work"
    }
  }
}

Security Considerations

When configuring ALLOWED_PATHS:

  • Only include directories that are safe for the MCP server to access
  • Use the most restrictive paths possible
  • Avoid including sensitive directories (e.g., /etc, /root, home directories with secrets)
  • For testing, consider using /tmp or dedicated test directories

Next Steps

  • Determine safe allowed paths for testing
  • Update MCP Gateway configuration with ALLOWED_PATHS
  • Update stress test configuration
  • Verify server works in next nightly test

Generated by Nightly MCP Stress Test
Test Session: stress-test-20260205-030515

AI generated by Nightly MCP Server Stress Test

<agent_instructions>Only update the .md file, not the .lock.yml file</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Custom agent used: agentic-workflows
GitHub Agentic Workflows (gh-aw) - Create, debug, and upgrade AI-powered workflows with intelligent prompt routing


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 5, 2026 03:18
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] Add ALLOWED_PATHS configuration for filesystem MCP server Add filesystem MCP server configuration with ALLOWED_PATHS Feb 5, 2026
Copilot AI requested a review from lpcox February 5, 2026 03:21
@lpcox lpcox marked this pull request as ready for review February 5, 2026 03:21
Copilot AI review requested due to automatic review settings February 5, 2026 03:21
@lpcox
Copy link
Collaborator

lpcox commented Feb 5, 2026

@copilot you don't need to change the example files. instead update the mcp/filesystem configuration in the mcp server stress test .md (don't change the .lock.yml file)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds filesystem MCP server configuration with the required ALLOWED_PATHS environment variable to enable local development and testing. The configuration was identified as missing during a nightly stress test.

Changes:

  • Added filesystem server configuration to three config files (TOML and JSON formats)
  • Configured Docker volume mount (/tmp:/workspace:rw) with corresponding ALLOWED_PATHS=/workspace whitelist
  • Added documented example with inline comments for developer reference

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
config.toml Adds filesystem server with Docker args, volume mount, and ALLOWED_PATHS environment variable
config.json Adds filesystem server in JSON format with mounts array and env object including ALLOWED_PATHS
config.example.toml Adds documented Example 3 for filesystem server with inline comments explaining configuration requirements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 5, 2026

@copilot you don't need to change the example files. instead update the mcp/filesystem configuration in the mcp server stress test .md (don't change the .lock.yml file)

Updated! I've reverted the changes to the example config files and instead updated the ALLOWED_PATHS in nightly-mcp-stress-test.md to include both /tmp and /workspace for the filesystem MCP server configuration.

Changes in commit f76c057.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • example.com
    • Triggering command: /tmp/go-build212444450/b270/launcher.test /tmp/go-build212444450/b270/launcher.test -test.testlogfile=/tmp/go-build212444450/b270/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true unset --global cal/bin/bash go (dns block)
  • invalid-host-that-does-not-exist-12345.com
    • Triggering command: /tmp/go-build212444450/b258/config.test /tmp/go-build212444450/b258/config.test -test.testlogfile=/tmp/go-build212444450/b258/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true go --global cal/bin/bash user.name (dns block)
  • nonexistent.local
    • Triggering command: /tmp/go-build212444450/b270/launcher.test /tmp/go-build212444450/b270/launcher.test -test.testlogfile=/tmp/go-build212444450/b270/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true unset --global cal/bin/bash go (dns block)
  • slow.example.com
    • Triggering command: /tmp/go-build212444450/b270/launcher.test /tmp/go-build212444450/b270/launcher.test -test.testlogfile=/tmp/go-build212444450/b270/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true unset --global cal/bin/bash go (dns block)
  • this-host-does-not-exist-12345.com
    • Triggering command: /tmp/go-build212444450/b279/mcp.test /tmp/go-build212444450/b279/mcp.test -test.testlogfile=/tmp/go-build212444450/b279/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true /x64=/_/GOROOT --local u/13/cc1 pull.rebase (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title Add filesystem MCP server configuration with ALLOWED_PATHS Update filesystem ALLOWED_PATHS in nightly stress test workflow Feb 5, 2026
@lpcox lpcox merged commit 86b664b into main Feb 5, 2026
@lpcox lpcox deleted the copilot/add-allowed-paths-configuration branch February 5, 2026 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[mcp-stress-test] Filesystem MCP server requires ALLOWED_PATHS configuration

2 participants