fix(ci): merge-base test fixture trips GitGuardian on every PR that merges main
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 77/100
Línea de trabajo
Empieza con .github/scripts/merge-base/post.test.sh en torno a las dos URL de fixtures cerca de las líneas 330 y 360; después, revisa .gitguardian.yaml si estás considerando la opción ignore-path. Reproduce o inspecciona el hallazgo de GitGuardian para el incidente 38106155 y ajusta el fixture o la configuración para que ya no active el detector. Se considera terminado cuando los PR que fusionen main ya no generen este incidente del fixture y el incidente del dashboard esté marcado como falso positivo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
GitGuardian reports "Basic Auth String" incidents on .github/scripts/merge-base/post.test.sh (added in #7190). The fixture sets a remote URL of the form https://x-access-token:<placeholder>@github.com/o/r at two places, around lines 330 and 360. The placeholder carries a GitHub token prefix but is not a real token: it is far shorter than a real one.
GitGuardian scans the commits in a PR, so every PR that merges main after #7190 picks up the incident and shows a failing check. It happened on #7204, #7206 and #7208. The check is not required, but each occurrence costs a manual triage.
Fix
Rewrite the fixture so it does not match the detector, for example with a value that has no token prefix or a credential built at runtime. Alternatively, add the path to the GitGuardian config (.gitguardian.yaml paths-ignore) as a test fixture. Then mark incident 38106155 as a false positive in the dashboard.
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 7 h 42 min
- PR fusionados (30 d)
- 923
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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 melodic-software/claude-code-plugins
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#7351 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: high
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
melodic-software/claude-code-plugins#7312 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7288 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdAbiertoneeds-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#7286 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#7282 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
[Bug]: Three default prefs with JSON values are never set (override.sh escapes their quotes twice)Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Floorp-Projects/Floorp#2913 ·
Los mantenedores suelen responder en 1 día
-
darwin zsh
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
carapace-sh/carapace-bin#3846 ·
Los mantenedores suelen responder en 1 día
-
devtools: new version 1.5.3Abiertoout-of-date
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
CachyOS/CachyOS-PKGBUILDS#1981 ·
Los mantenedores suelen responder en 1 día
-
area: backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
snapotter-hq/SnapOtter#2415 ·
Los mantenedores suelen responder en 1 día