552628 - [Autoconfiguration] Add support for email.it mail account

Status: RESOLVED FIXED

(Webtools :: ISPDB Server, defect)

Description

[[:Aureliano Buendía]]

http://ispdb.mozillamessaging.com/details/270

Comment 1

[[:Aureliano Buendía]]

Attachment: email.it pop3 *.xml

Comment 2

[[:Aureliano Buendía]]

Probably I make a little error: 

1. where is the right value of "Incoming authentication" when on account settings, server settings option "security settings-->use secure authentication" is unchecked between

<element name="authentication">
   <choice>
      <value></value>
      <value>plain</value>
      <value>secure</value>
   </choice>
</element>

and for 

Outgoing authentication also when "use secure authentication" is unchecked between 

<element name="authentication">
    <choice>
       <value></value>
       <value>plain</value>
       <value>secure</value>
       <value>smtp-after-pop</value>
    </choice>
</element>

I suppose that the right choice is <value></value> or not?

Comment 3

[[:Aureliano Buendía]]

Attachment: email.it pop3 *.xml v2

Changed <authentication></authentication> according to comment #2

Comment 4

[[:Aureliano Buendía]]

Attachment: email.it.pop3 (v3)

Fix issue on <username> value.

Comment 5

[[:Aureliano Buendía]]

comment #2 and comment #3 are obsolete

Comment 6

[[:Aureliano Buendía]]

Attachment: email.it.imap (v1)

add imap configuration for review

Comment 7

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 433121 [details] [diff] [review]
email.it.pop3 (v3)

>      <incomingServer type="pop3">
>         <hostname>popmail.email.it</hostname>
>         <port>993</port>

Shouldn't that be port 995?

>         <socketType>SSL</socketType>
>         <username>%EMAILADDRESS%</username>
>         <authentication>plain</authentication>
>      </incomingServer>

And since the name of the file has to be "email.it", I'ld really like to see the POP3 incomingServer as a commented-out block inside of the IMAP config file.

Thanks,
Blake.

Comment 8

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 433125 [details]
email.it.imap (v1)

>      <outgoingServer type="smtp">
>         <hostname>smtp.email.it</hostname>
>         <port>25</port>

It looks like smtp.email.it responds on port 587, which we prefer to use over port 25.

>         <socketType>plain</socketType>
>         <username>%EMAILADDRESS%</username>
>         <authentication>plain</authentication>

smtp.email.it also advertises STARTTLS, so sending the user's login and password over the internet in cleartext is even worse than normal.  ;)

Please change the port, the socketType, and add the POP3 settings as a commented-out block of XML.

Thanks,
Blake.

Comment 9

[[:Aureliano Buendía]]

Attachment: email.it (v4)

Fixes based on :bwinton comments.

Comment 10

[[:Aureliano Buendía]]

Attachment: email.it (v5)

I forgot to comment <pop3> node. I'm sorry.

Comment 11

[[:Aureliano Buendía]]

Attachment: email.it (v6)

Comment 12

[[:Aureliano Buendía]]

Attachment: email.it (v7)

I'm tired tonight, I'm sorry blake :-D

Comment 13

[[:Aureliano Buendía]]

Blake, maybe now I have to ask only for second review to :BenBu ?

Comment 14

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 434304 [details]
email.it (v7)

(In reply to comment #12)
> Created an attachment (id=434304) [details]
> email.it (v7)
> 
> I'm tired tonight, I'm sorry blake :-D

Heh.  No worries.  I'm happy with it.  Over to Ben for the second review.

Later,
Blake.

Comment 15

[Ben Bucksch (:BenB)]

1.
+ <leaveMessagesOnServer>true</leaveMessagesOnServer>

lalala

2. Please add URL where the config is described.
3. <authentication>plain</authentication>
IMAP and POP servers support CRAM-MD5. Use it:
<authentication>secure</authentication>
Not for SMTP, though.

SMTP SSL 465 doesn't work, but IMAP and POP SSL work, and SMTP STARTTLS works, so that'd fine.

Comment 16

[Ben Bucksch (:BenB)]

4. Please put the pop server underneath the imap server, not inside it, and remove the tabs.

Comment 17

[[:Aureliano Buendía]]

Attachment: email.it (v8)

according to comment #15 and comment #16

Comment 18

[Ben Bucksch (:BenB)]

5. sorry, I missed this:

<displayName>email.it - The professional email</displayName><displayShortName>email.it</displayShortName>

No advertising space here. :)
Seems like they have no proper name, so maybe just:

<displayName>email.it</displayName>
<displayShortName>email.it</displayShortName>

?

6.
From the ISP page, I see that they apparently want us to set the "IMAP server directory" to "INBOX". (That's the only non-default value I could see on that screen.) Is that necessary? If so, can you make a note about that in the comment section, below the URL, please?

Comment 19

[Ben Bucksch (:BenB)]

7. Oh, critical: IMAP SSL port is wrong, it must be 993.

Comment 20

[[:Aureliano Buendía]]

Attachment: email.it (v9)

Fixes based on :BenBu comment #18 and comment #19

Comment 21

[Ben Bucksch (:BenB)]

I made 2 minor changes:
1. removed tabs and
2. changed "must" to "should" for the "INBOX" server directory pref

Commited as revision 64702.
FIXED