Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WIP: Retry-on-failure using NIH approach #100

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

WIP: Retry-on-failure using NIH approach #100

wants to merge 11 commits into from

Commits on Sep 13, 2018

  1. Implementing simple mechanism for retrying failed requests

    @gzhytar @veox
    gzhytar authored and veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    3f48192 View commit details
    Browse the repository at this point in the history

  2. Connection: Adding an attribute to configure a number of connection r… 

    …ecovery attempts
    @gzhytar @veox
    gzhytar authored and veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    756fcb7 View commit details
    Browse the repository at this point in the history

  3. api: rework _query() retries code with suggestions from PR #65.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    afd6eb6 View commit details
    Browse the repository at this point in the history

  4. api: _query(): set nonce to -1 (not used) if query public.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    e12f94b View commit details
    Browse the repository at this point in the history

  5. api: add FIXME - getting logger for root ATM.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    de1c43d View commit details
    Browse the repository at this point in the history

  6. api: _query(): don't sleep if there's no retries to be made.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    beb873f View commit details
    Browse the repository at this point in the history

  7. api: retries=0 will attempt once. 

    Thanks to /u/FBAThrow for reminding!

https://www.reddit.com/r/krakenex/comments/778uvh/psa_http_error_502_does_not_mean_the_query_wont/dqwlc1a/
    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    1359536 View commit details
    Browse the repository at this point in the history

  8. misc: add CHANGELOG stub.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    b913d8d View commit details
    Browse the repository at this point in the history

  9. Merge branch 'master' into retry-on-failure (fixing conflicts) 

    Conflicts (resolved)
* CHANGELOG.rst
* krakenex/api.py
    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    f7bc08b View commit details
    Browse the repository at this point in the history

  10. api: minor, clean up whitespace.

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    7b93f3c View commit details
    Browse the repository at this point in the history

  11. api: give logger a name ('krakenex.api').

    @veox
    veox committed Sep 13, 2018
    Configuration menu
    Copy the full SHA
    9aabdaf View commit details
    Browse the repository at this point in the history