Skip to content

CI: add CodeQL and ASF AllowList checks#4132

Merged
adutra merged 1 commit intoapache:mainfrom
adutra:add-security-ci-checks
Apr 8, 2026
Merged

CI: add CodeQL and ASF AllowList checks#4132
adutra merged 1 commit intoapache:mainfrom
adutra:add-security-ci-checks

Conversation

@adutra
Copy link
Copy Markdown
Contributor

@adutra adutra commented Apr 7, 2026

This is a follow-up to #4097 that adds two more checks, taking Apache Iceberg CI as an example.

Checklist

  • 🛡️ Don't disclose security issues! (contact security@apache.org)
  • 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
  • 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
  • 💡 Added comments for complex logic
  • 🧾 Updated CHANGELOG.md (if needed)
  • 📚 Updated documentation in site/content/in-dev/unreleased (if needed)

This is a follow-up to apache#4097 that adds two more checks, taking Apache Iceberg CI as an example.
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

with:
persist-credentials: false
# Intentionally unpinned to always use the latest allowlist from the ASF.
- uses: apache/infrastructure-actions/allowlist-check@main # zizmor: ignore[unpinned-uses]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it not enforced at the org level for all runners?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess it's about more user-friendly error reporting per apache/infrastructure-actions#587 🤔

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Apr 7, 2026
@adutra adutra merged commit 3bbfb61 into apache:main Apr 8, 2026
23 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Apr 8, 2026
@adutra adutra deleted the add-security-ci-checks branch April 8, 2026 09:36
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.

5 participants