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

update FIDO spec references to link to latest revisions #898

Closed
equalsJeffH opened this issue May 8, 2018 · 4 comments · Fixed by #913
Closed

update FIDO spec references to link to latest revisions #898

equalsJeffH opened this issue May 8, 2018 · 4 comments · Fixed by #913

Comments

@equalsJeffH
Copy link
Contributor

for all FIDO spec refs in the webauthn spec, if the ref'd FIDO spec can be found amongst the specs presented in https://fidoalliance.org/specs/fido-v2.0-id-20180227/, then update the ref in webauthn spec to point to the FIDO spec in the latter collection.

@equalsJeffH equalsJeffH added this to the PR milestone May 8, 2018
@emlun emlun self-assigned this May 16, 2018
@emlun
Copy link
Member

emlun commented May 16, 2018

  1. @rlin1 This seems like a newer version of [FIDOEcdaaAlgorithm], correct?
  2. [FIDOMetadataService] is an informative reference, so there should be no problem updating that to this, right? The two look like they have largely the same contents, but with some differences.
  3. [FIDOSecRef] is an informative reference, so there should be no problem updating that to this, right? The two look like they have largely the same contents, but with some differences - the latter is much larger.

That looks to me like the only FIDO documents that have newer versions in https://fidoalliance.org/specs/fido-v2.0-id-20180227/ .

@rlin1
Copy link
Contributor

rlin1 commented May 16, 2018

regarding 1. yes, that is a newer version for FIDO ECDAA we should update the reference to that one.
regarding 2. Should also be updated and there is even a newer version in the works for that.
regarding 3. yes, should update reference to FIDO Security Ref as well.

@emlun
Copy link
Member

emlun commented May 16, 2018

Note also that all three of the mentioned newer versions are Implementation Drafts (although the existing [FIDOEcdaaAlgorithm] reference is also an Implementation Draft). Is that an issue?

@equalsJeffH
Copy link
Contributor Author

WRT the FIDO specs being impl-drafts, I do not think is issue at this time, for CR or PropRec. They are moving targets, as is webauthn. Once webauthn going to Rec maturity level, we nominally ought to be referring to PropStd (PS) FIDO specs.

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

Successfully merging a pull request may close this issue.

3 participants