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

Candidate recommendation exit criteria #837

Closed
samuelweiler opened this issue Mar 13, 2018 · 18 comments
Closed

Candidate recommendation exit criteria #837

samuelweiler opened this issue Mar 13, 2018 · 18 comments
Assignees
Milestone

Comments

@samuelweiler
Copy link
Member

The Director expects CR documents to explicitly state the criteria for getting to PR. We can set criteria of our choice, but the suggestion is: "In order for this specification to exit Candidate Recommendation there must be demonstrations of at least two interoperable implementations of every feature."

The general rule is that CRs "must document how adequate implementation experience will be demonstrated" and that in moving to PR we "must show adequate implementation experience except where an exception is approved by the Director." So we need to carefully set a standard that we expect to be able to meet.

We may identify features "at risk" or with different criteria than the bulk.

@samuelweiler samuelweiler added this to the CR milestone Mar 13, 2018
@samuelweiler
Copy link
Member Author

Some of this is from https://www.w3.org/2018/Process-20180201/#candidate-rec

@samuelweiler
Copy link
Member Author

here's an example: https://www.w3.org/TR/2014/CR-WebCryptoAPI-20141211/

For this specification to be advanced to Proposed Recommendation, there must be at least two independent, interoperable implementations of each feature.

Each cryptographic algorithm listed in the specification is currently an "at risk" feature. In particular, we are expecting a "browser profile" to be created of interoperable algorithms for Web browsers. To exit to Proposed Recommendation, each algorithm listed in the specification should have two interoperable implementations. Furthermore, there is an open dependency on the IRTF CFRG in terms of algorithm(s). These issues should be resolved by the interoperability testing necessary to exit CR.

@emlun
Copy link
Member

emlun commented Mar 14, 2018

Are the extensions included in those features? I.e., does every extension need to be implemented by at least two independent browsers and (where applicable) authenticators?

@samuelweiler
Copy link
Member Author

I suggest we mark all extensions as "at risk" or at least set a lower bar for them.

@nadalin
Copy link
Contributor

nadalin commented Mar 14, 2018 via email

@equalsJeffH
Copy link
Contributor

seems to me since this is "exit (CR) criteria" I'd attach it to the PR milestone....

@samuelweiler
Copy link
Member Author

JeffH: We need this in the CR draft.

Here's another example: https://www.w3.org/TR/wake-lock/

@samuelweiler
Copy link
Member Author

FYI, you might need to add this include file via a PR to:
https://github.com/tabatkins/bikeshed/tree/master/bikeshed/boilerplate/webauthn

@nadalin
Copy link
Contributor

nadalin commented Mar 14, 2018

"For the Web Authentication specification to move to Proposed Recommendation we must show two independent, interoperable implementations of the Web Authentication API in browsers. We have already completed a portion of this work at two informal interops with implementations from three browsers (Chrome, Firefox, and Edge). These initial compliance tests have yielded relevant datasets. We also have a suite of initial compliance tests that have been committed to the W3C Web Platform Test repo and additional improvements to the tests will be required for the move to Proposed Recommendation."

@selfissued
Copy link
Contributor

selfissued commented Mar 14, 2018

Per the call, I am to add a sentence to this description about the extensions framework and having multiple interoperable implementations of at least one extension to validate the extensions framework.

@nadalin
Copy link
Contributor

nadalin commented Mar 14, 2018

yes

@selfissued
Copy link
Contributor

All extensions are optional and all but AppID are at risk.

@samuelweiler
Copy link
Member Author

"All extensions are optional. All except for appid are at risk."

@samuelweiler
Copy link
Member Author

samuelweiler commented Mar 18, 2018

Consistent with https://lists.w3.org/Archives/Public/public-webauthn/2018Mar/0143.html, I propose the following text for the CR. Absent objection, I'll include this in the CR as published, and we'll need to backport it into the github source later:

For the Web Authentication specification to move to Proposed Recommendation we must show two independent, interoperable implementations of the Web Authentication API in browsers. We will also have multiple interoperable implementations of the AppID extension, validating the extensions framework. All other extensions are "at risk". If there are not multiple interoperable implementations, each may independently be removed or made informative at Proposed Recommendation.

We have had two informal interoperability tests with implementations in three browsers.

I observe that the "status of this doc" changes don't seem to be committed to either our repo nor the bikeshed repo, so my guess is that those changes are only in Angelo's local bikeshed environment. We need to get those committed at some point.

@selfissued
Copy link
Contributor

@nadalin to talk with @AngeloKai about addressing this.

@AngeloKai
Copy link
Contributor

@samuelweiler I just filed a PR for the CR include: speced/bikeshed#1285

@AngeloKai
Copy link
Contributor

@samuelweiler can you please help draft the PR include? I started a PR here: speced/bikeshed#1286. But feel free to update it as you see fit. Same goes for @nadalin

@AngeloKai
Copy link
Contributor

@samuelweiler The CR include was just merged. Please take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants