[SECURITY] Several command injections
Nobody has claimed this yet.
- Dominant language
- V
- Stars
- 1.5k
- Forks
- 79
- PR merge metrics
- No merged PRs in 30d
Description
There are several command injection vulnerabilities in Gitly:
https://github.com/vlang/gitly/blob/d0e1f3ad2fa3d76306a3de11642f5ff50e9e9ede/src/repo_service.v#L659
I think there are more possible injection points so it is probably not enough to fix these parts only.
If possible, consider Adding a security policy to your repository in the future.
Contributor guide
No contributing guide indexed for this repository
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.
Research direction
Start by reading the linked sections of src/repo_routes.v, src/commit_routes.v, and src/repo_service.v at the cited lines. Trace the inputs through these paths and audit for other possible injection points. Done means the reported vulnerabilities and any related injection points are addressed and verified without leaving command-injection paths.
Written by the indexing model from the issue text.
Assessment
- Domain
- backend, security
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100