search

Martchus / syncthingtray

Tray application and Dolphin/Plasma integration for Syncthing
https://martchus.github.io/syncthingtray/
Other
1.54k stars 43 forks source link

Make clear that "Syncthing URL" needs to start with `https://` #172

Closed TheJoeSchr closed 1 year ago

TheJoeSchr commented 1 year ago

I was lost for hours why this wasn't working :facepalm:

Martchus commented 1 year ago

Or http:// if you don't use TLS. I'm wondering how you tried to start the URL.

Note that the wizard should help you configure things automatically. The config dialog also allows to fill out the connection settings automatically from the Syncthing config. Did this not work for you? (Of course if you want to connect to a Syncthing instance on a different host then this is clearly not clearly not useful.)

I suppose I could add "https://" as placeholder text but that would be misleading in case Syncthing does not use TLS as it then needed to be "http://".

TheJoeSchr commented 1 year ago

I somehow never got to see the wizard until way later when I stumbled into it.

I guess I must have updated, restarted a bunch of times (remotely via ssh) before I was sitting in front of the computer again.

Once I was in the wizard, it correctly filled it out. But I didn't know there was a wizard until I found it

__

Typed on a mobile, please excuse any typing mistakes or short-spokeness.

On Fri, 27 Jan 2023, 22:38 Martchus, @.***> wrote:

Or http:// if you don't use TLS. I'm wondering how you tried to start the URL.

Note that the wizard should help you configure things automatically. The config dialog also allows to fill out the connection settings automatically from the Syncthing config. Did this not work for you? (Of course if you want to connect to a Syncthing instance on a different host then this is clearly not clearly not useful.)

I suppose I could add "https://" as placeholder text but that would be misleading in case Syncthing does not use TLS as it then needed to be "http://".

— Reply to this email directly, view it on GitHub https://github.com/Martchus/syncthingtray/issues/172#issuecomment-1407099778, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6WSHV5KHX7HZ54EGVN4YDWUQ56FANCNFSM6AAAAAAUJFZ3CI . You are receiving this because you authored the thread.Message ID: @.***>

TheJoeSchr commented 1 year ago

The error message said something about protocol "" not found which makes sense to me now, but not then. Maybe this would be a good point to mention URL should contain sth like http[s]

__

Typed on a mobile, please excuse any typing mistakes or short-spokeness.

On Fri, 27 Jan 2023, 22:38 Martchus, @.***> wrote:

Or http:// if you don't use TLS. I'm wondering how you tried to start the URL.

Note that the wizard should help you configure things automatically. The config dialog also allows to fill out the connection settings automatically from the Syncthing config. Did this not work for you? (Of course if you want to connect to a Syncthing instance on a different host then this is clearly not clearly not useful.)

I suppose I could add "https://" as placeholder text but that would be misleading in case Syncthing does not use TLS as it then needed to be "http://".

— Reply to this email directly, view it on GitHub https://github.com/Martchus/syncthingtray/issues/172#issuecomment-1407099778, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6WSHV5KHX7HZ54EGVN4YDWUQ56FANCNFSM6AAAAAAUJFZ3CI . You are receiving this because you authored the thread.Message ID: @.***>

Martchus commented 1 year ago

I have added explaining placeholder texts, see referencing commit message for details. This should be good enough.

Good to know that the wizard generally worked. Maybe you've just (accidentally) dismissed it when setting things up.

With that I suppose this issue can be considered resolved.

TheJoeSchr commented 1 year ago

Thanks, good call!