Disable vulnerable mcp/notion container integration (Critical/High CVEs, no upstream fix) - #49805
Conversation
…tion docker.io/mcp/notion has Critical/High CVEs with no upstream fix available. Follows the same pattern used for mcp/brave-search (issue #48546). - Gut shared/mcp/notion.md container definition, add security comment - Comment out notion.md import in mcp-inspector.md (like brave.md) - Remove mcp/notion pin from actions-lock.json and both action_pins.json mirrors - Recompile all 268 workflows; mcp/notion SHA no longer appears in any lock file Closes #49517 Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot resolve the merge conflicts on this branch. |
There was a problem hiding this comment.
Pull request overview
Disables the vulnerable mcp/notion container integration pending an upstream fix.
Changes:
- Removes the Notion container configuration and image pins.
- Excludes Notion from MCP Inspector.
- Regenerates affected workflow lock files.
Show a summary per file
| File | Description |
|---|---|
.github/aw/actions-lock.json |
Removes the Notion image pin. |
.github/workflows/mcp-inspector.md |
Excludes the Notion integration. |
.github/workflows/mcp-inspector.lock.yml |
Regenerates MCP Inspector without Notion. |
.github/workflows/notion-issue-summary.lock.yml |
Regenerates the summary workflow without Notion. |
.github/workflows/shared/mcp/notion.md |
Disables the shared integration. |
pkg/actionpins/data/action_pins.json |
Removes the runtime pin mirror. |
pkg/workflow/data/action_pins.json |
Removes the compiler pin mirror. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 7/7 changed files
- Comments generated: 1
- Review effort level: Balanced
| # Notion MCP Server | ||
| # SECURITY: docker.io/mcp/notion has Critical/High CVEs with no upstream fix available (issue #49517). | ||
| # The container definition has been removed until a patched image is published upstream. | ||
| # To re-enable, restore the mcp-servers and safe-outputs blocks and update the pinned digest in actions-lock.json. |
…ge-scan-mcp-notion # Conflicts: # .github/workflows/mcp-inspector.lock.yml Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…h markitdown and notion) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Resolved. The conflict was in |
|
@copilot This PR has merge conflicts. Please run Run details: https://github.com/github/gh-aw/actions/runs/30756126865
|
…ge-scan-mcp-notion Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Merged main (commit 9eb50e1 — typeutil refactor) with no conflicts. All 268 workflows compile and lock files are in sync. |
|
@copilot This PR has merge conflicts. Please run Run details: https://github.com/github/gh-aw/actions/runs/30757146330
|
…ge-scan-mcp-notion Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Merged latest main (fe1ccb6 — watchdog authentication fix) with no conflicts. All 268 workflows compile and lock files are in sync. |
|
🎉 This pull request is included in a new release. Release: |
docker.io/mcp/notioncarries 258 vulnerabilities (11 Critical, 74 High) including stdlib Go and glibc issues. Docker Hub has only one tag (latest) — no patched image exists upstream.Applies the same remediation pattern used for
mcp/brave-search(#48546):shared/mcp/notion.md— strippedmcp-serversandsafe-outputsblocks; replaced with a security comment referencing the issue and instructions for re-enabling once a patched image landsmcp-inspector.md— commented outshared/mcp/notion.mdimport with a CVE note (mirrors thebrave.mdexclusion comment)actions-lock.json+ bothaction_pins.jsonmirrors — removed themcp/notioncontainer pin entrymcp/notion@sha256:df0d6781...is no longer referenced in any.lock.yml;NOTION_API_TOKENdropped from compiled manifests