Closed Bug 582579 Opened 14 years ago Closed 14 years ago

Enable multiple new roots from NSS 3.12.8 for EV.

Categories

(Core :: Security: PSM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- -
status1.9.2 --- .14-fixed
status1.9.1 --- .17-fixed

People

(Reporter: KaiE, Assigned: KaiE)

References

Details

(Keywords: verified1.9.1, verified1.9.2)

Attachments

(1 file, 2 obsolete files)

Enable multiple new roots from NSS 3.12.8 for EV.

See blocker list.
Depends on: 582580
Attached patch Patch v1 (obsolete) — Splinter Review
Enabling the roots from the bugs listed in the dependency list.

Not yet requesting code review, pending test results from CAs.
Comment on attachment 462617 [details] [diff] [review]
Patch v1

(In reply to comment #1)
> Created attachment 462617 [details] [diff] [review]
> Patch v1
> 
> Enabling the roots from the bugs listed in the dependency list.
> 
> Not yet requesting code review, pending test results from CAs.

sorry, wrong bug... please ignore comment 1
Attachment #462617 - Attachment is obsolete: true
Attached patch Patch v2 (obsolete) — Splinter Review
Enabling the roots for EV which are listed in the bugs seens in the dependency list.

Not yet requesting review, pending positive test results (OK OCSP servers) and pending feedback from CAs.
No longer blocks: 562399
Removing Izpene from dependency list, will not get included with this batch.
No longer blocks: 578499
Attached patch Patch v3Splinter Review
Bob or Honza, could you please review?
Assignee: nobody → kaie
Attachment #463706 - Flags: superreview?(rrelyea)
Attachment #463706 - Flags: review?(honzab.moz)
Comment on attachment 463706 [details] [diff] [review]
Patch v3

I'm withdrawing and postponing the review request until bug 586414 gets resolved.
Attachment #463706 - Flags: superreview?(rrelyea)
Attachment #463706 - Flags: review?(honzab.moz)
Whiteboard: [EV candidate roots in 3.12.8 need more investigation]
Comment on attachment 463706 [details] [diff] [review]
Patch v3

Bug 586414 has been closed, so I'm re-requesting review for this patch.
Attachment #463706 - Flags: review?(rrelyea)
Attachment #462619 - Attachment is obsolete: true
Whiteboard: [EV candidate roots in 3.12.8 need more investigation]
Comment on attachment 463706 [details] [diff] [review]
Patch v3

r+
Attachment #463706 - Flags: review?(rrelyea) → review+
blocking2.0: --- → ?
Depends on: 595300
No longer depends on: 582580
Comment on attachment 463706 [details] [diff] [review]
Patch v3

Request to enable one new root for EV, which has been added to mozilla-2, FF 4, with the recent 3.12.8
Attachment #463706 - Flags: approval2.0?
An EV enablement like this wouldn't block release, but I'll a+ the patch momentarily for landing after the tree reopens post-b7
blocking2.0: ? → -
Attachment #463706 - Flags: approval2.0? → approval2.0+
Whiteboard: [can land after b7]
Finally landed, marking fixed.
http://hg.mozilla.org/mozilla-central/rev/4e66b4cb5e3a
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [can land after b7]
Comment on attachment 463706 [details] [diff] [review]
Patch v3

Branches 1.9.1 and 1.9.2 both use NSS 3.12.8, too,
so this should land on those stable branches.

Requesting approval for 1.9.1.16 and 1.9.2.13.

If it's too late, please push request to the following release, thanks.
Attachment #463706 - Flags: approval1.9.2.13?
Attachment #463706 - Flags: approval1.9.1.16?
Comment on attachment 463706 [details] [diff] [review]
Patch v3

a=LegNeato for 1.9.2.14 and 1.9.1.17
Attachment #463706 - Flags: approval1.9.2.14+
Attachment #463706 - Flags: approval1.9.2.13?
Attachment #463706 - Flags: approval1.9.1.17+
Attachment #463706 - Flags: approval1.9.1.16?
thanks for the approval.

It's a trivial patch with no risk.
If someone would like to land it, please go ahead, I'd appreciate your help!
Keywords: checkin-needed
Verified for 1.9.1 and 1.9.2 by looking at source.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: