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
Conversation
..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.
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>
There was a problem hiding this 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
There was a problem hiding this 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. :)
Co-Authored-By: Emil Lundberg <emil@yubico.com> Co-Authored-By: John Bradley <ve7jtb@ve7jtb.com>
..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