Skip to content

Commit f048c8d

Browse files
committed
changed the instruction to comment on PR instead of raising a new issue
Signed-off-by: Jiaxiao Zhou <duibao55328@gmail.com>
1 parent d9a8f37 commit f048c8d

2 files changed

Lines changed: 15 additions & 114 deletions

File tree

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

Lines changed: 7 additions & 104 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: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ on:
44
branches: [ "main" ]
55
workflow_dispatch:
66

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

1010
tools:
1111
fetch:
@@ -45,14 +45,12 @@ Test the MCP network permissions feature to validate that domain restrictions ar
4545

4646
## Reporting Instructions
4747

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-
5148
If there are any failures, security issues, or unexpected behaviors:
49+
5250
- 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
51+
- Which domains were successfully accessed vs blocked
52+
- Error messages received for blocked domains
53+
- Any security observations or recommendations
54+
- Specific failure details that need attention
5755

58-
The test should demonstrate that MCP containers are properly isolated and can only access explicitly allowed domains through the network proxy.
56+
Create a GitHub Comment on the pull request with the test results.

0 commit comments

Comments
 (0)