test(package-feeds): assert the cargo branch of the package-manager wrapper
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Lee wrapper_routes_only_its_own_process_and_carries_no_credential en src/compile/extensions/package_feeds.rs e inspecciona cómo divide y verifica las ramas generadas del wrapper. Añade aserciones para las cuatro variables de Cargo y comprueba que el helper del token solo genera ADO_MCP_TOKEN_SENTINEL, sin ninguna referencia a una variable de credenciales. Se considera terminado cuando estas aserciones unitarias cubran la rama de Cargo y el helper del token.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Follow-up from the #2315 review.
wrapper_routes_only_its_own_process_and_carries_no_credential (src/compile/extensions/package_feeds.rs) asserts the npm, pip, and dotnet branches of the generated package-manager wrapper, but not cargo). That branch carries the most delicate settings:
CARGO_HTTP_PROXYandCARGO_HTTP_CAINFO(process-scoped proxy and trust bundle);CARGO_CREDENTIAL_ALIAS_ADOAWPROXYplusCARGO_REGISTRY_GLOBAL_CREDENTIAL_PROVIDERS=adoawproxy.
Nothing checks that the Cargo token helper (PACKAGE_CARGO_TOKEN) renders only the non-secret ADO_MCP_TOKEN_SENTINEL.
Acceptance criteria
- Slice-assert the
cargo)case for all four variables. - Assert that the rendered token helper prints only the sentinel and references no credential variable.
- Cargo is not exercised by the live smoke (there is no Cargo runtime), so these unit assertions are its main guard.
- Lenguaje dominante
- Rust
- Estrellas
- 28
- Forks
- 8
- Merge medio
- 3 d 18 h
- PR fusionados (30 d)
- 10
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
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 githubnext/ado-aw
-
bug rust
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
githubnext/ado-aw#2359 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
perf(compile): avoid blocking fs reads in the async compile path for the package-config checkAbiertoenhancement refactor rust
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
githubnext/ado-aw#2358 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
docs documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
githubnext/ado-aw#1766 ·
Los mantenedores suelen responder en 2 días
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
githubnext/ado-aw#1041 ·
Los mantenedores suelen responder en 2 días
-
security
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
githubnext/ado-aw#855 ·
Los mantenedores suelen responder en 2 días
Todos los issues de githubnext/ado-aw
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
area:sim good first issue kind:feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 87/100
Los mantenedores suelen responder en 1 día
-
The demo's interval button and chart legend keep the previous interval after an interval changeAbiertobug ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
userFRM/egui-charts#670 ·
Los mantenedores suelen responder en 1 día
-
docs good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 84/100
Usenova-Labs/usenova-backend#35 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
lugassawan/swe-workbench#823 ·
Los mantenedores suelen responder en 1 día