-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add [SecureContext]
tags to the interfaces
#192
Conversation
@@ -361,6 +361,7 @@ <h2> | |||
the main <a href="https://w3c.github.io/gamepad/">Gamepad spec</a>. | |||
</p> | |||
<pre class="idl"> | |||
[SecureContext] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is valid... it's a partial interface.
@@ -361,6 +361,7 @@ <h2> | |||
the main <a href="https://w3c.github.io/gamepad/">Gamepad spec</a>. | |||
</p> | |||
<pre class="idl"> | |||
[SecureContext] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[SecureContext] |
@@ -1526,7 +1526,7 @@ <h2> | |||
<pre class="idl"> | |||
[Exposed=Window] | |||
partial interface Navigator { | |||
sequence<Gamepad?> getGamepads(); | |||
[SecureContext] sequence<Gamepad?> getGamepads(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This wouldn't be web compatible. See #113.
@@ -1526,7 +1526,7 @@ <h2> | |||
<pre class="idl"> | |||
[Exposed=Window] | |||
partial interface Navigator { | |||
sequence<Gamepad?> getGamepads(); | |||
[SecureContext] sequence<Gamepad?> getGamepads(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[SecureContext] sequence<Gamepad?> getGamepads(); | |
sequence<Gamepad?> getGamepads(); |
@@ -1071,7 +1071,7 @@ <h2> | |||
feedback. | |||
</p> | |||
<pre class="idl"> | |||
[Exposed=Window] | |||
[Exposed=Window, SecureContext] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure this one makes sense either, to be honest. Half of the API is exposed in a regular context, and taking this interface away would probably break things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure any of these work, tbh.
w3c/webref
repo automatically extracts syntaxes from these spec docs. At the moment some syntax sections are missing the[SecureContext]
tags so it is missing from extracted data in webref as well.The feature has been marked available in secure context in MDN docs.
The PR adds the tags to the remaining interfaces.
💥 Error: 502 Bad Gateway 💥
PR Preview failed to build. (Last tried on Feb 5, 2024, 2:46 PM UTC).
More
PR Preview relies on a number of web services to run. There seems to be an issue with the following one:
🚨 Spec Generator - Spec Generator is the web service used to build specs that rely on ReSpec.
🔗 [Related URL]([object Object])
If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.