Skip to content

Conversation

@realeatham
Copy link
Contributor

No description provided.

@tomasz1986
Copy link
Member

tomasz1986 commented Aug 14, 2025

Thanks for the update! Please just change the line breaks. The original text was written with up to 75 characters per line, so after the update, it should be something like:

``numConnections`` is a device setting that affects connection handling. A
zero value means to use the Syncthing default. As of version 2.0 the
default is to use three connections: one for index metadata and two for
data exchange. This may change in the future.

@tomasz1986 tomasz1986 changed the title Update device-numconnections.rst for st 2.0 as it allows 3 connections by default chore(advanced): update default number of connections to 3 as of Syncthing v2 Aug 14, 2025
@calmh
Copy link
Member

calmh commented Aug 14, 2025

Frankly, let's just remove the sentence entirely. I know I wrote this from the beginning, but really I'm convinced it's a bad idea to codify default values in the documentation as they tend to change...

@realeatham
Copy link
Contributor Author

Frankly, let's just remove the sentence entirely. I know I wrote this from the beginning, but really I'm convinced it's a bad idea to codify default values in the documentation as they tend to change...

Ok, i've done that.

@calmh calmh changed the title chore(advanced): update default number of connections to 3 as of Syncthing v2 chore(advanced): remove outdated note on default number of connections Aug 14, 2025
@calmh calmh merged commit e9a2971 into syncthing:main Aug 14, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants