Closed Bug 1374718 Opened 3 years ago Closed 3 years ago

Remove strictContentSandbox flag for mochitest

Categories

(Core :: Security: Process Sandboxing, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: Alex_Gaynor, Assigned: Alex_Gaynor)

References

Details

Attachments

(1 file)

It's pretty out of date. It sets the content process sandbox to level 1, which is _less_ strict than the current nightly default of 2. I don't think there's a real use case for this, as evidenced by the fact that it's been broken/actively harmful for a while now.
Duplicate of this bug: 1363788
Comment on attachment 8879640 [details]
Bug 1374718 - removed strictContentSandbox mochitest option;

https://reviewboard.mozilla.org/r/150994/#review155760

Just duped over Bug 1363788, from my comment there I don't think you've got all of this, there are some mozharness files at least.
Attachment #8879640 - Flags: review?(bobowencode)
Comment on attachment 8879640 [details]
Bug 1374718 - removed strictContentSandbox mochitest option;

https://reviewboard.mozilla.org/r/150994/#review155804
Attachment #8879640 - Flags: review?(bobowencode) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3d1f8cbf8836
removed strictContentSandbox mochitest option; r=bobowen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3d1f8cbf8836
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.