123069 - Mozilla ignores "encrypt always" preference!

Status: VERIFIED FIXED

(MailNews Core :: Security: S/MIME, defect, P1)

Description

[Kai Engert (:KaiE:)]

Make sure you have certificates and mailnews configured.
Configure to always sign and always encrypt.
Close and restart Mozilla.
Compose a message.
Enter a recipient subject and body text.
Do NOT open the security menu. Just send the message.

Actual behaviour: Message is NOT sent encrypted.

Could have to do with my NSS 3.4 build.

Comment 1

[Stephane Saux]

Let me try we my build (a bit stale - 2 days).

Comment 2

[Antonio Lam]

Tried with mozilla-win32-nss34-20020128-115010.zip, and I am not able to send 
the message.  It complained it can't find the recipient encrypting cert.  (I am 
sure I have the recipient cert in "Other People's" and it is verified to have 
the following usage: Email Recipient Certificate)

Comment 3

[Stephane Saux]

Antonio. You may have hit the bug whereby you must have sent yourself a signed
email in order to be able to send an encrypted email where you are on the
recipient list.

Comment 4

[Antonio Lam]

For my last comment #2, I didn't send the message to myself, but to other 
recipient.

But anyway, I am still hitting the same problem described in bug 122933 using 
the latest NSS 3.4 build (20020201).

Comment 5

[Kai Engert (:KaiE:)]

I retested with my latest build. I still see this bug as originally described.

Encryption is not used unless you manually open the Options/Security menu in
composer before sending.

Comment 6

[Kai Engert (:KaiE:)]

Attachment: Suggested fix

In case the menu has not been used, we query the wrong preference...

Comment 7

[Kai Engert (:KaiE:)]

Javi, can you please review?

Comment 8

[Stephane Saux]

S/MIME bugs are automatically nsbeta1 candidates. (this is a bulk update - there
may be some adjustment of the list).

Comment 9

[Javier Delgadillo]

Comment on attachment 68667 [details] [diff] [review]
Suggested fix

r=javi

Comment 10

[Kai Engert (:KaiE:)]

Scott, can you please sr ?

Comment 11

[Scott MacGregor]

Comment on attachment 68667 [details] [diff] [review]
Suggested fix

sr=mscott

Comment 12

[Asa Dotzler [:asa]]

Comment on attachment 68667 [details] [diff] [review]
Suggested fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 13

[Kai Engert (:KaiE:)]

checked in, fixed.

Comment 14

[John Unruh]

Verified fixed.