1758951 - [meta] Enable Eslint rule `no-insecure-url` globally on all test files

Status: NEW

(Core :: DOM: Security, task, P2)

Description

[Tomer Yavor]

Since the web is migrating towards HTTPS, we think our tests on mozilla-central should do so too.
For that reason we are working on a linter: https://bugzilla.mozilla.org/show_bug.cgi?id=1709150

If the linter would be enabled by default for all our components and tests, the number of warnings for existing test would be 5 times as high as it is currently.
We can't address all that warnings at once and won't be able to do it without the different teams that worked on the tests. But we think it is crucial to land that linter asap. To increase the landing process and to make it more reasonable we exempted components and tests that lead to many warnings.

Those tests should get updated to use https such that we can remove the exemptions for them and enabling the linter for more components.

Comment 1

[Tomer Yavor]

Attachment: WIP: Bug 1758951 - Adjust tests which are exempt by (insecure-url) linter to use https. r=freddyb

Comment 2

[Tomer Yavor]

Attachment: WIP: Bug 1758951 - Adjust tests which are exempt by (insecure-url) linter to use https. r=freddyb

Comment 3

[Tomer Yavor]

Attachment: WIP: Bug 1758951 - Add for all exisiting test. r=mossop

Comment 4

[Malte Jürgens [:maltejur]]

Attachment: Bug 1758951 - Update @microsoft/eslint-plugin-sdl to version 0.2.2 r=freddyb,standard8

Comment 5

[Phabricator Automation]

Comment on attachment 9363249 [details]
Bug 1758951 - Update @microsoft/eslint-plugin-sdl to version 0.2.2 r=freddyb,standard8

Revision D193423 was moved to bug 1786290. Setting attachment 9363249 [details] to obsolete.