ci: lint/go bodyclose in azureidentity_internal_test.go
@jdomeracki-coder ya está trabajando en esto.
Desde el 13/5/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
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)
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 3
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de coder/internal
-
flake
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
flake
-
flake