rattail/rattail-nationbuilder
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Should actually use requests session for web api...

Browse source
This commit is contained in:
Lance Edgar 2023-05-25 21:36:35 -05:00
parent baadb86d5d
commit ebf1e6eb60
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
rattail_nationbuilder/nationbuilder/webapi.py
View file

@ -38,7 +38,6 @@ class NationBuilderWebAPI(object):
max_retries=None, **kwargs): max_retries=None, **kwargs):
self.config = config self.config = config
self.app = self.config.get_app() self.app = self.config.get_app()
self.session = None
self.base_url = base_url or self.config.require( self.base_url = base_url or self.config.require(
'nationbuilder', 'api.base_url') 'nationbuilder', 'api.base_url')
@ -69,8 +68,8 @@ class NationBuilderWebAPI(object):
params['access_token'] = self.access_token params['access_token'] = self.access_token
if request_method == 'GET': if request_method == 'GET':
response = requests.get('{}/{}'.format(self.base_url, api_method), response = self.session.get('{}/{}'.format(self.base_url, api_method),
params=params) params=params)
else: else:
raise NotImplementedError("unknown request method: {}".format( raise NotImplementedError("unknown request method: {}".format(