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

Replace [[FIDOReg]] with [[FIDO-Registry]]? #894

Closed
emlun opened this issue May 2, 2018 · 8 comments · Fixed by #897
Closed

Replace [[FIDOReg]] with [[FIDO-Registry]]? #894

emlun opened this issue May 2, 2018 · 8 comments · Fixed by #897

Comments

@emlun
Copy link
Member

emlun commented May 2, 2018

While working on #893 and adding [[FIDO-Registry]] to the bibliography, I noticed there is an existing [[FIDOReg]] biblio entry. [[FIDO-Registry]] looks to me like a successor of [[FIDOReg]].

Our only reference into [[FIDOReg]] is currently from our §10.8. User Verification Method Extension (uvm) to their Section 3.1 User Verification Methods, which is practically identical between the two. Shall we replace our references to [[FIDOReg]] with references to [[FIDO-Registry]]?

@emlun
Copy link
Member Author

emlun commented May 2, 2018

@herrjemand @gmandyam Please correct me if my assessment here is incorrect.

@equalsJeffH
Copy link
Contributor

I suspect that replacing [[FIDOReg]] with [[FIDO-Registry]] is fine. @rlin1 ?

@rlin1
Copy link
Contributor

rlin1 commented May 2, 2018

whether we call it FIDOReg or FIDO-Registry doesn't matter to me, but
today FIDOReg points to FIDO UAF Registry. Note that there also is a FIDO Registry.
All the things related to multiple specs are in FIDO Registry, the things only used in UAF are in FIDO UAF Registry.
--> We need to check whether the relevant items really are in the registry we refer to here.

@rlin1
Copy link
Contributor

rlin1 commented May 2, 2018

Matcher Protection Types, Key Protection Typs and User Verification Methods are defined in FIDO Registry - not FIDO UAF Registry.

@rlin1
Copy link
Contributor

rlin1 commented May 2, 2018

This means that (at least for the existing references) we need to change it to reference FIDO Registry.

@nadalin nadalin added this to the PR milestone May 2, 2018
@selfissued
Copy link
Contributor

The change seems good to me.

@equalsJeffH
Copy link
Contributor

thx @rlin1 !

@emlun
Copy link
Member Author

emlun commented May 4, 2018

Ah, my mistake - we do indeed have references to sections 3.2 and 3.3 as well, but those also seem to be identical between the two documents.

I'll interpret the comments here as an OK to do this.

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.

5 participants