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.

Sign up for GitHub

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

Jump to bottom

Conflicting definition of AlgorithmIdentifier #526

Closed
dontcallmedom opened this issue Aug 9, 2017 · 2 comments
Closed

Conflicting definition of AlgorithmIdentifier #526

dontcallmedom opened this issue Aug 9, 2017 · 2 comments
Labels
subtype:renaming type:editorial
Milestone
WD-07

Comments

@dontcallmedom
Copy link
Member

AlgorithmIdentifier is defined in Web Crypto as typedef (object or DOMString) AlgorithmIdentifier; the latest editor draft of WebAuthn redefines it as typedef (short or ByteString) AlgorithmIdentifier - is it meant to override the Web Crypto definition? if not, a different name should be used.
@selfissued selfissued added this to the WD-07 milestone Aug 9, 2017
@selfissued
Copy link
Contributor

What do people think of the name COSEAlgId?

@selfissued selfissued mentioned this issue Aug 9, 2017
Merged
@selfissued
Copy link
Contributor

This was fixed by PR #531 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
subtype:renaming type:editorial
Projects
None yet
Milestone
WD-07
Development

No branches or pull requests
2 participants
@dontcallmedom @selfissued