Manual GitHub release dispatch can publish a branch as latest #49

Open
opened 2026-10-20 18:18:52 +08:00 by Breadway · 0 comments
Owner

Found during fresh independent PR46 review at 2dff10b59c; also noted in prior comment 5672. Confidence: high from workflow source, not executed. Deduplicated against current CRUMB issue cards.

.github/workflows/release.yml:7-8 enables workflow_dispatch without a tag-only job guard. Lines 35-44 derive VERSION from GITHUB_REF_NAME, publish to /srv/breadway-dl/breadcrumbs/, and repoint latest. A manual dispatch on main therefore publishes under breadcrumbs/main and sets latest to main; a feature branch yields a nested path. It subsequently regenerates the shared index. The automatic duplicate-tag race in CRUMB-37 is fixed, but the retained manual fallback needs a safe ref contract.

Suggested follow-up: remove the redundant fallback or require a validated version-tag ref before any build/publish effects. Add a source/fixture check rejecting branch dispatch. No dispatch, tags, release, deployment or implementation was performed during review.

Found during fresh independent PR46 review at 2dff10b59ceaa61cb391132121080603055c3c46; also noted in prior comment 5672. Confidence: high from workflow source, not executed. Deduplicated against current CRUMB issue cards. .github/workflows/release.yml:7-8 enables workflow_dispatch without a tag-only job guard. Lines 35-44 derive VERSION from GITHUB_REF_NAME, publish to /srv/breadway-dl/breadcrumbs/<VERSION>, and repoint latest. A manual dispatch on main therefore publishes under breadcrumbs/main and sets latest to main; a feature branch yields a nested path. It subsequently regenerates the shared index. The automatic duplicate-tag race in CRUMB-37 is fixed, but the retained manual fallback needs a safe ref contract. Suggested follow-up: remove the redundant fallback or require a validated version-tag ref before any build/publish effects. Add a source/fixture check rejecting branch dispatch. No dispatch, tags, release, deployment or implementation was performed during review. <!-- breadboard-import:CRUMB-49 -->
Sign in to join this conversation.
No labels
bug
ci
enhancement
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Breadway/breadcrumbs#49
No description provided.