-
Notifications
You must be signed in to change notification settings - Fork 408
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
Use Static Code Analysis Tooling #1696
Comments
This is also needed to get the passing badge of OpenSSF Best Practices.
|
The checks currently looks for known GitHub apps such as CodeQL (github-code-scanning) or SonarCloud in the recent (~30) merged PRs, or the use of "github/codeql-action" in a GitHub workflow. (I don't know exactly how that 2 tools work...) Of course we can also decide to not used that ones and use instead some classic one : That last ones could probably be integrated in our maven build but with that one we will not get scorecard point. |
@jvermillard @JaroslawLegierski @netomi any opinion ? 🙏 |
I would start with CodeQL and see if it produces useful results. If you go to the configuration tab and security you can setup a workflow from there for your project. The other options are also pretty good imho, but integration with GitHub might be a bit more work. |
CodeQL seems easy to set-up but this is not open-source at all just "free" for Open Source project. Maybe I'm wrong but I do not like so much the idea... |
I will give a try with sonar because :
I asking myself if there is SonarQube Community Build instance hosted by Eclipse Foundation 🤔 |
(I was not able to add eclipse-leshan organisation to sonarcloud, so I created an eclipse help desk issue) |
To improve our OpenSSF score, it is recommended to use Static Application Security Testing (SAST), also known as static code analysis.
See documentation for more details.
This issue aims to discuss about that and decide If we should add it or not and which one should be used ?
(Even if increasing the note is a criteria, this move should not be only driven by that)
The text was updated successfully, but these errors were encountered: