Skip to content

Update gitleaks report to sarif format#466

Open
elvin03 wants to merge 9 commits intomainfrom
gitleak-sarif
Open

Update gitleaks report to sarif format#466
elvin03 wants to merge 9 commits intomainfrom
gitleak-sarif

Conversation

@elvin03
Copy link
Contributor

@elvin03 elvin03 commented Mar 5, 2026

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • Documentation has been updated to reflect the changes (or no doc update needed)
  • Ready to merge

Description

Update gitleaks report to sarif format

Any Newly Introduced Dependencies

How Has This Been Tested?

Trigger gitleaks workflow

Signed-off-by: Teoh Suh Haw <suh.haw.teoh@intel.com>
Copilot AI review requested due to automatic review settings March 5, 2026 03:22
@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

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s Gitleaks GitHub Actions workflow to generate SARIF output and upload results to GitHub code scanning (Security tab).

Changes:

  • Adds security-events: write (and actions: read) permissions to the Gitleaks workflow.
  • Switches Gitleaks report format from csv to sarif.
  • Updates the workflow input path from a .csv file to a .sarif file.

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