The Wayback Machine - https://web.archive.org/web/20220411052111/https://github.com/imsnif/bandwhich/issues/127
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

Jump to bottom

Retry DNS only after timeout #127

Open
imsnif opened this issue Jan 16, 2020 · 4 comments
Open

Retry DNS only after timeout #127

imsnif opened this issue Jan 16, 2020 · 4 comments
Labels
good first issue help wanted

Comments

@imsnif
Copy link
Owner

@imsnif imsnif commented Jan 16, 2020

Right now, when we do our reverse-dns queries (getting the hostname of an ip address), if the request failed, we retry immediately. This can cause a lot of connections (also visible in the app!) to be opened if a request repeatedly fails.

It would be nice if we have some sort of progressive backoff mechanism for this... something like: wait for 1 second before retrying, then wait for 2, 4, etc.
@imsnif imsnif added help wanted good first issue labels Jan 16, 2020
@zhangxp1998
Copy link
Collaborator

@zhangxp1998 zhangxp1998 commented Jan 17, 2020

Correct me if I'm wrong, currently we will retry every second?
@imsnif
Copy link
Owner Author

@imsnif imsnif commented Jan 17, 2020

Yep. I have a bit of a spotty internet connection though, so sometimes my whole connections pane is filled with our own DNS requests for minutes on end (which we somehow classify as unknown, btw). I thought this can make things a little friendlier.
bigtoast added a commit to bigtoast/bandwhich that referenced this issue Jan 17, 2020
@bigtoast
(imsnif#127) Degrading retry DNS resolve. Needs tests
ac4e4bc
bigtoast added a commit to bigtoast/bandwhich that referenced this issue Jan 18, 2020
@bigtoast
(imsnif#127) Degrading retry DNS resolve. Needs tests
669e54b
@captain-yossarian
Copy link
Contributor

@captain-yossarian captain-yossarian commented Mar 2, 2020

Hi @imsnif I would like to work on it. Could you please give me start point?
@imsnif
Copy link
Owner Author

@imsnif imsnif commented Mar 2, 2020

@bigtoast is already working on it here: #133
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue help wanted
3 participants
@imsnif @zhangxp1998 @captain-yossarian