Allow Go module downloads in MCP stress test workflow#625
Merged
Conversation
This allows the workflow to download Go modules from: - proxy.golang.org (Go module proxy) - golang.org (Go official site) - go.dev, pkg.go.dev (Go documentation) - goproxy.io (alternative proxy) - sum.golang.org (Go checksum database) Fixes HTTP 403 Forbidden errors when building the awmg binary. Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix network restrictions for Go module downloads in stress test
Allow Go module downloads in MCP stress test workflow
Feb 4, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the nightly MCP stress test workflow’s network configuration so Go module downloads are allowed through the firewall, unblocking make build for the MCP gateway.
Changes:
- Added the
goecosystem to thenetwork.allowedlist innightly-mcp-stress-test.mdso the workflow can use the Go network allowlist. - Updated the generated lock file to include
goinallowed_domainsand to extend bothawf --allow-domainsandGH_AW_ALLOWED_DOMAINSwith Go-related domains (proxy.golang.org,golang.org,go.dev,pkg.go.dev,sum.golang.org,goproxy.io). - Updated the
frontmatter-hashin the lock file to reflect the new frontmatter configuration.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.github/workflows/nightly-mcp-stress-test.md |
Adds network.allowed entries for defaults and go so the MCP stress test job can use Go’s network allowlist. |
.github/workflows/nightly-mcp-stress-test.lock.yml |
Regenerates the lock file to record allowed_domains: ["defaults","go"] and expands the firewall/domain allowlists with Go module and checksum hosts. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The nightly MCP stress test workflow fails during
make buildwith HTTP 403 errors when downloading Go modules. The firewall blocksproxy.golang.organdgolang.orgbecause the workflow usesnetwork: defaultswhich only permits basic infrastructure.Changes
Added
goecosystem identifier to network allowlist:This enables access to:
proxy.golang.org(Go module proxy)golang.org,go.dev,pkg.go.devsum.golang.org(checksum database)goproxy.io(fallback proxy)Impact
Unblocks the stress test workflow. The gateway binary can now build, allowing testing of all 20 configured MCP servers.
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.