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

use '(client-side) discoverable credential' terminology #1398

Merged
merged 8 commits into from Apr 27, 2020

Conversation

equalsJeffH
Copy link
Contributor

@equalsJeffH equalsJeffH commented Apr 1, 2020

..rather than the 'resident credential' and 'resident key' terms. Also changed 'non-resident credential' to 'server-side credential', along with other related fixups. Marked the former terms as DEPRECATED.

fixes #1379 (may only "improve" it due to not addressing the notion of "forbidden to create a client-side cred", though the latter perhaps ought to be in its own issue separate from #1379)


Preview | Diff

..rather than the 'resident credential' and 'resident key' terms.  Also changed 'non-resident credential' to 'server-side credential', along with other related fixups. Marked the latter terms as DEPRECATED.
@equalsJeffH equalsJeffH added this to the L2-WD-03 milestone Apr 1, 2020
@equalsJeffH equalsJeffH self-assigned this Apr 1, 2020
@equalsJeffH equalsJeffH requested a review from ve7jtb April 1, 2020 21:18
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
equalsJeffH and others added 5 commits April 25, 2020 11:38
Co-Authored-By: Emil Lundberg <emil@yubico.com>
Co-Authored-By: Emil Lundberg <emil@yubico.com>
Co-Authored-By: Emil Lundberg <emil@yubico.com>
thx emlun!

Co-Authored-By: Emil Lundberg <emil@yubico.com>
Copy link
Contributor

@ve7jtb ve7jtb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a slight preference for Client-side-discoverable vs Client-side discoverable.
But that is just me.
I suggested getting rid of discoverable in the server-side Public key credential source

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
Copy link
Member

@emlun emlun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! See comments for some thoughts on minor word polish. :)

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
Co-Authored-By: Emil Lundberg <emil@yubico.com>
Co-Authored-By: John Bradley <ve7jtb@ve7jtb.com>
@equalsJeffH
Copy link
Contributor Author

Ok, I've accepted @emlun & @ve7jtb's comments, merging....

@equalsJeffH equalsJeffH merged commit 8d0060a into master Apr 27, 2020
WebAuthnBot pushed a commit that referenced this pull request Apr 27, 2020
WebAuthnBot pushed a commit that referenced this pull request Apr 27, 2020
@emlun emlun deleted the jeffh-fix-1379-discoverable-cred branch June 22, 2022 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

factor-in the notion of client-side discoverable cred and revise & deprecate the "resident cred"
5 participants