555860 - Enable Certplus Class 2 Primary CA for EV in PSM

Status: RESOLVED FIXED

(Core :: Security: PSM, enhancement)

Description

[Kathleen Wilson]

Per bug 497917 the request from Keynectis/Certplus has been approved to enable its Certplus “Class 2 Primary CA” root certificate for EV use. Please make the corresponding changes to PSM.

The relevant information is as follows:

Friendly name: Certplus Class 2 Primary CA

SHA1 Fingerprint: 
74:20:74:41:72:9C:DD:92:EC:79:31:D8:23:10:8D:C2:81:92:E2:BB

EV policy OID:  1.3.6.1.4.1.22234.2.5.2.3.1 

Test URL: https://www.keynectis.com

Comment 1

[Kathleen Wilson]

Erwann, Please confirm that the above information is correct.

Comment 2

[Erwann Abalea]

I double checked the SHA1 fingerprint, and the EV Policy OID. Everything mentioned above is correct.
BTW, thank you for your implication.

Comment 3

[Kathleen Wilson]

Thanks for confirming that the data in this bug is correct.

Root inclusions/updates are usually grouped and done as a batch when there is either a large enough set of changes or about every 3 months.

At some point in the next 3 months a test build will be provided and this bug will be updated to request that you test it. Since you are cc'd on this bug, you will get notification via email when that happens.

Comment 4

[Kai Engert (:KaiE:)]

Attachment: Patch v1

Using this patch I get the expected green EV indicator.

Comment 5

[Kai Engert (:KaiE:)]

Kathleen, I wonder if we could simplify the verification procedure (this time only), given that no NSS update is necessary.

My proposal is:
- I've already tested that it appears to work
- we get the code review
- we add the code to the experimental Firefox nightly builds
- the CA representatives verify correctness using an 
  Firefox 3.7 nightly developer build
- if correct, we could proceed (if desired) to add it to stable branches

Kathleen, would you be OK with this simplified procedure?

Erwann Abalea, would you be OK to perform the test using an "alpha quality" version of Firefox?

Comment 6

[Kai Engert (:KaiE:)]

For you and my reference, I'm talking about the nightly builds that are updated each night at this location:
ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-trunk/

(your root is NOT yet enabled in those builds, we'll update the bug if the proposal is accepted and once the code has been added)

Comment 7

[Erwann Abalea]

I'm OK to test such an "alpha quality" version of Firefox.

Comment 8

[Kathleen Wilson]

Kai, That sounds like a good plan.  Thanks!

Comment 9

[Honza Bambas (:mayhemer)]

Comment on attachment 437711 [details] [diff] [review]
Patch v1

Works for me.

>diff --git a/security/manager/ssl/src/nsIdentityChecking.cpp b/security/manager/ssl/src/nsIdentityChecking.cpp
>+    //    CN=Class 2 Primary CA,O=Certplus,C=FR

Maybe just adjust to one space after // to be consistent with the rest of the file.

r=honzab

Comment 10

[Robert Relyea]

Comment on attachment 437711 [details] [diff] [review]
Patch v1

r+ rrelyea

Comment 11

[Kai Engert (:KaiE:)]

Checked in
http://hg.mozilla.org/mozilla-central/rev/e6b82019c7a7

Erwann: Please wait until tomorrow, then go to the address from comment 6, and grab a alpha quality build with a timestamp of May 04.
Thanks in advance for testing.
Please let us know if it works right for you.

Comment 12

[Erwann Abalea]

Sorry I couldn't check yesterday, too busy.
I just downloaded and tested the May 05 version, for Linux i686, that's right for me, I get the green bar.

Thanks, Kai.

Comment 13

[Kai Engert (:KaiE:)]

Comment on attachment 437711 [details] [diff] [review]
Patch v1

requesting EV approval for stable firefox branches

Comment 14

[Daniel Veditz [:dveditz]]

Comment on attachment 437711 [details] [diff] [review]
Patch v1

Approved for 1.9.2.6 and 1.9.1.11, a=dveditz for release-drivers

Comment 15

[Kai Engert (:KaiE:)]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a1fd2f3881ff

Comment 16

[Kai Engert (:KaiE:)]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/61b116a969ef