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

CP-2231 CP-2107 Update autoRetry docs to clarify its expectation of a failed reqeust #199

Open
evanweible-wf opened this issue Jul 25, 2016 · 1 comment
Open

CP-2231 CP-2107 Update autoRetry docs to clarify its expectation of a failed reqeust #199

evanweible-wf opened this issue Jul 25, 2016 · 1 comment
Labels
area-docs
Milestone
3.1.0

Comments

@evanweible-wf
Copy link
Contributor

https://github.com/Workiva/w_transport#automatic-request-retrying

Currently, it could be interpreted that any status code added to autoRetry.forStatusCodes will be retried. In reality, this only applies to failed requests, so any status code in the 2xx range would have no effect in that list.
@evanweible-wf evanweible-wf added this to the 3.1.0 milestone Jul 25, 2016
@rmconsole3-wf
Copy link
Contributor

It looks like you have not included any tickets in the title of this issue. Please create a ticket for this issue, or click here to have Rosie create one for you.

@evanweible-wf evanweible-wf mentioned this issue Jul 25, 2016
Closed
@rmconsole3-wf rmconsole3-wf changed the title Update autoRetry docs to clarify its expectation of a failed reqeust CP-2107 Update autoRetry docs to clarify its expectation of a failed reqeust Jul 28, 2016
@charliekump-wf charliekump-wf changed the title CP-2107 Update autoRetry docs to clarify its expectation of a failed reqeust CP-2231 CP-2107 Update autoRetry docs to clarify its expectation of a failed reqeust Jul 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-docs
Projects
None yet
Milestone
3.1.0
Development

No branches or pull requests
2 participants
@evanweible-wf @rmconsole3-wf