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

advanced: Number of connections #803

Merged
merged 4 commits into from
Sep 7, 2023
Merged

advanced: Number of connections #803

merged 4 commits into from
Sep 7, 2023

Conversation

calmh
Copy link
Member

@calmh calmh commented Jun 1, 2023

For syncthing/syncthing#8918. (We should add a link in the GUI as well, since this setting is not only available through the advanced interface.)

Syncthing will, however, only keep connections with the best priority; by
default, TCP has better priority than QUIC, so establishing a TCP connection
will cause existing QUIC connections to be closed. Connection priorities can
be configured.
Copy link
Member

Choose a reason for hiding this comment

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

If its the same priority, can you have mixed transports? If no, why not. If yes, how is that represented in the UI?

Copy link
Member Author

Choose a reason for hiding this comment

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

You can indeed, and it isn't really represented in the GUI at all. Currently it shows the info about the primary (oldest) connection, and a count of additional connections, but no details on the additionals. It's available in the REST API though, so it could be represented if we wanted to.

@calmh calmh marked this pull request as ready for review September 6, 2023 11:05
@calmh calmh changed the title advanced: Multiple connections advanced: Number of connections Sep 6, 2023
@tomasz1986
Copy link
Contributor

tomasz1986 commented Sep 6, 2023

This may be a material for a different PR, but is there something to add to https://docs.syncthing.net/users/tuning regarding the connection number? I'm asking because the commit message specifically mentions possible tweaks for low-end hardware already (see syncthing/syncthing@c6334e6).

@calmh
Copy link
Member Author

calmh commented Sep 6, 2023

I can add some words

@calmh calmh merged commit 4d205e5 into main Sep 7, 2023
5 checks passed
@calmh calmh deleted the multicon branch September 7, 2023 05:53
@syncthing syncthing locked and limited conversation to collaborators Sep 6, 2024
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.

5 participants