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

Put relay setting in main settings #2433

Closed
uok opened this issue Nov 6, 2015 · 6 comments
Closed

Put relay setting in main settings #2433

uok opened this issue Nov 6, 2015 · 6 comments
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion

Comments

@uok
Copy link
Contributor

uok commented Nov 6, 2015

Since this is a really great feature I suggest to put it into the main settings like this

newsettings

@AudriusButkevicius
Copy link
Member

I am refactoring connection, which means a relay will become just a listen address. If you are not listening on a relay, they are disabled. Though that is probably for v0.13 as its a protocol change again.

@uok
Copy link
Contributor Author

uok commented Nov 6, 2015

Ok, I just wanted to find a way to quickly enable/disable relaying, e.g. at home I don't need relaying

@calmh
Copy link
Member

calmh commented Nov 6, 2015

Yeah in the meantime we should expose this as a normal setting I think. People will legitimately want to disable it.

@calmh calmh added the enhancement New features or improvements of some kind, as opposed to a problem (bug) label Nov 6, 2015
@calmh calmh added this to the v0.13 "Copper Cockroach" milestone Nov 22, 2015
@Eddy2909
Copy link

why is relaying enabled per default?
isnt this a feature only for users who have connection problems (thru firewall etc)?

@AudriusButkevicius
Copy link
Member

It's only used when no direct connection is possible.

@canton7
Copy link
Member

canton7 commented Dec 16, 2015

The people who need a relay are those who probably won't know that relaying exists: i.e. the most technologically incompetent people who are new to Syncthing, and haven't yet invested the time to learn how it works and what you can do with it.

It needs to be enabled by default.

@calmh calmh closed this as completed in 1df924f Jan 5, 2016
AudriusButkevicius added a commit that referenced this issue Jan 5, 2016
Add relaying to main settings dialog (fixes #2433)
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Jun 16, 2017
@syncthing syncthing locked and limited conversation to collaborators Jun 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Development

No branches or pull requests

6 participants