8000 Blog: Add article about SAST scanning using semgrep by malexmave · Pull Request #150 · secureCodeBox/documentation · GitHub
[go: up one dir, main page]

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

Blog: Add article about SAST scanning using semgrep#150

Merged
Weltraumschaf merged 4 commits intomainfrom
blog/sast-scanning
Oct 28, 2021
Merged

Blog: Add article about SAST scanning using semgrep#150
Weltraumschaf merged 4 commits intomainfrom
blog/sast-scanning

Conversation

@malexmave
Copy link
Member
@malexmave malexmave commented Oct 27, 2021

I opted to present the new features as part of a larger narrative about how you can use secureCodeBox to run one-off analyses your codebase, using the UA-parser.js backdoor as a motivating example (which makes for a more compelling story than saying "secureCodeBox: Like SAST in your CI, but worse!" 😁).

Looking forward to your feedback.

Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
@malexmave malexmave added the blog Everything related to blog posts. label Oct 27, 2021
@malexmave malexmave self-assigned this Oct 27, 2021
@Weltraumschaf Weltraumschaf self-requested a review October 28, 2021 08:06
Copy link
Member
@Weltraumschaf Weltraumschaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work <3

key: token
```

If you load this cascading rule and start the git-repo-scanner scan we defined above, it automatically starts scans for all detected (public) repositories (make sure the [git-repo-scanner][gitreposcanner] and [semgrep][semgrep-scb] scantypes as well as the [CascadingScans hook][cascadingscans] are installed).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why public? Above I saw some Git token.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe you should outline more prominent at the beginning that this example operates on public GitHub repos. First I thought this is a generic example where I have to fill in the gaps with my git server URI.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The idea was to also showcase the filtering possibilities of cascading scan jobs (it is explained in greater detail the first time the cascading scan job is discussed). I have made some changes with the last commit, let me know if you like it like this, or if you would prefer if I completely remove the filtering examples.

< 8000 /svg>
malexmave and others added 2 commits October 28, 2021 12:45
Co-authored-by: Sven Strittmatter <sven.strittmatter@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
Signed-off-by: Max Maass <max.maass@iteratec.com>
@malexmave
Copy link
Member Author

Seems like there was an issue with the release of the docker images for the semgrep parser, so maybe we should hold off on publishing this until that is resolved (see secureCodeBox/secureCodeBox#784).

@Weltraumschaf
Copy link
Member

IMHO the release issue is not a big deal.

@Weltraumschaf Weltraumschaf merged commit cca2c18 into main Oct 28, 2021
@Weltraumschaf Weltraumschaf deleted the blog/sast-scanning branch February 9, 2022 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

blog Everything related to blog posts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0