Skip to content

Commit 0d74800

Browse files
committed
api: _query(): don't sleep if there's no retries to be made.
1 parent ca9a54a commit 0d74800

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

krakenex/api.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ def __init__(self, key='', secret=''):
6969
self.response = None
7070

7171
# retry-on-failure configuration
72-
self.retries = 1
72+
self.retries = 1 # FIXME: retries==1 does 0 retries!
7373
self.cooldown = 15
7474
self.successcodes = [200, 201, 202]
7575
self.retrycodes = [504, 520]
@@ -129,6 +129,7 @@ def _query(self, urlpath, data, headers=None):
129129

130130
attempt = 0
131131
while attempt < self.retries:
132+
attempt += 1
132133
nonce = -1 if 'nonce' not in data.keys() else data['nonce'] # UGLY
133134
logger.debug('Posting query: nonce %d, attempt %d.', nonce, attempt)
134135
self.response = self.session.post(url, data = data, headers = headers)
@@ -140,7 +141,7 @@ def _query(self, urlpath, data, headers=None):
140141
logger.debug('HTTP error %d', status)
141142
logger.debug('Sleeping for %d seconds', self.cooldown)
142143
time.sleep(self.cooldown)
143-
attempt += 1
144+
continue
144145
else:
145146
self.response.raise_for_status()
146147

0 commit comments

Comments
 (0)