【架构优化】合并 domain.PublicRunnerTask 中单仓库/多仓库重复的凭据脱敏为 publicRunnerRepository
Maintainer thường phản hồi trong vòng 1 ngà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
- 20/100
Hướng nghiên cứu
The duplicated scrubbing lives in PublicRunnerTask in internal/domain/runner.go, once for the single Repository field and once for the Repositories list. Read both blocks and the existing tests TestPublicRunnerTaskScrubsMultiRepositoryTokens and TestPublicRunnerTaskStripsCloneURLCredentials in runner_test.go. Work is already in progress in linked PR #420, so check that first. Done means one shared helper handles both paths and the tests pass under go test ./internal/domain/.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
现状问题
internal/domain/runner.go 的 PublicRunnerTask 对 Invocation.Repository（旧的单仓库字段）和 Invocation.Repositories（多仓库列表）各写了一遍相同的脱敏逻辑：清空 GitToken、用 StripURLCredentials 去掉 CloneURL 里的凭据。两处必须保持同步，以后给 RunnerRepository 加敏感字段时容易只改一处，造成凭据泄漏到管理接口响应。
重构方案
抽出包内函数 publicRunnerRepository(repository RunnerRepository) RunnerRepository（值传入、值返回，不改调用方数据），单仓库和多仓库两处都改为调用它。PublicRunnerTask 签名与行为不变。
影响范围
仅 internal/domain/runner.go 一个函数内部，以及 internal/domain/runner_test.go 新增测试；不改对外接口、不改 JSON 结构。
验证方式
- 新增
publicRunnerRepository单元测试：脱敏 token 与 URL 凭据、保留其他字段、不修改入参。 - 原有
TestPublicRunnerTaskScrubsMultiRepositoryTokens/TestPublicRunnerTaskStripsCloneURLCredentials继续通过。 gofmt、go vet ./internal/domain/、go test ./internal/domain/。
- Ngôn ngữ chính
- Go
- Star
- 141
- Fork
- 28
- Merge trung bình
- 1 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 113
Chuẩn bị môi trường
- Có Dockerfile hoặc 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 Tencent/LoopForge
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
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 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」Có thể đã có người làm @newexploreor đã nhận 1 ngày trước. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Tencent/LoopForge#156 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentityCó thể đã có người làm @fx-kurumi đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 40/100
Maintainer thường phản hồi trong vòng 1 ngày
-
【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStageCó thể đã có người làm @fx-kurumi đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 30/100
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của Tencent/LoopForge
Issue tương tự
-
team:library-maintainers
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
padok-team/burrito#1208 · 1 bình luận ·
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 62/100
bazel-contrib/rules_go#4784 · 1 bình luận ·
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 75/100
-
Уязвимые компоненты в GoĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
POSIdev-community/aictl#40 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
prime-radiant-inc/evener#6306 ·
Maintainer thường phản hồi trong vòng 1 ngày