Closed
Bug 1408080
Opened 8 years ago
Closed 8 years ago
October 2017 batch of root CA changes
Categories
(NSS :: CA Certificates Code, task)
Tracking
(Not tracked)
RESOLVED
FIXED
3.34
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
Attachments
(2 files)
|
168.50 KB,
patch
|
kathleen.a.wilson
:
review+
KaiE
:
checked-in+
|
Details | Diff | Splinter Review |
|
33.86 KB,
patch
|
kathleen.a.wilson
:
review+
KaiE
:
checked-in+
|
Details | Diff | Splinter Review |
October 2017 batch of root CA changes
| Assignee | ||
Updated•8 years ago
|
| Assignee | ||
Comment 1•8 years ago
|
||
| Assignee | ||
Comment 2•8 years ago
|
||
Test build with the attached patch v1:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cddbbca4c9ec9c9c4b112043d082d374a6bb75ac
After the builds are complete, in order to find the binary, for example, for Mac OSX:
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.
| Assignee | ||
Comment 3•8 years ago
|
||
Comment on attachment 8917993 [details] [diff] [review]
1408080-v1.patch
Please test the build and review this patch, thanks.
Attachment #8917993 -
Flags: review?(kwilson)
Comment 4•8 years ago
|
||
I have verified that the changes are as expected.
I have tested with the Mac OSX test build and confirm the correct removals and additions.
Testing succeeded for the addition of GDCA TrustAUTH R5 ROOT, TrustCor RootCert CA-1, TrustCor RootCert CA-2.
Testing failed for the addition of TrustCor ECA-1 -- probably something wrong with the test site.
I will ask the CAs to test now.
Thanks!
Comment 5•8 years ago
|
||
(In reply to Kathleen Wilson from comment #4)
> Testing failed for the addition of TrustCor ECA-1 -- probably something
> wrong with the test site.
I had the wrong test website URL. The correct one is:
https://valid.epki.external.trustcor.ca
And it works.
Waiting for the CAs to do their testing.
Comment 6•8 years ago
|
||
CAs have finished testing.
Comment 7•8 years ago
|
||
Comment on attachment 8917993 [details] [diff] [review]
1408080-v1.patch
I confirm that the changes are as requested, I have tested the changes, and the CAs have verified their root additions.
Thanks!
Attachment #8917993 -
Flags: review?(kwilson) → review+
| Assignee | ||
Comment 8•8 years ago
|
||
Comment on attachment 8917993 [details] [diff] [review]
1408080-v1.patch
https://hg.mozilla.org/projects/nss/rev/946f134980f3
Attachment #8917993 -
Flags: checked-in+
| Assignee | ||
Updated•8 years ago
|
Attachment #8917993 -
Attachment description: 1395726-v1.patch → 1408080-v1.patch
Attachment #8917993 -
Attachment filename: 1395726-v1.patch → 1408080-v1.patch
| Assignee | ||
Comment 10•8 years ago
|
||
Incremental patch, part 2, adds roots from bug 1410954
| Assignee | ||
Comment 11•8 years ago
|
||
Test build that contains both attached patches:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=28d1a40bfeeeccbe5c8d7ac657d7683ab61055a7
| Assignee | ||
Comment 12•8 years ago
|
||
Comment on attachment 8921186 [details] [diff] [review]
1408080-part2.patch
Please test the new build and review this patch, thanks.
Attachment #8921186 -
Flags: review?(kwilson)
Comment 13•8 years ago
|
||
I confirm that the patch is correct, and I have verified the changes in the test build on Mac OS.
I will ask the CA to do their testing.
Thanks!
Comment 14•8 years ago
|
||
Comment on attachment 8921186 [details] [diff] [review]
1408080-part2.patch
I confirm that the changes are as requested, I have tested the changes, and the CAs have verified their root additions.
Thanks!
Attachment #8921186 -
Flags: review?(kwilson) → review+
| Assignee | ||
Comment 15•8 years ago
|
||
Comment on attachment 8921186 [details] [diff] [review]
1408080-part2.patch
https://hg.mozilla.org/projects/nss/rev/3dfd171371fa
Attachment #8921186 -
Flags: checked-in+
| Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•