Check for operating repos doesn't normalize to lowercase
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 8
- Forks
- 6
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
I just installed this for a repo with an organization whose name contains mixed case. I did not think to mix the case when I provided the GITHUB_REPO_NAME environment variable. This cased a 403 error with an error of {"message":"Not configured for this repository"}. I updated the organization's name in the environment variable to match the case as found in the payload and then the webhook succeeded.
Recommend normalizing both repoSplit[0] and checkSplit[0] to lowercase for comparison.
https://github.com/helm/pull-sizer/blob/bb557b11dfb698c211d6a9dd70935c93a97642ee/main.go#L176
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.
Línea de trabajo
Empieza en main.go alrededor de la línea 176, donde se comparan los nombres de los repositorios, y revisa la ruta de código relacionada con la comprobación del webhook. Verifica que los nombres de las organizaciones con mayúsculas y minúsculas combinadas funcionen incluso cuando GITHUB_REPO_NAME use una combinación de mayúsculas y minúsculas diferente; se considera terminado cuando el repositorio se acepta sin el error 403.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- go
- Área
- backend
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 55/100