DoS on wrong password
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 1.7k
- Forks
- 408
- Merge medio
- 2 d 57 min
- PR fusionados (30 d)
- 7
Descripción
As discussed in upstream bug report saltstack/salt#38230 pygit2 tries to authenticate indefinite with wrong credentials instead of failing after two consecutive failures with same authentication token as specified by https://tools.ietf.org/html/rfc7235#section-3.1
Possible a libgit2 Bug if it's reproducable with other language bindings.
Testcase:
nginx dummy server configuration
This is just a stub configuration, no real repository needed
location ~ /git(/.*) {
add_header "WWW-Authenticate" "Basic realm=\"Closed Areas\"" always;
return 401;
}
test client
#!/bin/python2.7
import pygit2
username = 'dummyUser'
password = 'dummyPassword'
credentials = pygit2.UserPass(username, password)
callbacks = pygit2.RemoteCallbacks(credentials)
pygit2.clone_repository("http://git.demo.local/git/test.git", "test.git", bare=True, callbacks=callbacks)
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.
Línea de trabajo
Comienza con el caso de prueba de Python usando pygit2.UserPass, pygit2.RemoteCallbacks y pygit2.clone_repository contra la configuración de nginx proporcionada. Reproduce los intentos de autenticación repetidos y compara el comportamiento con RFC 7235 sección 3.1. Se considera terminado cuando el clonado falla después de dos fallos consecutivos con el mismo token de autenticación, y se confirma si el comportamiento es específico de pygit2 o de libgit2.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- git, python
- Área
- authentication, networking, security
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100