AnyLog-co / AnyLog-co/AnyLog-API

timeout fix

Abierto
#10 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
0
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

so there's 2/2 parts to the issue:
in the remote-gui, we need to add an option to update the timeout and update the class variables accordingly
timeout needs to be broken into 2 parts connections_timeout and read_tieout

Code to be updated::
[AnyLog-API/anylog_api/anylog_connector.py at main · AnyLog-co/AnyLog-API](https://github.com/AnyLog-co/AnyLog-API/blob/main/anylog_api/anylog_connector.py)
```
try:
response = requests.get(f'http://{self.conn}', headers=headers, auth=self.auth, timeout=self.timeout)
except Exception as e:
error = str(e)
response = False
else:
if int(response.status_code) < 200 or int(response.status_code) > 299:
error = int(response.status_code)
response = False
Expect:
try:
response = requests.get(f'http://{self.conn}', headers=headers, auth=self.auth, timeout=(connection_timeout, read_timeout)
except Exception as e:
error = str(e)
response = False
else:
if int(response.status_code) < 200 or int(response.status_code) > 299:
error = int(response.status_code)
response = False
[anylog_connector.py](https://github.com/AnyLog-co/AnyLog-API/blob/main/anylog_api/anylog_connector.py)

both the connection_timeout and read_timeout should be user configurable via the remote-gui

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.