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.

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

Authorizer should only try x number of times otherwise we get stuck in recursion #5

Open
SvenW opened this issue Apr 25, 2014 · 3 comments

Comments

@SvenW
Copy link
Collaborator

SvenW commented Apr 25, 2014

No description provided.

@zond
Copy link
Owner

zond commented Apr 25, 2014

Or just use an exponential backoff for all errors, including auth errors?

On Fri, Apr 25, 2014 at 11:45 AM, SvenW notifications@github.com wrote:


Reply to this email directly or view it on GitHubhttps://github.com//issues/5
.

@jimmyw
Copy link
Collaborator

jimmyw commented Apr 25, 2014

I Think this is best don cliemt-side as it is thé client that initiates thé request. Exponential back off seems nice, if the error is server side the bus will start working automatically when the error is resolved..

@SvenW
Copy link
Collaborator Author

SvenW commented Apr 25, 2014

It should definitely be done client side, just had two write the issue here since I can't do it in my fork :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants