Skip to content

chore: updated semgrep error outputs to include stdout and stderr#1383

Open
art1f1c3R wants to merge 1 commit intomainfrom
semgrep-error-output
Open

chore: updated semgrep error outputs to include stdout and stderr#1383
art1f1c3R wants to merge 1 commit intomainfrom
semgrep-error-output

Conversation

@art1f1c3R
Copy link
Copy Markdown
Member

@art1f1c3R art1f1c3R commented Apr 23, 2026

Summary

This is a quick PR that just includes the standard output and error of Semgrep when it fails to run.

Description of changes

This PR includes Semgrep stderr and stdout, which makes it easier to determine what caused Semgrep to fail, e.g.:

│ [INFO] 06:36:37 2026-04-23 06:36:37,891  Unable to run semgrep scan --metrics off --disable-version-check --oss-only --disable-nosem --config                                                                                     │
│ /scratch/cflottma/python-malware-dev/macaron/src/macaron/resources/pypi_malware_rules /tmp/Django-5.0.6_qvu0216_/Django-5.0.6 --json-output=/tmp/tmpusseunzu on /tmp/Django-5.0.6_qvu0216_/Django-5.0.6: Command '['semgrep',     │
│ 'scan', '--metrics', 'off', '--disable-version-check', '--oss-only', '--disable-nosem', '--config', '/scratch/cflottma/python-malware-dev/macaron/src/macaron/resources/pypi_malware_rules',                                      │
│ '/tmp/Django-5.0.6_qvu0216_/Django-5.0.6', '--json-output=/tmp/tmpusseunzu']' returned non-zero exit status 2.                                                                                                                    │
│                 Return code: 2                                                                                                                                                                                                    │
│                 stdout:                                                                                                                                                                                                           │
│                 stderr: Failed to find semgrep-core in PATH or in the semgrep package.

Checklist

  • I have reviewed the contribution guide.
  • My PR title and commits follow the Conventional Commits convention.
  • My commits include the "Signed-off-by" line.
  • I have signed my commits following the instructions provided by GitHub. Note that we run GitHub's commit verification tool to check the commit signatures. A green verified label should appear next to all of your commits on GitHub.
  • I have updated the relevant documentation, if applicable.
  • I have tested my changes and verified they work as expected.

@art1f1c3R art1f1c3R requested a review from behnazh-w as a code owner April 23, 2026 06:54
@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Apr 23, 2026
@art1f1c3R art1f1c3R self-assigned this Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant