ci: lint/go bodyclose in azureidentity_internal_test.go
@jdomeracki-coder đang làm issue này rồi.
Từ ngày 13/5/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
CI Run Link: https://github.com/coder/coder/actions/runs/25791859937
Failed Job: https://github.com/coder/coder/actions/runs/25791859937/job/75759237891 (lint)
Commit Info:
- SHA: fb3aef1883eeb2bca266a267f474e8ddb062b406
- Author: Jakub Domeracki
- https://github.com/coder/coder/commit/fb3aef1883eeb2bca266a267f474e8ddb062b406
Error Evidence:
##[error]coderd/azureidentity/azureidentity_internal_test.go:67:31: response body must be closed (bodyclose)
_, err := certFetchClient.Get(srv.URL)
^
Root Cause:
TestCertFetchClientRejectsLoopback calls certFetchClient.Get(srv.URL) without closing the response body, tripping the bodyclose linter in make lint.
Precise Assignment Analysis:
- git blame is not available via the API; used file history instead.
git log --oneline -10 --follow coderd/azureidentity/azureidentity_internal_test.go-> fb3aef188 (Jakub Domeracki) most recent change.
Assigning tojdomeracki-coderas the author of the latest changes to this test.
Classification:
E. Code Change (lint violation).
Related Issues Searched:
- "response body must be closed" + bodyclose
- "azureidentity_internal_test.go"
- "azureidentity" + lint
Reproduction:
make lint(orgolangci-lint runwith bodyclose enabled)
- Ngôn ngữ chính
- Không có dữ liệu ngôn ngữ
- Star
- 3
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 coder/internal
-
flake
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
chore: cleanup http transport debtCó thể đã có người làm @sreya đã nhận 5 ngày trước. Đang mở
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
-
flake: TestPubsubCluster/ClusterAuthRequiredCó thể làm lại được @sreya đã nhận 56 ngày trước và không có pull request nào đang mở. Đang mởflake