Closed Bug 1566386 Opened 5 years ago Closed 5 years ago

Evaluate and Assert that all about: pages on Android ship with a strong CSP

Categories

(Core :: DOM: Security, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

It seems we have !defined(ANDROID) where we do the assert to make sure all about pages ship with a strong CSP [1]. We should remove that and make sure all about: pages on android also have a strong CSP.

[1] https://hg.mozilla.org/mozilla-central/rev/42ac00d4125c#l1.30

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task

Snorp, we are currently applying strong CSPs to all of our about: pages (at least for desktop). In a first try run [1] I wanted to see how much work there is on android. It seems like a lot :-(

I assume every single about: page is different on android and desktop versions of Firefox? I can certainly tackle some of them, but I was wondering if we coordinate the effort somehow to get some help with that. Happy to hop on a meeting to discuss details. Just wanted to get your initial feedback on how we could tackle that and also apply strong CSPs to all of android's about: pages.

[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=b15c60948bd05b80050a329be8b21c509c04caf6&selectedJob=258266917

Flags: needinfo?(snorp)

For GeckoView, I think we really only have about:config as special. Fennec has more[1], and we might be accidentally including some of those in GeckoView. Maybe there should be some kind of audit done at the same time?

[1] https://searchfox.org/mozilla-central/source/mobile/android/chrome/content

Flags: needinfo?(snorp)
Pushed by mozilla@christophkerschbaumer.com: https://hg.mozilla.org/integration/autoland/rev/afc198c3f9fe Assert that every about: page on Android ships with a CSP. r=snorp
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: