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
Add SAML/OIDC comparison by David Chadwick #74
Conversation
comparison.html
Outdated
</p> | ||
|
||
<p> | ||
The FIM trust model is wrong as it requires the Identity Provider (IdP) |
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.
Let's strike "is wrong as it". It's a subjective value judgment that detracts from the point.
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.
Accepted
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.
Fixed.
comparison.html
Outdated
</section> | ||
|
||
<section> | ||
<h1>Comparison of Verificable Credentials with SAML and OIDC</h1> |
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.
small typo: "Verificable" -> "Verifiable"
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.
Fixed.
As a novice to SAML and OIDC I would have no idea that this was a comparison since they are referenced but not mentioned by name. Rather, this reads to me like one of the most compact explanations of why we are doing the VC work that I have ever read. I was surprised to find it a separate doc and not part of our spec. The text itself is wonderful and I am willing to accept it as-is to get something in place, but I do have the two concerns above. |
Add a comparison between SAML and OpenID Connect done by @David-Chadwick.