Skip to content
This repository was archived by the owner on Sep 2, 2025. It is now read-only.

SECENG-1297 [skip ci] Create non-blocking workflow for static code analysis upon PR#7

Open
hseuping wants to merge 1 commit intomasterfrom
semgrep-patch
Open

SECENG-1297 [skip ci] Create non-blocking workflow for static code analysis upon PR#7
hseuping wants to merge 1 commit intomasterfrom
semgrep-patch

Conversation

@hseuping
Copy link

What is this?

The Security Team would like to run semgrep the static analysis tool Semgrep on every Merge Request (MR) across all Navan code repositories. Semgrep will be used to identify security vulnerabilities within code written by Navan developers.

How will this impact my development workflows?

Semgrep should have zero impact. It will run on every single merge request but it is not going to to block or fail your MR right now. We know some of you have had concerns with Apiiro blocking your merge requests, and we worked hard to resolve them. In case of semgrep we are simply not enabling it in blocking mode.

How it is different from Apiiro? Why do we need both?

Semgrep analyzes data and code flow, and identifies issues within the source code. Apiiro currently only analyzes third party dependencies used in each code repository or secrets used for source code. These tools are complimentary and we need both to identify different classes of security issues.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant