Skip to content

Commit

Permalink
FF Clarify Feature-Policy header not sent (#20729)
Browse files Browse the repository at this point in the history
* FF Clarify header not sent

* Change notes to not-recognised

* Fix typo recognised
  • Loading branch information
hamishwillee authored Sep 22, 2023
1 parent 2445bb1 commit 0338141
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 40 deletions.
40 changes: 19 additions & 21 deletions http/headers/Feature-Policy.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"firefox": {
"version_added": "74",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand All @@ -26,7 +26,7 @@
"safari": {
"version_added": "11.1",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
Expand Down Expand Up @@ -118,15 +118,13 @@
"edge": "mirror",
"firefox": {
"version_added": "74",
"flags": [
{
"type": "preference",
"name": "dom.security.featurePolicy.header.enabled",
"value_to_set": "true"
}
]
"partial_implementation": true,
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": {
"version_added": false,
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": "mirror",
"ie": {
"version_added": false
},
Expand All @@ -141,7 +139,7 @@
"webview_android": "mirror"
},
"status": {
"experimental": true,
"experimental": false,
"standard_track": true,
"deprecated": false
}
Expand Down Expand Up @@ -195,7 +193,7 @@
"firefox": {
"version_added": "74",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -237,7 +235,7 @@
"firefox": {
"version_added": "74",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": {
"version_added": false
Expand All @@ -251,7 +249,7 @@
"safari": {
"version_added": "13",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"safari_ios": {
"version_added": false
Expand Down Expand Up @@ -367,7 +365,7 @@
"version_added": "74",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>).",
"Before Firefox 80, applying <code>fullscreen</code> to an <code>&lt;iframe&gt;</code> (i.e. via the <code>allow</code> attribute) does not work unless the <code>allowfullscreen</code> attribute is also present."
]
},
Expand Down Expand Up @@ -413,7 +411,7 @@
"version_added": "91",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>).",
"The default allowlist is <code>*</code> instead of <code>self</code> (as required by the specification)."
]
},
Expand Down Expand Up @@ -451,7 +449,7 @@
"firefox": {
"version_added": "74",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -660,7 +658,7 @@
"firefox": {
"version_added": "74",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -927,7 +925,7 @@
"firefox": {
"version_added": "116",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"firefox_android": {
"version_added": false,
Expand Down Expand Up @@ -1102,14 +1100,14 @@
"version_added": "81",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>).",
"Firefox recognizes the <code>web-share</code> permissions policy, but this has no effect in versions of Firefox that do not support the <a href='https://developer.mozilla.org/docs/Web/API/Navigator/share'><code>share()</code></a> method."
]
},
"firefox_android": {
"version_added": "81",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1627890'>bug 1627890</a>)"
},
"ie": {
"version_added": false
Expand Down
33 changes: 14 additions & 19 deletions http/headers/Permissions-Policy.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"version_added": "74",
"alternative_name": "Feature-Policy",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand All @@ -34,7 +34,7 @@
"version_added": "11.1",
"alternative_name": "Feature-Policy",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"safari_ios": "mirror",
"samsunginternet_android": "mirror",
Expand Down Expand Up @@ -145,13 +145,8 @@
"firefox": {
"version_added": "74",
"alternative_name": "Feature-Policy: autoplay",
"flags": [
{
"type": "preference",
"name": "dom.security.featurePolicy.header.enabled",
"value_to_set": "true"
}
]
"partial_implementation": true,
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -228,7 +223,7 @@
"version_added": "74",
"alternative_name": "Feature-Policy: camera",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -270,7 +265,7 @@
"version_added": "74",
"alternative_name": "Feature-Policy: display-capture",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": {
"version_added": false
Expand All @@ -285,7 +280,7 @@
"version_added": "13",
"alternative_name": "Feature-Policy: display-capture",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"safari_ios": {
"version_added": false
Expand Down Expand Up @@ -515,7 +510,7 @@
"alternative_name": "Feature-Policy: fullscreen",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>).",
"Before Firefox 80, applying <code>fullscreen</code> to an <code>&lt;iframe&gt;</code> (i.e. via the <code>allow</code> attribute) does not work unless the <code>allowfullscreen</code> attribute is also present."
]
},
Expand Down Expand Up @@ -555,7 +550,7 @@
"alternative_name": "Feature-Policy: gamepad",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>).",
"The default allowlist is <code>*</code> instead of <code>self</code> (as required by the specification)."
]
},
Expand Down Expand Up @@ -600,7 +595,7 @@
"version_added": "74",
"alternative_name": "Feature-Policy: geolocation",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)."
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -865,7 +860,7 @@
"version_added": "74",
"alternative_name": "Feature-Policy: microphone",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": "mirror",
"ie": {
Expand Down Expand Up @@ -1249,7 +1244,7 @@
"version_added": "116",
"alternative_name": "Feature-Policy: speaker-selection",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"firefox_android": {
"version_added": false,
Expand Down Expand Up @@ -1372,14 +1367,14 @@
"alternative_name": "Feature-Policy: web-share",
"partial_implementation": true,
"notes": [
"Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements.",
"Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>).",
"Firefox recognizes the <code>web-share</code> permissions policy, but this has no effect in versions of Firefox that do not support the <a href='https://developer.mozilla.org/docs/Web/API/Navigator/share'><code>share()</code></a> method."
]
},
"firefox_android": {
"version_added": "81",
"partial_implementation": true,
"notes": "Only supported through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements."
"notes": "Header not recognized but policy can be set through the <code>allow</code> attribute on <code>&lt;iframe&gt;</code> elements (see <a href='https://bugzil.la/1694922'>bug 1694922</a>)"
},
"ie": {
"version_added": false
Expand Down

0 comments on commit 0338141

Please sign in to comment.