express icon indicating copy to clipboard operation
express copied to clipboard

Add CodeQL (SAST)

Open UlisesGascon opened this issue 1 year ago • 1 comments

Main Changes

This change include the addition of CodeQL as the SAST tool for the project. It will run an analysis every day (200fa10)

Impact in the OSSF Scorecard

Screenshot 2024-02-02 at 16 56 32

Context

Changes related

Team discussion related

  • Ref: https://github.com/expressjs/discussions/issues/162
  • Report: https://kooltheba.github.io/openssf-scorecard-api-visualizer/#/projects/github.com/expressjs/express/commit/2a00da2067b7017f769c9100205a2a5f267a884b

Changelog

  • 200fa10 chore: add codeql pipeline by @UlisesGascon

UlisesGascon avatar Feb 02 '24 15:02 UlisesGascon

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.

I applied your suggestions @inigomarquinez. I will love to have one approval before merging it (cc: @expressjs/express-tc ) so we can review the suggestions (if any) on the source code :+1:

UlisesGascon avatar Apr 17 '24 09:04 UlisesGascon