Skip to content
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

Flag Direct Sockets API as pending #1553

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Flag Direct Sockets API as pending #1553

merged 1 commit into from
Nov 8, 2024

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Nov 8, 2024

The API used to have a "pending" standing because its status was "Unofficial Draft Proposal". That seems good (and the proposal has a negative standard position from Mozilla). The status of the spec changed to "Draft Community Group Report" recently, which was enough to make our code switch the standing to "good".

I missed that change when I reviewed the diff of web-specs and released a version of web-specs with that updated standing. This means Webref now tries to curate the IDL defined in the spec, and fails because the IDL references the [IsolatedContext] extended attribute that currently exists as a monkey patch in the Isolated Contexts spec.

We may of course update Strudy to accept [IsolatedContext] but I'm thinking that the right standing for the spec remains "pending" for now. This update forces the standing accordingly.

The API used to have a "pending" standing because its status was "Unofficial
Draft Proposal". That seems good (and the proposal has a negative standard
position from Mozilla). The status of the spec changed to "Draft Community
Group Report" recently, which was enough to make our code switch the standing
to "good".

I missed that change when I reviewed the diff of web-specs and released a
version of web-specs with that updated standing. This means Webref now tries
to curate the IDL defined in the spec, and fails because the IDL references the
`[IsolatedContext]` extended attribute that currently exists as a monkey patch
in the Isolated Contexts spec.

We may of course update Strudy to accept `[IsolatedContext]` but I'm thinking
that the right standing for the spec remains "pending" for now. This update
forces the standing accordingly.
@tidoust tidoust merged commit 10a9a70 into main Nov 8, 2024
1 check passed
@tidoust tidoust deleted the direct-sockets branch November 8, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants