digital signature preferences won't stick

VERIFIED FIXED

Status

SeaMonkey
MailNews: Account Configuration
--
major
VERIFIED FIXED
16 years ago
13 years ago

People

(Reporter: Patrick, Assigned: racham)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

16 years ago
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:0.9.7+)
Gecko/20020101
BuildID:    2002010103

A while ago we had a bug that MESSAGE COPIES wouldn't stick for IMAP accounts.
Now i have the same problems with digital signatures (S/MIME). Steps are listed
below

Reproducible: Always
Steps to Reproduce:
1.get a digital signature (mine is from Thawte.com)
2.request and install one under Mozilla
3.verify if it is there (PREFERENCES - PRIVACY - CERTIFICATES - MANAGE
CERTIFICATES - tab YOUR CERTIFICATES)
4.In mozilla mail : EDIT - MAIL & NEWSGROUP ACCOUNT SETTINGS - expand your
(IMAP) mail account, go to SECURITY - check the box, and select the certificate
to use - click OK
5.repeat step 4, and you'll notice that the box is unchecked again

Actual Results:  mozilla unchecked the boz
Expected Results:  that it would remain checked

Comment 1

16 years ago
I've also seen this happen for POP and NEWS, so it's all horked :(

See my bug 118405

Comment 2

16 years ago
Confirming.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 3

16 years ago
*** Bug 118405 has been marked as a duplicate of this bug. ***

Comment 4

16 years ago
*** Bug 118586 has been marked as a duplicate of this bug. ***
(Assignee)

Comment 5

16 years ago
I will take care of this today. Thanks for filing the bug.
Status: NEW → ASSIGNED

Comment 6

16 years ago
Created attachment 63848 [details] [diff] [review]
proposed patch.

Comment 7

16 years ago
racham: I don't have cvs write access. If you agree to the patch, you can take
it from there.
(Assignee)

Comment 8

16 years ago
Sure. Thanks for your patch. I have looked at the patch. It seemed to have done
things to enable and disable depending on the cert availability, which certainly
seemed to be missing today. Anyway, I will apply your patch and will go on from
there. Thanks again.
(Assignee)

Comment 9

16 years ago
Stephane, 
Never mind. Your patch looks right. I will just polish it little bit and land
it. thanks.

Comment 10

16 years ago
*** Bug 116622 has been marked as a duplicate of this bug. ***

Comment 11

16 years ago
This really prevents users from configuring s/mime. Can we get it in?
(Assignee)

Comment 12

16 years ago
Created attachment 64081 [details] [diff] [review]
patch, v1.1 - updated patch with proper identity pref retrieval mechanism

I had to fix couple of other places (besides the patch Stephane posted) to come
up with a complete patch. Pref reflection via Identity interface is the right
way of doing this. I have tested this latest updated patch on accounts
with/without certificates. It works fine.

Comment 13

16 years ago
Comment on attachment 64081 [details] [diff] [review]
patch, v1.1 - updated patch with proper identity pref retrieval mechanism

ack please don't check this in Bhuvan. All of the smime attributes are generic
attributes on the identity. They are all stored generically via a mechanism
Seth and I wrote. You just bypassed all of that and put back raw security
properties in the identity =). 

I'll take a look at this bug but right now setting smime properties via the
account manager stick just fine for me. I'm not sure if there's a specific
scenario where they aren't.
(Assignee)

Comment 14

16 years ago
Created attachment 64106 [details] [diff] [review]
patch, v1.2 - Fresh patch, reworked keeping smime generic pref saving mechanism (as mentioned by Scott) intact

Please ignore the patch 64081. Sorry about that mess.

Here is another attempy. Posting a new patch.

In order to reproduce the problem, 

1. Create a new account inside a profile that has a ccsee to certs. 
2. Click on Security item under that account in AccountManager window. 
3. If you are running a debug build, you should see the following js error on
console
JavaScript error:
chrome://messenger/content/am-smime.js line 61: gEncryptAlways is not defined
4. Notice that all the fields on the panel are enabled. None of items could be
saved as prefs.
5. Fixed the error in step 4, by defining gEncryptAlways.
6. Now, make changes to items on Security panel. Click on some other account 
items like Addressing or Server Settings. Click back on Security. Notice that
all the changes made are gone.
7. We need to save those values as prefs as user leaves the panel. The new
patch achieves that.

Note : In step 6, if you do not switch panels and click on OK to close the
AccountManager window, changes made are saved properly.

Please do let me know if there are issues with this patch. thanks.
Attachment #64081 - Attachment is obsolete: true

Comment 15

16 years ago
*** Bug 118045 has been marked as a duplicate of this bug. ***

Comment 16

16 years ago
In step 6, Even I did not switch panels and click OK to close the AccountManager 
window, the change in the security pane still NOT saved properly when I open the 
AccountManager window again...

I used 2002010810 trunk build.

Does anyone know any workaround for this bug? (like changing the prefs.js file)

Comment 17

16 years ago
Comment on attachment 64106 [details] [diff] [review]
patch, v1.2 - Fresh patch, reworked keeping smime generic pref saving mechanism (as mentioned by Scott) intact

sr=mscott
Attachment #64106 - Flags: superreview+
Comment on attachment 64106 [details] [diff] [review]
patch, v1.2 - Fresh patch, reworked keeping smime generic pref saving mechanism (as mentioned by Scott) intact

r=sspitzer
Attachment #64106 - Flags: review+
(Assignee)

Comment 19

16 years ago
Fix landed. Thanks to Stephane, Scott & Seth.

Marking fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 20

16 years ago
Verified fixed with 2002011103 build.

1. Change the security mail options in the Mail Account Setting (check/uncheck 
checkbox, selecting sign/encrypt cert), switch to other panel, back to security 
setting, the previous changes are preserved.

2. Change the security mail options in the Mail Account Setting (check/uncheck 
checkbox, selecting sign/encrypt cert), without switching to other panel, click 
OK to save and exit the Mail Account Setting. Verified the previous changes are 
saved when open the Mail Account Setting again.
Status: RESOLVED → VERIFIED

Comment 21

16 years ago
Yes, it is fixed however you cannot select the encrypt if possible option.  It
is  grayed out.

David

Comment 22

16 years ago
But still.... taht isn't a fix is it? It should save them regardless of if you
switch to other tabs.

Comment 23

16 years ago
There is already a separate bug regarding unable to select encrypt if possible 
option. Please see bug 117763 for details.
(Assignee)

Comment 24

16 years ago
*** Bug 119558 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.