Skip to content
This repository has been archived by the owner on Apr 13, 2022. It is now read-only.

Removed references to TLS as primary auth method #171

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jaxoncreed
Copy link

TLS as a primary form of authentication should be deprecated in favor of a oidc. TLS may still be used as a form of credentials under oidc. All concerns about needing completely decentralized identity systems will be solved with an eventual implementation of DiD.

@RubenVerborgh
Copy link
Contributor

Not opposed, just checking if this has been checked with @timbl?

@jaxoncreed
Copy link
Author

@RubenVerborgh I'm going to submit this for discussion at the w3c working group meeting too, so we'll have plenty of time for input.

michielbdejong
michielbdejong previously approved these changes May 14, 2019
Copy link
Contributor

@michielbdejong michielbdejong left a comment

Choose a reason for hiding this comment

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

LGTM, but agree with Ruben that @timbl should look at this before we merge.

@elf-pavlik
Copy link
Member

👍 and if this PR will not land this month it should at least get marked with at risk

@michielbdejong
Copy link
Contributor

Feedback we got from @timbl and @kidehen (in my own words): webid-tls should still be mentioned as a 'MAY' for client-storage, and as a 'SHOULD' for user-IDP login method ('openid + TLS bridge')

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@@ -97,9 +97,6 @@ statements:
to make the display of the user's contributions identifiable.
3. A profile MAY provide a `foaf:nick` nickname as a short string for use by user interfaces where
space is limited.
3. A profile SHOULD include `cert:key` public key certificate information, for
use with WebID+TLS (which is currently the primary Solid authentication
mechanism).
Copy link
Contributor

Choose a reason for hiding this comment

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

MAY

Copy link
Contributor

@michielbdejong michielbdejong left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants