Configurations like timeout, proxy, and ssl_verify should be parameterised

Abierto
#885 1 comentario 0 reacciones 6 asignados Ver en GitHub

@mwwoda ya está trabajando en esto.

Desde el 2/12/2024.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

enhancement

Currently, in the Box SDK, configuring timeout, proxy, and SSL verify requires overriding the DefaultNetwork.request method.

class ProxyableBoxNetwork(DefaultNetwork):
    def __init__(self, proxy_conf, verify, logger=None):
        super(ProxyableBoxNetwork, self).__init__()

    def request(self, method, url, access_token, **kwargs):
        return super(ProxyableBoxNetwork, self).request(
            method, url, access_token,
            proxies=self._proxies,
            verify=self._verify,
            timeout=self._timeout,
            **kwargs
        )

Shouldn't these configurations be parameterised to allow straightforward setup without overriding?

Lenguaje dominante
Python
Estrellas
460
Forks
223
Merge medio
14 h 18 min
PR fusionados (30 d)
21

Guía de contribución

Abrir la guía de contribución

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 box/box-python-sdk

Todos los issues de box/box-python-sdk

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.