frappe / frappe/semgrep-rules

Why are subprocess exec rule not included in the semgrep rules

Open
#44 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
66
Forks
54
PR merge metrics
No merged PRs in 30d

Description

Right now, while publishing the app the Audit fails with subprocess exec

```
frappe-subprocess-exec

subprocess call spawns an external process. Needs to be reviewed by review/security team to confirm arguments are a static list and any interpolated values are validated.
```

It should also be part of this semgrep rule if this needs to be enforced

Contributor guide

No contributing guide indexed for this repository

Research direction

No file or test is named. Start by locating the semgrep rule for frappe-subprocess-exec and the publishing app Audit configuration; determine how subprocess exec calls are matched, then verify that the intended calls are included and the Audit reports them consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
security, tooling
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.