Closed
Bug 1456920
Opened 7 years ago
Closed 5 months ago
Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml on Android
Categories
(Core :: DOM: Security, defect, P3)
Core
DOM: Security
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: vinoth, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [domsecurity-backlog1])
Attachments
(1 file)
Followup of Bug 1453560 for making changes to netError.xhtml and related mobile pages.
|
||
Updated•7 years ago
|
Whiteboard: [domsecurity-active]
|
||
Comment 1•7 years ago
|
||
|
Reporter | |
Comment 2•7 years ago
|
||
Comment on attachment 8973652 [details]
Bug 1456920 - Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml
Please review the patch and let me know if changes are needed.
Attachment #8973652 -
Flags: review?(ckerschb)
|
|
Reporter | |
Comment 3•7 years ago
|
||
(In reply to Vinothkumar Nagasayanan [:vinoth] from comment #2)
> Comment on attachment 8973652 [details]
> Bug 1456920 - Apply Meta CSP to Content Privileged neterror.xhtml and
> aboutCertError.xhtml
>
> Please review the patch and let me know if changes are needed.
Please review by enabling "Whitespace changes ignore all" in History tab of phabricator, to see the new changes I added clearly.
Or use the URL like this,
https://phabricator.services.mozilla.com/D1147?whitespace=ignore-all
|
|
||
Comment 4•7 years ago
|
||
Comment on attachment 8973652 [details]
Bug 1456920 - Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml
Christoph Kerschbaumer [:ckerschb] has approved the revision.
https://phabricator.services.mozilla.com/D1147
Attachment #8973652 -
Flags: review+
|
|
||
Updated•7 years ago
|
Attachment #8973652 -
Flags: review?(ckerschb)
|
|
Reporter | |
Comment 5•7 years ago
|
||
Comment on attachment 8973652 [details]
Bug 1456920 - Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml
Please review the patch by enabling "Whitespace changes ignore all" in History tab of phabricator, to see the new changes I added clearly.
Or use the URL like this,
https://phabricator.services.mozilla.com/D1147?whitespace=ignore-all
Attachment #8973652 -
Flags: review?(gijskruitbosch+bugs)
|
||
Updated•7 years ago
|
Attachment #8973652 -
Flags: review?(gijskruitbosch+bugs) → review-
|
|
||
Updated•7 years ago
|
Summary: Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml → Apply Meta CSP to Content Privileged neterror.xhtml and aboutCertError.xhtml on Android
|
|
Reporter | |
Updated•7 years ago
|
Assignee: cegvinoth → nobody
Status: ASSIGNED → NEW
Priority: P2 → P3
Whiteboard: [domsecurity-active] → [domsecurity-backlog1]
|
||
Updated•3 years ago
|
Severity: normal → S3
|
||
Updated•2 years ago
|
Blocks: necko-error
|
|
||
Comment 6•5 months ago
|
||
Both desktop and mobile error pages have CSP now, I think.
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•