GitHub release can attach to an existing tag on another commit
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 72/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- github, typescript
Hướng nghiên cứu
Đọc src/lib/github.ts để so sánh bước kiểm tra ref và SHA hiện có của createTag với createRelease, sau đó lần theo đường đi mặc định githubRelease: true trong src/release/index.ts. Xác nhận cách xử lý tag hiện có trỏ đến một commit khác và thêm bước kiểm tra hoặc tạo tag trước khi tạo release. Hoàn thành khi release thất bại hoặc dùng SHA mong đợi nếu tag đã trỏ đến commit khác.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Issue description
createRelease (src/lib/github.ts) sends tag_name + target_commitish. When the tag already exists on another commit, GitHub ignores target_commitish and attaches the release to the existing tag, so the release silently points at the wrong commit. createTag checks that an existing tag is on the expected sha, but the default path (githubRelease: true, src/release/index.ts) never calls it.
Expected: before createRelease, check the tag like createTag does (getRef, compare the sha) and fail when it points elsewhere, or create the tag through createTag first.
Likelihood is low: the config checks prevent tag collisions between groups. A stale or hand-made tag still triggers it.
Found by the quality gate of the v3 rework, branch feat/v3-shared-workflows (commit 4617e86, not pushed yet when this was filed).
Code sample
_No response_
Versions
- @nanoforge-dev/actions v3 (branch feat/v3-shared-workflows)
Issue priority
Low (slightly annoying)
I have tested this issue on a development release
No response
- Ngôn ngữ chính
- TypeScript
- Star
- 0
- Fork
- 0
- Merge trung bình
- 15 giờ 51 phút
- Pull request đã merge (30 ngày)
- 21
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của NanoForge-dev/actions
-
bug dependencies
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
NanoForge-dev/actions#137 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
NanoForge-dev/actions#136 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Registry release silently skips manifests with no name and does not name invalid JSON manifestsĐang mởbug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
NanoForge-dev/actions#135 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
NanoForge-dev/actions#134 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
NanoForge-dev/actions#131 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của NanoForge-dev/actions
Issue tương tự
-
Mend: dependency security vulnerability
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 65/100
fdm-monster/fdm-monster#5563 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
Maintainer thường phản hồi trong vòng 1 ngày
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
growilabs/growi#12246 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededCó thể đã có người làm @github-actions đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
SSWConsulting/SSW.Rules#2809 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày