Closed
Bug 81257
Opened 23 years ago
Closed 23 years ago
Accepting Root CA- View and Policy buttons not working.
Categories
(Core Graveyard :: Security: UI, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
psm2.1
People
(Reporter: junruh, Assigned: rangansen)
References
()
Details
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
1.) Visit the above URL to accept the Root CA.
2.) In the dialog box, click on View and Policy.
What happens: Nothing.
Updated•23 years ago
|
Keywords: nsenterprise
Reporter | ||
Comment 8•23 years ago
|
||
Here is also an in-house test case - https://junruh.mcom.com:444/GetCAChain.html
Reporter | ||
Comment 9•23 years ago
|
||
How about hooking up View for now, and removing the Policy button?
Comment 10•23 years ago
|
||
I wouldn't mind doing this soon on the trunk, but it's not likely going to be
fixed for 0.9.2.
There is a workaround: accept the CA cert, and then view it in the Cert
Manager. If you don't want to accept it, you can delete it.
Comment 11•23 years ago
|
||
*** Bug 89637 has been marked as a duplicate of this bug. ***
Comment 12•23 years ago
|
||
XUL polish->P2
Comment 14•23 years ago
|
||
*** Bug 91393 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Comment 17•23 years ago
|
||
The fix should remove the policy button and make sure that the view cert button
pops up the cert viewer. I will file a separate bug to get the cert viewer
display the policy.
Assignee | ||
Comment 18•23 years ago
|
||
Assignee | ||
Comment 19•23 years ago
|
||
Patch to remove the Policy button, and get the view button working.
Comment 20•23 years ago
|
||
r=javi
Comment 22•23 years ago
|
||
sr=hewitt
Assignee | ||
Comment 23•23 years ago
|
||
We need this additional patch to ensure that ca certs which does not have a
proper issuer can be viewed as well ....
Assignee | ||
Comment 24•23 years ago
|
||
Comment 25•23 years ago
|
||
+
der.data = (unsigned char *)mByteData;
+ der.len = mContentLength;
Funny tabbing there. Other than that sr=blizzard. I guess you need another review.
Assignee | ||
Comment 26•23 years ago
|
||
The fix suggested in the last patch will be included in javi's patch
to bug# 91407. Marking this as 'depends on' for keeping track
Depends on: 91407
Assignee | ||
Comment 27•23 years ago
|
||
Patch checked in...
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 28•23 years ago
|
||
Verified.
Status: RESOLVED → VERIFIED
QA Contact: ckritzer → junruh
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•