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

Update Authenticator Definition #1931

Merged
merged 2 commits into from
Aug 16, 2023
Merged

Update Authenticator Definition #1931

merged 2 commits into from
Aug 16, 2023

Conversation

nicksteele
Copy link
Contributor

@nicksteele nicksteele commented Jul 26, 2023 •
edited by pr-preview bot

This addresses #1743 by updating the authenticator definition to denote that an authenticator does not have to be wholly local without going into more detail on concepts like an identity fabric.

Preview | Diff

@timcappalli
Copy link
Member

a local context, and can generate or store a [=credential key pair=] in a server outside of the [=client device=].

In the case of hybrid, the authenticator is not part of the client device. I think this gets into the authenticator boundary discussion. I don't have a suggestion as to what to put instead of client device, but I don't think it should be "client device".

Anyone else have suggestions?

emlun
emlun reviewed Jul 26, 2023
View reviewed changes
index.bs Show resolved Hide resolved
@emlun
Copy link
Member

emlun commented Jul 26, 2023

In the case of hybrid, the authenticator is not part of the client device. I think this gets into the authenticator boundary discussion. I don't have a suggestion as to what to put instead of client device, but I don't think it should be "client device".

Anyone else have suggestions?

Perhaps something like outside of [=client-side=] hardware?

Client-Side
This refers in general to the combination of the user’s client platform, authenticators, and everything gluing it all together.

timcappalli
timcappalli approved these changes Jul 27, 2023
View reviewed changes
Copy link
Member

@timcappalli timcappalli left a comment

Choose a reason for hiding this comment

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

LGTM

@nicksteele nicksteele merged commit 84ef4ac into main Aug 16, 2023
2 checks passed
github-actions bot added a commit that referenced this pull request Aug 16, 2023
@nicksteele @github-actions
Merge pull request #1931 from w3c/nsteele-authntctr-def-1743
783d711 
SHA: 84ef4ac
Reason: push, by nicksteele

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@MasterKale MasterKale mentioned this pull request Aug 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emlun emlun emlun approved these changes

@MasterKale MasterKale MasterKale approved these changes

@timcappalli timcappalli timcappalli approved these changes

@akshayku akshayku akshayku approved these changes

Assignees

@nicksteele nicksteele

Labels
type:editorial
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@nicksteele @timcappalli @emlun @MasterKale @akshayku