Skip to content

fix: resource not accessible by integration#91

Open
hieu-w wants to merge 3 commits into
mainfrom
hieu/fix-permission-issue
Open

fix: resource not accessible by integration#91
hieu-w wants to merge 3 commits into
mainfrom
hieu/fix-permission-issue

Conversation

@hieu-w

@hieu-w hieu-w commented Jun 23, 2026

Copy link
Copy Markdown

Summary

Adds actions: read permission to the zizmor job in the security scan workflow to fix "resource not accessible by integration" errors.

Changes

  • Grant actions: read on the zizmor job alongside existing contents: read and security-events: write permissions

Why

The zizmor job analyzes GitHub Actions workflows and needs read access to Actions resources. Without this permission, the job fails when running via the GitHub App integration.

Test plan

  • Run the security scan workflow on a test repo and confirm the zizmor job completes without permission errors
  • Confirm zizmor SARIF is still uploaded to code scanning

Note

Low Risk
Single read-only permission on one CI job; aligns zizmor with existing scan jobs and does not broaden write or deployment access.

Overview
Fixes the zizmor job in the security scan workflow failing with "resource not accessible by integration" when the scan runs via the GitHub App (notably on private repos).

Adds actions: read to that job’s permissions so it matches CodeQL and Semgrep, which already use the same trio (actions: read, contents: read, security-events: write). zizmor needs Actions API access to analyze workflow files.

CHANGELOG records the fix under Unreleased → Fixed.

Reviewed by Cursor Bugbot for commit 9f7a73b. Bugbot is set up for automated code reviews on this repo. Configure here.

@hieu-w hieu-w requested a review from a team as a code owner June 23, 2026 17:02

@Mrtenz Mrtenz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Blocking this while waiting for response on Slack.

Comment thread CHANGELOG.md Outdated
Co-authored-by: Maarten Zuidhoorn <maarten@zuidhoorn.com>
@hieu-w hieu-w requested a review from Mrtenz June 24, 2026 09:44
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