Verify hashes for the Serena package installation
Maintainers usually reply within 1 day
@lpcox is already working on this.
Since Oct 16, 2026.
- #14998 by @copilot-swe-agent — closed without merging
Assessment
This issue has not been assessed yet.
Description
Code scanning finding
- Alert: Code scanning alert #6
- Scanner: Opengrep OSS
- Severity: High
- Rule:
dockerfile-pip-install-no-hash-check - Location:
containers/serena-mcp-server/Dockerfile:41-43
Finding
The image installs Serena directly from an unpinned GitHub branch and falls back to an unhashed PyPI package:
RUN pip install --no-cache-dir git+https://github.com/oraios/serena.git || \
(echo "GitHub installation failed, trying PyPI..." && \
pip install --no-cache-dir serena-agent)
Neither source is integrity-verified. The Git dependency is also not pinned to an immutable commit, and the fallback can silently produce materially different artifacts depending on transient availability.
Recommended remediation
Use a locked dependency artifact with hashes and install using pip --require-hashes, or otherwise pin the Git dependency to a reviewed commit and verify the downloaded artifact. Reconsider the automatic fallback so a failed primary install does not silently change the dependency source.
Acceptance criteria
- Serena and its transitive dependencies are pinned to reviewed immutable versions.
- Package artifacts are verified with hashes or an equivalent integrity mechanism.
- Installation fails explicitly rather than silently switching to an unverified source.
- The Serena container still builds successfully.
- Code scanning alert #6 is resolved, or dismissed with documented justification.
- Dominant language
- Go
- Stars
- 181
- Forks
- 53
- Avg merge
- 7h 8m
- Merged PRs (30d)
- 215
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- Ships a Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github/gh-aw-mcpg
-
automation mcp-gateway test
Difficulty 1/5 Under an hour Newbie friendliness 8/100
github/gh-aw-mcpg#15068 ·
Maintainers usually reply within 1 day
-
Support per-server custom CA certificates for HTTP MCP upstreamsPossibly taken @lpcox claimed this today. Openenhancement from-gh-aw
github/gh-aw-mcpg#15065 · 1 reaction · 2 assignees ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 5/100
github/gh-aw-mcpg#15055 ·
Maintainers usually reply within 1 day
-
automation repo-assist
Difficulty 5/5 Over a week Newbie friendliness 15/100
github/gh-aw-mcpg#14238 ·
Maintainers usually reply within 1 day
-
automation mcp-gateway test
Difficulty 1/5 Under an hour Newbie friendliness 5/100
github/gh-aw-mcpg#6522 ·
Maintainers usually reply within 1 day
All issues in github/gh-aw-mcpg
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 81/100
apache/rocketmq-dashboard#6895 ·
Maintainers usually reply within 3 days
-
Difficulty 1/5 Under an hour Newbie friendliness 87/100
777genius/agent-notifications#490 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
devantler-tech/ksail#7836 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
keyxmakerx/Chronicle#1469 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
Maintainers usually reply within 1 day