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

leverage "credential source" term from credential management spec #430

Closed
equalsJeffH opened this issue Apr 27, 2017 · 4 comments
Closed

Comments

@equalsJeffH
Copy link
Contributor

when we say something like:

If there is any credential |X| present on this |authenticator| that:

"credential" here is actually what is presently termed a "credential source" in the credential management spec.

presently, this is a wide-ranging conflation in the webauthn spec and should be addressed. This may involve aligning the webauthn and credman specs' notions of "credential".

h/t @jyasskin

@jyasskin
Copy link
Member

To be clear, the "credential source" term in Credential Management is new and could change if y'all have a better suggestion. The important bit is to distinguish the credential, which is actually presented to the RP, from the thing that can generate credentials, which is locked inside a secure element.

There may be a third concept—the identifier, handle, or descriptor for a credential source—but that's not formally described yet.

@equalsJeffH
Copy link
Contributor Author

@jyasskin wrote:

The important bit is to distinguish the credential, which is actually presented to the RP, from the thing that can generate credentials, which is locked inside a secure element.

Agreed. Note that scoped credential's definition aligns with that, although the term used in RFC4949 for what credman is (presently) terming "credential source" is "authentication information" (see the "tutorial" portion of the latter reference).

"credential source" is offhand fine by me, but perhaps credman could reference RFC4949's definition and term as a footnote/sidebar/whatever.

though, a credential source (aka authn info) may or may not be "locked inside a secure element". whether it is or not is yet another facet of all of this...

@AngeloKai
Copy link
Contributor

Given that there's no API renaming proposal in this issue, I am taking out the renaming label.

@nadalin nadalin modified the milestones: CR, PR Sep 14, 2017
@equalsJeffH
Copy link
Contributor Author

@jyasskin fixed this with PR #620 which is merged. closing.

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

4 participants