Skip to content

Fix: Grant OIDC and write permissions to deleteDevReleases job - #279

Merged
cx-atish-jadhav merged 1 commit into
mainfrom
other/fix-release
Sep 3, 2026
Merged

Fix: Grant OIDC and write permissions to deleteDevReleases job#279
cx-atish-jadhav merged 1 commit into
mainfrom
other/fix-release

Conversation

@cx-atish-jadhav

Copy link
Copy Markdown
Collaborator

Summary

  • Fix GitHub Actions validation error when calling reusable workflow
  • Reusable workflow job requires explicit permissions to be inherited properly
  • Add explanatory comments for all permissions (Zizmor security requirement)

Changes

  • Add explicit permissions block to deleteDevReleases job in release.yml
  • Include id-token: write, contents: write, and packages: write with comments
  • Maintains restrictive top-level permissions: {} default

Verification

  • ✅ GitHub Actions validation passes
  • ✅ Zizmor security audit passes (0 findings)

Adds the required workflow permissions for the dev-release cleanup job. This enables OIDC token generation and allows the job to delete git tags/releases and clean up package artifacts during release processing.
@cx-atish-jadhav
cx-atish-jadhav requested a review from a team September 3, 2026 09:10
@cx-atish-jadhav
cx-atish-jadhav merged commit a5aa58d into main Sep 3, 2026
6 of 8 checks passed
cx-rakesh-kadu pushed a commit that referenced this pull request Sep 3, 2026
#281)

The deleteDevReleases job's permission gap (workflow granted none while
the reusable delete-dev-releases.yml requires id-token/contents/packages
write) was already fixed on main in PR #279. This follows up on the
decision to drop automatic dev-release cleanup from the release pipeline
entirely, matching other Checkmarx plugin release workflows.

Job is commented out (not deleted) so it can be restored easily. Cleanup
is now done manually via the standalone "Delete dev releases" workflow,
and dev releases no longer trigger a second environment approval prompt.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

3 participants