feat(core): report a clear error when the configured branch doesn't exist
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- Nửa ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
The branch is read from githubbranch or LGSM_GITHUBBRANCH, and bootstrap and update-lgsm are the places the check would run; the payload names no specific files, so locate the download helpers by searching for the GitHub fallback logic. Done means one branch-existence check per run that fails with the requested message on a 404 only, while timeouts, 429 and 5xx keep the existing fallback. Confirm the design with maintainers before writing code, since the issue reads as a proposal rather than a settled fix.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
If the configured branch (githubbranch, or LGSM_GITHUBBRANCH) doesn't exist on githubuser/githubrepo, for example after a typo or once a feature branch has been deleted after merging, LinuxGSM's module downloads fail with generic download errors. The real cause isn't reported.
Proposal
Check once, during bootstrap and in update-lgsm, that the branch exists, and fail with a clear message if it doesn't:
Branch "<branch>" not found on <githubuser>/<githubrepo>
- Check once per run, not on every file download.
- Use a connect timeout, like the existing repo selection (
--connect-timeout 3). - Treat only a 404 as "branch missing". Other errors (timeouts, 429, 5xx) should keep the current GitHub/Bitbucket fallback.
- Fail clearly rather than silently switching to
master, so nobody ends up on code they didn't choose.
Replaces the draft #4170 (closed): it had bugs, including a sed -i ...;q in update-lgsm that would have cut the game server script down to one line, and it was 486 commits behind develop.
- Ngôn ngữ chính
- Shell
- Star
- 4.9k
- Fork
- 870
- Merge trung bình
- 14 giờ 50 phút
- Pull request đã merge (30 ngày)
- 109
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- 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 GameServerManagers/LinuxGSM
-
command: install command: start distro: Debian engine: source2 game: Counter-Strike 2 regression: yes reproducible: always severity: high type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
GameServerManagers/LinuxGSM#5106 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
command: start engine: unity3d game: Assetto Corsa priority: high scope: single game type: feature
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
GameServerManagers/LinuxGSM#5099 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
engine: source game: Team Fortress 2 game: Team Fortress 2 Classified info: docker info: steamcmd type: bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
GameServerManagers/LinuxGSM#5094 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
type: bug
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 42/100
GameServerManagers/LinuxGSM#5048 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
engine: unity3d engine: unreal4 Epic game: 7 Days to Die game: ARK: Survival Evolved game: Rust info: dependency type: feature
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 8/100
GameServerManagers/LinuxGSM#5047 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của GameServerManagers/LinuxGSM
Issue tương tự
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
glossyio/traffic-monitor#256 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
source:reader type:content-error
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 84/100
stephrobert/blog-roadmap#121 ·
-
lesson-submission pending-review
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 74/100
Ikalus1988/MisakaNet#3899 ·
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 68/100
awsdocs/amazon-eks-user-guide#1257 ·
-
eks-recon: six more null-iteration crashes (networking env/args, observability ConfigMap data)Đang mởdeep-dive-feedback
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
aws-samples/sample-apex-skills#292 ·
Maintainer thường phản hồi trong vòng 1 ngày