box / box/box-python-sdk

Configurations like timeout, proxy, and ssl_verify should be parameterised

Offen
#885 1 Kommentar 0 Reaktionen 6 zugewiesene Personen Beansprucht von @mwwoda Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Python
Sterne
459
Forks
223
Ø Merge
8 Std. 57 Min.
Gemergte PRs (30 T.)
13

Beschreibung

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?

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.