Skip to content

Commit

Permalink
(incomplete) mention PoP token issuer for app origin
Browse files Browse the repository at this point in the history
Creating this as a placeholder, it's still incomplete in several aspects:
* we should mention somewhere why PoP token issuer is better than Origin header (namely because of solid/web-access-control-spec#34)
* we should mention that a PoP token is only valid if the issuer is listed in the id token's audiences (is this something the IDP checks and then signs for? I don't even know)
  • Loading branch information
michielbdejong authored Jun 18, 2019
1 parent 2b2c5a3 commit 0310703
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ For example, here is what happens when Alice tries to request the resource
example, `alice.solidtest.space`), or entering her email.
3. [Local
Authentication](example-workflow.md#3-local-authentication-to-provider):
Alice gets redirected towards her service provider's own Sign In page, thus requesting
Alice gets redirected towards her identity provider's own Sign In page, thus requesting
`https://alice.solidtest.space/signin`, and authenticates using her preferred
method (password, WebID-TLS certificate, FIDO 2 /
[WebAuthn](https://w3c.github.io/webauthn/) device, etc).
Expand All @@ -137,6 +137,9 @@ For example, here is what happens when Alice tries to request the resource
`bob.example` (the server controlling the resource) validates the ID Token, and
extracts Alice's WebID URI from inside it. She is now signed in to
`bob.example` as user `https://alice.solidtest.space/#i`.
7. Deriving an app origin:
`bob.example` (the server controlling the resource) can use the PoP token's
issuer to establish the web app's origin.
7. [WebID Provider Confirmation](#webid-provider-confirmation):
`bob.example` confirms that `solidtest.space` is indeed Alice's authorized OIDC
provider (by matching the provider URI from the `iss` claim with Alice's
Expand Down

0 comments on commit 0310703

Please sign in to comment.