Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Security scanning in CI #487

Open
daniel-ac-martin opened this issue May 25, 2022 · 11 comments
Open

Security scanning in CI #487

daniel-ac-martin opened this issue May 25, 2022 · 11 comments

Comments

@daniel-ac-martin
Copy link
Owner

daniel-ac-martin commented May 25, 2022

It would be nice to have some sort of security scanning functionality in CI to try to catch any security problems.

SCA / Dependency scanning

SAST

DAST

Perhaps with ZAP?

Could like at this: https://github.com/marketplace/actions/owasp-zap-full-scan
Also: https://github.com/marketplace/actions/owasp-zap-baseline-scan
(But it might be better to have a way to run it locally as well.)

For APIs (inc. GraphQL) perhaps we should consider: https://github.com/zaproxy/action-api-scan

See also:

@daniel-ac-martin
Copy link
Owner Author

daniel-ac-martin commented Jun 14, 2022

It looks like SLScan might do everything aside from DAST.
So the solution might just be SLScan + ZAP.

@daniel-ac-martin
Copy link
Owner Author

Looks like SLScan doesn't actually do proper SAST on Type/JavaScript.

@daniel-ac-martin
Copy link
Owner Author

Sounds like it used to do it with njsscan.
I've asked whether they will do it again in the future: ShiftLeftSecurity/sast-scan#367

@daniel-ac-martin
Copy link
Owner Author

Looks like SLScan is deprecated: ShiftLeftSecurity/sast-scan#352
But it might still be the best open-source option. (When combined with a separate SAST tool for TypeScript.)

@daniel-ac-martin
Copy link
Owner Author

CodeQL looks good for most of the static side. (Besides dependencies.)
Possibly, we could use Snyk for dependencies with https://www.npmjs.com/package/snyk-pnpm-deptree-api-tool
Zap is probably good for dynamic/DAST but we need to find a way to stand-up a copy of the app in CI.

@daniel-ac-martin
Copy link
Owner Author

OWASP dependency check supports SARIF output and so might fit well with CodeQL.

@daniel-ac-martin
Copy link
Owner Author

Perhaps we should replace SLScan with https://github.com/marketplace/actions/dependency-check ?
It seems to be lighter weight and should integrate with GitHub via SARIF output.
(Snyk might be another option but depends on a SaaS account.)

@daniel-ac-martin
Copy link
Owner Author

daniel-ac-martin commented Jun 27, 2022

Can we use ZAP's API scanner?: https://github.com/zaproxy/action-api-scan

@daniel-ac-martin
Copy link
Owner Author

Can we scan Docker images with Trivy?: https://www.aquasec.com/products/trivy/

@daniel-ac-martin
Copy link
Owner Author

@prabhu
Copy link

prabhu commented Feb 1, 2023

@daniel-ac-martin I am working on a proper OSS SAST tool for javascript/typescript that is powered by joern. Will get in touch with you as soon as it's ready.

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

No branches or pull requests

2 participants