Skip to content
This repository was archived by the owner on Apr 21, 2022. It is now read-only.

HTTP Exception Handling#62

Merged
vilyapilya merged 7 commits intomasterfrom
dontReconnect
Aug 9, 2019
Merged

HTTP Exception Handling#62
vilyapilya merged 7 commits intomasterfrom
dontReconnect

Conversation

@vilyapilya
Copy link
Copy Markdown
Contributor

If logger receives an unsuccessful response, it will bring the message back to the buffer and will schedule the next incoming message to be sent in back_off period instead of sending it in the regular period.

Once the the back off period passes, the message will be sent and the backoff flag switched off. It also makes sure that the backoff flag switched back to off when a successful res is received.

This PR also makes changes in the test as the tests run in parallel, and using the same servers and global result placeholders can produce unreliable results

@smusali smusali changed the title HTTP Exception Handling with saving the messages in buf HTTP Exception Handling Aug 8, 2019
@smusali smusali requested review from mikehu and smusali August 8, 2019 14:34
Copy link
Copy Markdown
Contributor

@smusali smusali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add/update documentation for this HTTP Exception and Add/Remove Property (Ambient Meta) stuff? If you want, you can do it in separate branch/PR

@smusali smusali self-requested a review August 8, 2019 20:28
@vilyapilya vilyapilya merged commit bfc608c into master Aug 9, 2019
@vilyapilya vilyapilya deleted the dontReconnect branch August 9, 2019 23:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants