Skip to content

Commit 4b84184

Browse files
CopilotMossaka
andcommitted
Update test-proxy.md to use comments instead of creating issues
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
1 parent acc8707 commit 4b84184

2 files changed

Lines changed: 121 additions & 16 deletions

File tree

.github/workflows/test-proxy.lock.yml

Lines changed: 108 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/test-proxy.md

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ on:
44
branches: [ "main" ]
55
workflow_dispatch:
66

7-
permissions:
8-
issues: write # needed to write the output report to an issue
7+
output:
8+
issue_comment: {}
99

1010
tools:
1111
fetch:
@@ -21,7 +21,6 @@ tools:
2121

2222
github:
2323
allowed:
24-
- "create_issue"
2524
- "create_comment"
2625
- "get_issue"
2726

@@ -44,10 +43,16 @@ Test the MCP network permissions feature to validate that domain restrictions ar
4443
- Verify that all blocked requests fail at the network level (proxy enforcement)
4544
- Confirm that only example.com is accessible through the Squid proxy
4645

47-
Create a GitHub issue with the test results, documenting:
48-
- Which domains were successfully accessed vs blocked
49-
- Error messages received for blocked domains
50-
- Confirmation that network isolation is working correctly
51-
- Any security observations or recommendations
46+
## Reporting Instructions
47+
48+
If the test runs successfully and network isolation is working correctly:
49+
- Write a brief summary to ${{ env.GITHUB_AW_OUTPUT }} confirming the test passed
50+
51+
If there are any failures, security issues, or unexpected behaviors:
52+
- Write a detailed report to ${{ env.GITHUB_AW_OUTPUT }} documenting:
53+
- Which domains were successfully accessed vs blocked
54+
- Error messages received for blocked domains
55+
- Any security observations or recommendations
56+
- Specific failure details that need attention
5257

5358
The test should demonstrate that MCP containers are properly isolated and can only access explicitly allowed domains through the network proxy.

0 commit comments

Comments
 (0)