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)

Product:
Core Graveyard
Component:
Security: UI
Version:
1.0 Branch
Platform:
x86
Windows NT
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
psm2.1

People

(Reporter: junruh, Assigned: rangansen)

References

(
URL
)

Details

Attachments

(2 files)

patch
3.85 KB, patch
Details | Diff | Splinter Review
patch
1.57 KB, 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.
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 — — 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
        — 
          — 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: