Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[BUG] AsyncTwilioHttpClient.request does not use self.timeout

Abierto Apto para principiantes
  • ¿Algo no está bien?
#925 0 comentarios 0 reacciones 0 asignados Ver en GitHub

@Hashim1999164 ya está trabajando en esto.

Desde el 16/8/2026.

  • #928 de @nileshpatil6 — cerrado sin fusionar
  • #955 de @Hashim1999164 — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
74/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
python
Área
api

Línea de trabajo

Comienza en el punto de entrada AsyncTwilioHttpClient.request mostrado en el issue y compara su gestión del timeout con TwilioHttpClient.request. Revisa las pruebas existentes para estos clientes antes de realizar el cambio. Se considera terminado cuando un timeout ausente usa self.timeout, mientras que los valores explícitos no positivos siguen generando ValueError, con cobertura para ambos casos.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

type: bug
Describe the bug

AsyncTwilioHttpClient.request never falls back to self.timeout if timeout is None

Code snippet
        if timeout is not None and timeout <= 0:
            raise ValueError(timeout)

        basic_auth = None
        if auth is not None:
            basic_auth = BasicAuth(login=auth[0], password=auth[1])

        kwargs = {
            "method": method.upper(),
            "url": url,
            "params": params,
            "data": data,
            "headers": headers,
            "auth": basic_auth,
            "timeout": timeout,
            "allow_redirects": allow_redirects,
        }

        self.log_request(kwargs)
        self._test_only_last_response = None

        temp = False
        session = None
        if self.session:
            session = self.session
        else:
            session = ClientSession()
            temp = True
        self._test_only_last_request = TwilioRequest(**kwargs)
        response = await session.request(**kwargs)
Actual behavior

response is passed "timeout": None in kwargs instead of "timeout": self.timeout when no timeout argument is passed

Expected behavior

response is passed "timeout": self.timeout in kwargs when no timeout argument is passed, this is what (sync) TwilioHttpClient does:

        if timeout is None:
            timeout = self.timeout
        elif timeout <= 0:
            raise ValueError(timeout)
twilio-python version

9.10.9

Python version

3

Logs or error messages

No response

Additional context

No response

Lenguaje dominante
Python
Estrellas
2.1k
Forks
814
Merge medio
1 d 14 h
PR fusionados (30 d)
4

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de twilio/twilio-python

Todos los issues de twilio/twilio-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.