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

Update Proposed-WG-FederatedIdentity.md #21

Merged
merged 2 commits into from
Dec 19, 2023
Merged

Conversation

hlflanagan
Copy link
Contributor

Revised based on horizontal review, direct feedback

Revised based on horizontal review, direct feedback
Copy link

@samuelgoto samuelgoto left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -48,13 +49,15 @@ specifications:

- Federated Credential Management API
- Login Status API
- Web platform tests

Choose a reason for hiding this comment

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

Suggested change
- Web platform tests
- Web Platform Tests

WPTs is a well-known acronym, so figured that it would be useful to capitalize

@achimschloss achimschloss self-requested a review December 13, 2023 17:18

The *W3C Web Identity Credential Working Group* will develop living standard specifications defining an API that allows websites to request a federated identity credential or assertion with the purpose of authenticating a user and/or requesting a set of claims in a compatible way OIDC or SAML.
The *W3C Federated Identity Working Group* will develop living standard specifications defining an API that allows websites to request a federated identity credential or assertion with the purpose of authenticating a user and/or requesting a set of claims in a compatible way OIDC or SAML.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The *W3C Federated Identity Working Group* will develop living standard specifications defining an API that allows websites to request a federated identity credential or assertion with the purpose of authenticating a user and/or requesting a set of claims in a compatible way OIDC or SAML.
The *W3C Federated Identity Working Group* will develop living standard specifications defining an API that allows websites to request a federated identity credential or assertion with the purpose of authenticating a user and/or requesting a set of claims in a way compatible with OIDC or SAML.

@@ -1,6 +1,6 @@
# PROPOSED Web Identity Credential Working Group Charter
# PROPOSED Federated Identity Working Group Charter
Copy link
Contributor

@TallTed TallTed Dec 13, 2023

Choose a reason for hiding this comment

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

Perhaps should change the title of this PR (and the draft .md document) to match the above?

[Web Authentication (WebAuthn) Working Group](https://www.w3.org/groups/wg/webauthn/)

While we are not developing an authentication mechanism, it still must work in conjunction with existing authentication mechanisms. The WebAuthn Working Group may provide input and guidance for this requirement.

Copy link
Contributor

Choose a reason for hiding this comment

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

It's worth noting that the WebID CG is aiming to charter a WebID WG in the near future, so it's good that this group's proposed name is changing from Web Identity Credentials to Federated Identity. Also, the WebID CG/WG work should be kept in mind as complementary and/or conflicting and/or redundant, etc., depending on overall timing....

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 based on discussions on the recent CG calls

@hlflanagan hlflanagan merged commit c7bcb18 into main Dec 19, 2023
@hlflanagan hlflanagan changed the title Update Proposed-WG-WebIdentityCredentials.md Update Proposed-WG-FederatedIdentity.md Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants