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

Present a user-friendly error-message when a Tor endpoint is enabled, but Tor is not installed #438

Open
ELLIOTTCABLE opened this issue Sep 7, 2021 · 3 comments

Comments

@ELLIOTTCABLE
Copy link

Title pretty much says it all: at least on my machine, without Tor installed, selecting the 'Tor version 3 address (proton-mail)' API entry point, predictably, causes you to be unable to login.

However, instead of any sort of error-message or any indication whatsoever why you can't login, the webview just … freezes, offering no feedback, and disallowing the user from logging in.

I'd suggest adding a network-timeout and coherent error-message to the login-submission process!

(Beginners following the setup flow are presented "Tor" as an option without necessarily knowing what it is, or that they need to install it separately — a related-but-slightly-different task may be 'improve the onboarding process for Tor-curious newbies overall.')

@vladimiry
Copy link
Owner

The readme/project-front-page includes the #113 (comment) link, see the following bullet:

  • 📫 Multi accounts support including supporting individual API entry points. For example, you can force the specific email account added in the app connect to the email provider via the Tor only by selecting the Tor version 3 address API entry point in the dropdown list and configuring a proxy as described in this message.

That's Windows-specific but could also be translated to macOS by advanced users. Of course I agree that it's not enough in terms of UX, ie the UX could be improved although I generally believe that those who use Tor normally understand what that thing is.

@ELLIOTTCABLE
Copy link
Author

Yep! As with all these issues i'm opening, these are low-ish priority, polish, user-experience things. Hopefully such are welcome in your triage process. <3

@vladimiry
Copy link
Owner

vladimiry commented Sep 7, 2021

Hopefully such are welcome in your triage process. <3

Yep, I like to paint the labels, not the same thing as handling the actual issue though.

low-ish priority, polish, user-experience things.

I don't normally specify the priority here, as, you know, "not a product". Unless this is not a severe crash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants