ci: lint/go bodyclose in azureidentity_internal_test.go
@jdomeracki-coder ci sta già lavorando.
Dal 13/5/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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)
- Lingua principale
- Nessun dato sulla lingua
- Stelle
- 3
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di coder/internal
-
flake
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
-
flake
-
flake