Skip to content

CI: Add CodeQL workflow for GitHub Actions security scanning#1484

Open
kevinjqliu wants to merge 2 commits intoapache:mainfrom
kevinjqliu:add-codeql-workflow
Open

CI: Add CodeQL workflow for GitHub Actions security scanning#1484
kevinjqliu wants to merge 2 commits intoapache:mainfrom
kevinjqliu:add-codeql-workflow

Conversation

@kevinjqliu
Copy link
Contributor

This adds a CodeQL workflow to scan GitHub Actions workflow files for security issues such as script injection, use of untrusted input, and other misconfigurations.

Reference: https://github.blog/security/application-security/how-to-secure-your-github-actions-workflows-with-codeql/

Triggers:

  • Push and PR to main
  • Weekly scheduled scan (Mondays at 4:16 UTC)

This is based on Apache Infra recommendation,

IMPORTANT! You should enable CodeQL "actions" scanning in your repositories as described in https://github.blog/security/application-security/how-to-secure-your-github-actions-workflows-with-codeql/ - this will scan and flag those issues described below and many more automatically for you


This PR was generated by https://gist.github.com/kevinjqliu/97d24733c7b75cd92b68bf8f5b247891

@github-advanced-security
Copy link

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.

@milenkovicm
Copy link
Contributor

thanks @kevinjqliu
once it gets merged in datafusion we can merge this, following comments at apache/datafusion#20636 at the

@kevinjqliu
Copy link
Contributor Author

updated with version pinning, thanks!

Copy link
Contributor

@milenkovicm milenkovicm left a comment

Choose a reason for hiding this comment

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

thanks @kevinjqliu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants