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)

1.0 Branch
x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED
psm2.1

People

(Reporter: junruh, Assigned: rangansen)

References

()

Details

Attachments

(2 files)

1.) Visit the above URL to accept the Root CA. 2.) In the dialog box, click on View and Policy. What happens: Nothing.
Target Milestone --- -> 2.0.
Target Milestone: --- → 2.0
*** Bug 82623 has been marked as a duplicate of this bug. ***
*** Bug 83436 has been marked as a duplicate of this bug. ***
->p2
Priority: -- → P2
*** Bug 84060 has been marked as a duplicate of this bug. ***
Assigning to javi.
Assignee: ddrinan → javi
Mass reassigning target to 2.1
Target Milestone: 2.0 → 2.1
Keywords: nsenterprise
Here is also an in-house test case - https://junruh.mcom.com:444/GetCAChain.html
How about hooking up View for now, and removing the Policy button?
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.
*** Bug 89637 has been marked as a duplicate of this bug. ***
XUL polish->P2
rangansen
Assignee: javi → rangansen
*** Bug 91393 has been marked as a duplicate of this bug. ***
making this a P1
Priority: P2 → P1
Status: NEW → ASSIGNED
Mass assigning QA to ckritzer.
QA Contact: junruh → ckritzer
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.
Attached patch patch (deleted) — Splinter Review
Patch to remove the Policy button, and get the view button working.
r=javi
adding patch, review keywords.
Keywords: patch, review
sr=hewitt
We need this additional patch to ensure that ca certs which does not have a proper issuer can be viewed as well ....
Attached patch patch (deleted) — Splinter Review
+ der.data = (unsigned char *)mByteData; + der.len = mContentLength; Funny tabbing there. Other than that sr=blizzard. I guess you need another review.
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
Patch checked in...
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified.
Status: RESOLVED → VERIFIED
QA Contact: ckritzer → junruh
Product: PSM → Core
Version: psm2.0 → 1.0 Branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: