Skip to content

[actions] Support Semgrep by Github Actions#283

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
maipbui:semgrep
Jun 19, 2023
Merged

[actions] Support Semgrep by Github Actions#283
qiluo-msft merged 1 commit intosonic-net:masterfrom
maipbui:semgrep

Conversation

@maipbui
Copy link
Contributor

@maipbui maipbui commented Jun 19, 2023

Why I did it
Semgrep is a static analysis tool to find security vulnerabilities.
When opening a PR or commtting to PR, Semgrep performs a diff-aware scanning, which scans changed files in PRs.
When merging PR, Semgrep performs a full scan on master branch and report all findings.

Ref: - Supported Language - Semgrep Rules

How I did it
Integrate Semgrep into this repository by committing a job configuration file
- How to verify it

- Description for the changelog

Signed-off-by: Mai Bui <[email protected]>
@qiluo-msft qiluo-msft merged commit 4948ea3 into sonic-net:master Jun 19, 2023
@maipbui maipbui deleted the semgrep branch June 20, 2023 00:45
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Jun 20, 2023
Update sonic-snmpagent submodule pointer to include the following:
* 4948ea3 add semgrep ([sonic-net#283](sonic-net/sonic-snmpagent#283))

Signed-off-by: dprital <[email protected]>
ssithaia-ebay pushed a commit to ssithaia-ebay/sonic-snmpagent that referenced this pull request May 23, 2025
**Why I did it**
[Semgrep](https://github.com/returntocorp/semgrep) is a static analysis tool to find security vulnerabilities.
When opening a PR or commtting to PR, Semgrep performs a diff-aware scanning, which scans changed files in PRs.
When merging PR, Semgrep performs a full scan on master branch and report all findings.

Ref: - [Supported Language](https://semgrep.dev/docs/supported-languages/#language-maturity) - [Semgrep Rules](https://registry.semgrep.dev/rule)

**How I did it**
Integrate Semgrep into this repository by committing a job configuration file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants