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

Retry an initial channel produce failure #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jbarlow-mcafee
Copy link
Contributor

Previously, if a call to the channel produce method were to fail due
to the channel token having expired, no attempt was made to
automatically re-authenticate, attempt to obtain a new token, and
repeat the call to the produce method.

With the changes in this commit, an initial failure in a call to the
produce method will be retried, with an attempt made to
re-authenticate and obtain a new token before the second attempt to
perform the produce call. An error is delivered to the produce
callback if consecutive calls fail.

Previously, if a call to the channel `produce` method were to fail due
to the channel token having expired, no attempt was made to
automatically re-authenticate, attempt to obtain a new token, and
repeat the call to the `produce` method.

With the changes in this commit, an initial failure in a call to the
`produce` method will be retried, with an attempt made to
re-authenticate and obtain a new token before the second attempt to
perform the `produce` call. An error is delivered to the `produce`
callback if consecutive calls fail.
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

Successfully merging this pull request may close these issues.

None yet

1 participant