Skip to content

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

Merged
alamb merged 1 commit intoapache:mainfrom
kevinjqliu:add-codeql-workflow
Mar 2, 2026
Merged

CI: Add CodeQL workflow for GitHub Actions security scanning#152
alamb merged 1 commit 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.

Copy link
Contributor

@alamb alamb 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 -- this looks great. Let's give it a try!

@alamb alamb merged commit 7ddb694 into apache:main Mar 2, 2026
4 checks passed
@alamb
Copy link
Contributor

alamb commented Mar 2, 2026

@kevinjqliu do we need to do anything more to get the scan results in this repo?

@alamb
Copy link
Contributor

alamb commented Mar 2, 2026

Specifically, https://github.com/apache/datafusion-site/security still shows that we need to setup security scans 🤔

Screenshot 2026-03-02 at 9 59 09 AM

@alamb
Copy link
Contributor

alamb commented Mar 2, 2026

Actually, it is working now: https://github.com/apache/datafusion-site/security

Screenshot 2026-03-02 at 10 00 50 AM

Now we just need to go sort out those alerts

@kevinjqliu kevinjqliu deleted the add-codeql-workflow branch March 2, 2026 19:20
@kevinjqliu
Copy link
Contributor Author

Thanks @alamb I opened #153 as a follow up

@kevinjqliu
Copy link
Contributor Author

btw i think only those with write permissions (committer/pmc) can see the Security tab.

This is my view for https://github.com/apache/datafusion-site/security
Screenshot 2026-03-02 at 11 33 49 AM

But I can see results in the Iceberg repos.

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

Successfully merging this pull request may close these issues.

2 participants