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

Exponential backoffs for all connection attempts #144

Open
realcr opened this issue Jan 1, 2019 · 0 comments
Open

Exponential backoffs for all connection attempts #144

realcr opened this issue Jan 1, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@realcr
Copy link
Member

realcr commented Jan 1, 2019

There are some parts in offst that attempt connections:

  • Relay client
  • Channeler
  • Index client

Currently each of those have a very basic backoff mechanism for reconnection after a failed connection attempt. This might cause DoS on a server.

We should change the backoff mechanism to exponential backoff, and find reasonable ways to test the new exponential backoff logic. Maybe there is a way to isolate this logic and reuse it in all clients that connect to servers in offst.

@realcr realcr added the enhancement New feature or request label Jan 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant