Closed
Bug 1493249
Opened 6 years ago
Closed 6 years ago
Add support for security devices
Categories
(Firefox :: Enterprise Policies, enhancement, P1)
Firefox
Enterprise Policies
Tracking
()
VERIFIED
FIXED
Firefox 64
People
(Reporter: mkaply, Assigned: mkaply)
References
Details
Attachments
(4 files)
46 bytes,
text/x-phabricator-request
|
Felipe
:
review+
|
Details | Review |
7.32 KB,
patch
|
lizzard
:
approval-mozilla-esr60+
|
Details | Diff | Splinter Review |
2.19 KB,
patch
|
lizzard
:
approval-mozilla-esr60+
|
Details | Diff | Splinter Review |
1.21 KB,
patch
|
Felipe
:
review+
RyanVM
:
approval-mozilla-esr60+
|
Details | Diff | Splinter Review |
Add support for adding PKCS11 security devices.
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Note: this should have tests, both for the policy itself, but also for the parsing of patternProperties (you could add it to https://searchfox.org/mozilla-central/source/toolkit/components/utils/test/browser/browser_JsonSchemaValidator.js )
Assignee | ||
Comment 3•6 years ago
|
||
There's really no good way to test the policy. We're calling a known API that is tested. I wasn't sure where to add the validator test.
Comment 4•6 years ago
|
||
Comment on attachment 9011041 [details] Bug 1493249 - Add policy for security devices. :Felipe Gomes (needinfo me!) has approved the revision.
Attachment #9011041 -
Flags: review+
Pushed by mozilla@kaply.com: https://hg.mozilla.org/integration/autoland/rev/86baeb291e66 Add policy for security devices. r=Felipe
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/86baeb291e66
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 7•6 years ago
|
||
This bug was covered by the overall testing efforts invested in the Additional Enterprise Policies feature. Marking this as verified fixed using Firefox 64.0a1 (BuildId:20181011220118) on Windows 10 64bit and macOS 10.13.6
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 8•6 years ago
|
||
[ESR Uplift Approval Request] If this is not a sec:{high,crit} bug, please state case for ESR consideration: Policy change to align with Firefox 64 User impact if declined: Fix Landed on Version: 64 Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): Policy only change String or UUID changes made by this patch: None
Attachment #9024119 -
Flags: approval-mozilla-esr60?
Comment 9•6 years ago
|
||
Comment on attachment 9024119 [details] [diff] [review] Patch for ESR Verified in 64, OK to uplift this enterprise policy change.
Attachment #9024119 -
Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
Updated•6 years ago
|
status-firefox-esr60:
--- → affected
tracking-firefox-esr60:
--- → 64+
Comment 10•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/b9c34e2086c2
Flags: in-testsuite+
Comment 11•6 years ago
|
||
Hi Mike, I tried verifying this issue using the esr build (provided in comment 10) but it seems that the policy is not recognized. Console output: Enterprise Policies:Unknown policy: SecurityDevices
Flags: needinfo?(mozilla)
Assignee | ||
Comment 12•6 years ago
|
||
[ESR Uplift Approval Request] If this is not a sec:{high,crit} bug, please state case for ESR consideration: Missing part of previous patch User impact if declined: Function doesn't work Fix Landed on Version: Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): corrects schema change String or UUID changes made by this patch:
Flags: needinfo?(mozilla)
Attachment #9025722 -
Flags: approval-mozilla-esr60?
Assignee | ||
Comment 13•6 years ago
|
||
I screwed up and forgot a piece of the patch. This is needed before bug 1498223 will apply.
Comment 14•6 years ago
|
||
Thanks Emil and Mike! Good catch.
Comment 15•6 years ago
|
||
Comment on attachment 9025722 [details] [diff] [review] Followup Fix for new enterprise policy on ESR.
Attachment #9025722 -
Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
Assignee | ||
Comment 17•6 years ago
|
||
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #16) > Mike, did this ever actually land on ESR60? No, thanks for the reminder. https://hg.mozilla.org/releases/mozilla-esr60/rev/ff3b417e363f65040aa4eb38d249a740191f7c87
Flags: needinfo?(mozilla)
Assignee | ||
Comment 18•6 years ago
|
||
Emil Can you reverify when the next esr nightly is available?
Flags: needinfo?(emil.ghitta)
Assignee | ||
Comment 19•6 years ago
|
||
[ESR Uplift Approval Request] If this is not a sec:{high,crit} bug, please state case for ESR consideration: Another followup to patch User impact if declined: Feature won't work Fix Landed on Version: ESR only Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): Liz, I'm very sorry about this. I should have tested more thoroughly instead of just assuming this will work. All other ESR patches will be built and tested locally. String or UUID changes made by this patch:
Attachment #9028359 -
Flags: review?(felipc)
Attachment #9028359 -
Flags: approval-mozilla-esr60?
Updated•6 years ago
|
Attachment #9028359 -
Flags: review?(felipc) → review+
Updated•6 years ago
|
Comment 20•6 years ago
|
||
Comment on attachment 9028359 [details] [diff] [review] Correct esr60 iterator problem Another follow-up fix for ESR, approved for 60.4.0esr. Glad we've got QA verifying these!
Attachment #9028359 -
Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
Comment 21•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/641d73998992
Comment 22•6 years ago
|
||
This is verified fixed using Firefox 60.3.1esr (provided in comment 21) on Windows 10 64bit and macOS 10.14
Flags: needinfo?(emil.ghitta)
You need to log in
before you can comment on or make changes to this bug.
Description
•