28099 - Need IMAP SSL support

Status: VERIFIED FIXED

(MailNews Core :: Networking, defect, P3)

Description

[Scott MacGregor]

We now have imap over ssl support. Pav did most of the work a while ago and
alecf hanlded the UI which is already checked into the tree.

It turned out there was a small layer of code I needed to write to detect if we
were already in the authenticated state when the connection is made over SSL.

With that piece of code, imap SSL is running like a champ. At least on my high
end machine I don't even notice much of a performance degradation.

Of course this comes a day late for general checkin. I would need to get PDT+,
beta approval in order to check it in now.

Sol, is this something you'd be interesting in taking for beta1? We could go
argue for it if so...

Comment 1

[Scott MacGregor]

Attachment: Here are the remaining changes we need to checkin to get this to work

Comment 2

[sol]

Scott - I didn't think you were going to have time for this, so it's not on the 
Beta 1 list. We would lose credibility if we tried to promote a feature that was 
not on that list to pdt+ status.

On the other hand, I don't want this work to be thrown away - that would be a 
huge waste. Can the work be "held" in a build outside the tree and checked in 
post B1? If not, or if bit-rot is likely, can you work with me and Phil to plead 
our case to the PDT gods :-)

In any event, this is a problem I'm happy to have.

Comment 3

[Stuart Parmenter]

Well, I've had this done for nearly two months now.  It broke a little while ago
and mscott just got around to fixing it.  Not having this blocks me (and others)
from being able to use mailnews in Mozilla.  This is very important to have in
so that I can check my mail.

Comment 4

[sol]

Pav: Not sure why this blocks you from using mailnews in mozilla - can you give
me the reason to strengthen the case as to why this should be PDT+

Comment 5

[Stuart Parmenter]

My mail comes from a server that only accepts imaps connections.

Comment 6

[sol]

Is this a high or low risk check in?

Comment 7

[Stuart Parmenter]

very low

Comment 8

[sol]

Sorry - do not pass go on this one. At least not for beta1. The rules are pretty 
clear about what should go in, and checking this in would break those rules.

Please check this into a development branch.

Comment 9

[Stuart Parmenter]

marking dogfood.

Comment 10

[Scott MacGregor]

I checked it in this afternoon. QA, to test, you need to have a certificate for
your account.

Then, edit your mail account settings and select the server. You'll see a check
box which says "use ssl". Check this box.

Quit the application and restart. You should now see a new icon next to the
server with a lock in it. This means that the server is secure.

Now try to open that server and select your inbox. You should be prompted by the
security manager for a certificate to use. Select your certificate and hit okay.

You should then see your mail get downloaded and be able to read it.

Many thanks to Pavlov for doing most of the work for this feature. Thanks!

Comment 11

[Judson Valeski]

thanks for fixing this. I haven't been able to use 5.0 mail because of this. My
mail server requires cert auth. THANKS for the dogfood.

Comment 12

[Karen Huang]

Verified this bug along with the verification of bug 39154.