Conversation
Runs on PR, push to main, daily at 03:00 UTC, and manual dispatch.
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
|
Note: Dependabot focuses on known CVEs in direct/transitive dependencies and can auto-create PRs to bump versions. It's reactive - OSV Scanner uses the broader OSV database which aggregates multiple sources beyond just CVEs (GitHub Advisories, PyPI, Go, That said, for a pure npm project like sci-react-ui, the overlap is significant. The main added value of OSV Scanner here is |
akademy
left a comment
There was a problem hiding this comment.
Now that the vulnerabilities are being highlighted, it would be a good time to fix them!
See if you can fix the current security concerns with some package updates...
Sure, that was the plan right after this is merged. Current vulnerability highlights come from dependabot, this workflow had not uploaded any findings yet but when it does it will publish to: https://github.com/DiamondLightSource/sci-react-ui/security/code-scanning I want to merge this before addressing because merging will trigger a full run (not just on PR changes) and I want to see that first |
Summary
Test plan