Enable multiple roots for EV (and remove 1): Geotrust, Thawte, Verisign, Trustwave, Comodo

RESOLVED FIXED

Status

()

Core
Security: PSM
P2
normal
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: kaie, Assigned: kaie)

Tracking

Trunk
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

6.24 KB, patch
Robert Relyea
: review+
Nelson Bolyard (seldom reads bugmail)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

10 years ago
This bug is meant to land a single patch that enables multiple roots for EV.
(Assignee)

Updated

10 years ago
Blocks: 418902
(Assignee)

Updated

10 years ago
Depends on: 425469
(Assignee)

Comment 1

10 years ago
Created attachment 312142 [details] [diff] [review]
Patch v1
(Assignee)

Updated

10 years ago
Depends on: 425538
(Assignee)

Updated

10 years ago
Blocks: 426572, 426575
Summary: Enable multiple roots for EV (and remove 1): Geotrust, Thawte, Verisign, Trustwave → Enable multiple roots for EV (and remove 1): Geotrust, Thawte, Verisign, Trustwave, Comodo
(Assignee)

Updated

10 years ago
Flags: blocking1.9?
(Assignee)

Comment 2

10 years ago
Created attachment 313270 [details] [diff] [review]
Patch v2

This patch has Comodo's roots in addition.
Attachment #312142 - Attachment is obsolete: true
(Assignee)

Updated

10 years ago
Depends on: 426681
+'ing this so we get a resolution.  Don't want to hold this progress back.
Flags: blocking1.9? → blocking1.9+
Priority: -- → P2
(Assignee)

Updated

10 years ago
Attachment #313270 - Flags: review?(rrelyea)
Comment on attachment 313270 [details] [diff] [review]
Patch v2

I see nothing obviously wrong with this patch.  
It is correct in form.  
I have not verified that the serial numbers of the 8 certs being added are the expected 8 serial numbers.
Attachment #313270 - Flags: review+

Comment 5

10 years ago
Comment on attachment 313270 [details] [diff] [review]
Patch v2

r+ rrelyea
Attachment #313270 - Flags: review?(rrelyea) → review+
(Assignee)

Comment 6

10 years ago
checked in, marking fixed.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.