aRustyDev / aRustyDev/helm-charts

feat(ci): enhance W6 trigger security with input validation and tag protection

Open
#136 0 comments 0 reactions 1 assignee Claimed by @aRustyDev View on GitHub
cicd enhancement
Dominant language
Go Template
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

## Context

W6 (Release Atomic Chart) currently uses `workflow_dispatch` which can be triggered externally via the GitHub API by anyone with write access to the repository.

We're implementing `repository_dispatch` + GitHub App token as the primary approach, but there are additional security enhancements to consider.

## Enhancement Ideas

### Input Validation
- Add regex validation for tag format in the dispatch handler
- Validate that the tag exists before processing
- Validate that the tag points to a commit on main

### Branch/Tag Protection Rules
- Configure tag protection rules to limit who can create release tags
- Pattern: `*-v*` (matches `cloudflared-v0.1.0`, etc.)
- Require specific actors (GitHub App, admins) for tag creation

### Audit Logging
- Log all dispatch events with actor information
- Track manual vs automated releases

## Related
- W6 workflow: `.github/workflows/release-atomic-chart.yaml`
- W6-Tag workflow: `.github/workflows/tag-atomic-chart.yaml`

## Labels
- `enhancement`
- `cicd`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.