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

Add OpenID connect support #460

Closed
leplatrem opened this issue Feb 2, 2018 · 3 comments
Closed

Add OpenID connect support #460

leplatrem opened this issue Feb 2, 2018 · 3 comments

Comments

@leplatrem
Copy link
Contributor

leplatrem commented Feb 2, 2018

  • Read identity providers from OpenID capability (root URL)
  • Fetch ${idp}/.well-known/openid-configuration
  • Start redirection to do login «dance»
  • Read userinfo from ID token (display name top right menu)
  • Put access token in session storage
  • Use access token in Authorization: Bearer {} requests headers

Depends on Kinto/kinto#1425 (comment)

@leplatrem
Copy link
Contributor Author

leplatrem commented Feb 21, 2018

The kinto-oidc-demo demonstrates how to do it:
https://github.com/leplatrem/kinto-oidc-demo

@leplatrem
Copy link
Contributor Author

Might be interesting to have this #331

@leplatrem
Copy link
Contributor Author

Note from Julien:

On storing tokens in local storage: we're actively trying to define a
standard to follow in services, and so far it seems that tokens in local
storage with strong CSP to reduce the XSS surface is reasonable.

This might change over time (like, well, everything) but it should not
be blocking work. Just don't forget the CSP ;)

According to https://hacks.mozilla.org/2016/02/implementing-content-security-policy/ a <meta> tag is enough...

See also Kinto/kinto#1075

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

No branches or pull requests

2 participants