Skip to content

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

Merged
art1f1c3R merged 1 commit into
mainfrom
semgrep-error-output
Apr 27, 2026
Merged

chore: updated semgrep error outputs to include stdout and stderr#1383
art1f1c3R merged 1 commit into
mainfrom
semgrep-error-output

Conversation

@art1f1c3R

@art1f1c3R art1f1c3R commented Apr 23, 2026

Copy link
Copy Markdown

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
Signed-off-by: Carl Flottmann <carl.flottmann@oracle.com>
@art1f1c3R art1f1c3R force-pushed the semgrep-error-output branch from 4989952 to 0891e4f Compare April 27, 2026 01:05
@art1f1c3R art1f1c3R merged commit 1564fb5 into main Apr 27, 2026
18 checks passed
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.

2 participants