Skip to content

[wip] adding link2xt's dc_config.txt -- listing TODOs for configure issues … #1832

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

Closed
wants to merge 1 commit into from

Conversation

hpk42
Copy link
Contributor

@hpk42 hpk42 commented Aug 14, 2020

…found from user testing.

Comment on lines +88 to +101
9. configure: connection refused error suggests to check the inbox, even if IMAP can't connect

If connection is refused, it does not make sense to suggest checking
the inbox. Especially if IMAP failed or domain can't be resolved.

Current error message is too broad and usually irrelevant and not helpful:

Some providers place additional information in your inbox; you can
check them eg. in the web frontend. Consult your provider or friends
if you run into problems.

Which providers place information about failed login attempts into
inbox? Is it helpful or just "someone from ... tried to login with
incorrect password"?
Copy link
Collaborator

Choose a reason for hiding this comment

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

GMX and Yahoo do send you a helpful error email IIRC. However, the provider-db also shows this information, so it is actually not needed.

Moreover, many users will probably not understand "Some providers place additional information in your inbox; you can check them eg. in the web frontend." And "Consult your provider or friends if you run into problems" is a nice advice but many users will ask their techie friends anyway and the provider will probably not help.

Together with point 8 we could do:
IMAP login failed.
Trying example.org ... port closed
Trying imap.example.org ... could not resolve address
Trying mail.example.org ... incorrect login

(and if it seems to be the password, tell the user to check the pw, as we already do)

@link2xt
Copy link
Collaborator

link2xt commented Aug 15, 2020

(4) is not correct, provider database strict_tls is used in imap/mod.rs and smtp/mod.rs, nothing to fix there.

Incorrect, in fact I just fixed it in #1835. In current master, the problem is here for IMAP:
https://github.com/deltachat/deltachat-core-rust/blob/0bf8017e8ff13b0e03512305ea589b407af86cd8/src/configure/mod.rs#L379
and here for SMTP:
https://github.com/deltachat/deltachat-core-rust/blob/0bf8017e8ff13b0e03512305ea589b407af86cd8/src/configure/mod.rs#L388

@link2xt
Copy link
Collaborator

link2xt commented Aug 23, 2020

Some of the issues are already fixed, some I have moved to https://github.com/orgs/deltachat/projects/26

I keep track of all of them locally too, so none of them will be lost.

@link2xt link2xt closed this Aug 23, 2020
@link2xt link2xt deleted the config_issues branch August 23, 2020 19:14
@r10s
Copy link
Contributor

r10s commented Aug 24, 2020

thanks for taking care!

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.

4 participants