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

Discovery server connection test #4716

Open
calmh opened this issue Jan 28, 2018 · 3 comments
Open

Discovery server connection test #4716

calmh opened this issue Jan 28, 2018 · 3 comments
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug)

Comments

@calmh
Copy link
Member

calmh commented Jan 28, 2018

The discovery server could make connection tests to the announced addresses/ports and return the result to the client when it does the announcement. This could provide one small part of the "connection doctor" like functionality, in that Syncthing could display an indicator for success/failure.

In case of failure it could say something like Incoming connections on port 22000 blocked, check port forwards or firewall settings.

In case of success it can instead report the actual port used and that port forwarding works.

The test would have to be fairly "shallow" and use short timeouts, in order to not put too much extra work on the discovery servers. We need to plan on doing several hundred connection tests / second if we do this. We can skip checks on addresses that are reannounced and have succeeded within the last 24 hours or something.

@canton7
Copy link
Member

canton7 commented Jan 28, 2018

Alternatively, have a big "Troubleshoot connection issues" button, and just do the connection test when clicked. That should reduce the load a lot.

It also means you could hive the connection test off to another server if necessary.

@calmh
Copy link
Member Author

calmh commented Jan 28, 2018

True

@calmh calmh added the enhancement New features or improvements of some kind, as opposed to a problem (bug) label May 15, 2018
@seniorm0ment
Copy link

this would be nice on more complex networks to easily see what ports syncthing has access to and what it doesn't

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug)
Projects
None yet
Development

No branches or pull requests

3 participants