box / box/box-python-sdk

Configurations like timeout, proxy, and ssl_verify should be parameterised

Ouverte
#885 1 commentaire 0 réactions 6 personnes assignées Réclamée par @mwwoda Voir sur GitHub
enhancement
Langage dominant
Python
Étoiles
459
Forks
223
Merge moyen
8 h 57 min
PR mergées (30 j)
13

Description

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?

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.