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

UserInfo should be TokenInfo #32

Open
tomreay opened this issue Oct 16, 2018 · 1 comment
Open

UserInfo should be TokenInfo #32

tomreay opened this issue Oct 16, 2018 · 1 comment
Labels
enhancement New feature or request

Comments

@tomreay
Copy link
Contributor

tomreay commented Oct 16, 2018

During the implementation of #30 it was decided that the user info endpoint should become a token info endpoint.

Questions:

  1. Is it intended that this be the endpoint described here
  2. How will auth be handled for this endpoint? Should it require a bearer token with a token_info scope?
  1. How many of the optional response fields should be supported? I think all.
@adhesivee
Copy link
Collaborator

I have introduced the userInfo endpoint, because with RFC6749 there is no standard way of doing this. But still wanted to provide an endpoint that some basic user information is visible. I have seen RFC7662, but haven't taken the time to really dive into it.

To answer your questions:

  1. It is not intended to be like RFC7662 altough I do think it could be a good option to implement
  2. If not using RFC7662, then it should use bearer token and scopes should be decided by the implementation (default no scopes). But I am not sure yet how to provide this in a clear way. Perhaps something as passing CallRouter to the userInfoCallback and create an extension method for CallRouter, like respondInvalidSopces or missingScopes which responds in the correct way of oauth2. This gives the implementor more flexibility if they want to deny on scopes or something else.
  3. With optional response fields are you referring to this. If so, then yes all

@adhesivee adhesivee added the enhancement New feature or request label Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants