Closed Bug 1380941 Opened 7 years ago Closed 7 years ago

July 2017 batch of root CA changes

Categories

(NSS :: CA Certificates Code, task)

3.31
task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: KaiE, Assigned: KaiE)

References

Details

Attachments

(1 file)

July 2017 batch of root CA changes

See dependency list for included changes.
Attached patch 1380941-v1.patchSplinter Review
After applying 1380941-v1.patch, certdata.txt no longer contains any lines like
  CKA_TRUST_CODE_SIGNING CK_TRUST CKT_NSS_TRUSTED_DELEGATOR

The only lines starting with CKA_TRUST_CODE_SIGNING, and the number of occurrences, are:

$ grep ^CKA_TRUST_CODE_SIGNING lib/ckfw/builtins/certdata.txt  | sort | uniq -c
    159 CKA_TRUST_CODE_SIGNING CK_TRUST CKT_NSS_MUST_VERIFY_TRUST
     28 CKA_TRUST_CODE_SIGNING CK_TRUST CKT_NSS_NOT_TRUSTED

The patch is big because I've included 50 context lines, to make it easier to see to which root CA the trust changes are related to.
Attachment #8886511 - Flags: review?(kwilson)
(In reply to Kai Engert (:kaie:) from comment #2)
> test build started:
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=33ff6eaa05457339ef6219d0c7de73617f501b1b
> 
> Once completed, builds will be available at:
> https://archive.mozilla.org/pub/firefox/try-builds/kaie@kuix.de-
> 33ff6eaa05457339ef6219d0c7de73617f501b1b/

I only see try-win32 and try-win64. (which I don't have)

Would you please create a try build for MacOS? 

PS: I reviewed the diff of the patch, and it looks correct. So, now I just need to test.
Flags: needinfo?(kaie)
(In reply to Kathleen Wilson from comment #3)
> Would you please create a try build for MacOS? 

Thanks to RyanVM who explained to me that things have changed a bit.

In order to find the binary, it's necessary to use the treeherder link from command 2.

Find the line with OSX that contains tc(B). Click the green B inside that tc(B), and the bottom part of the page will change to show more details of that build.

In the right hand colum, you see several "artifact uploaded" entries.

Scroll down that list to find "target.dmg". That's the Mac disk image that you're looking for. Does that work?
Flags: needinfo?(kaie) → needinfo?(kwilson)
(In reply to Kai Engert (:kaie:) from comment #4)
> In order to find the binary, it's necessary to use the treeherder link from
> command 2.

from "comment 2".
That worked.  I have tested, and confirmed the changes.
Flags: needinfo?(kwilson)
Attachment #8886511 - Flags: review?(kwilson) → review+
https://hg.mozilla.org/projects/nss/rev/f212be04f3d0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: