-
Notifications
You must be signed in to change notification settings - Fork 136
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 mention that subject could get claims from server side sessions. #507
Conversation
Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-ocean-048b8e60f-507.eastus2.azurestaticapps.net |
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.
I think we should incorporate this into the previous paragraph to make which claims you'll get more obvious. If server side sessions are on, we always use that. If they're off, then we give you different claims depending on the context.
And maybe we can explain the the claims identity at the time of grant creation is stored in the grant store. So if you are in the profile service in the context of using a grant (exchanging a code for a token, etc), we can give you those claims, but if you are in the user info endpoint, we don't have anything except what is in the access token sent to the user info endpoint. |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-ocean-048b8e60f-507.eastus2.azurestaticapps.net |
@josephdecock I've used bullets now to create an if..then structure which should be more clear. |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-ocean-048b8e60f-507.eastus2.azurestaticapps.net |
IdentityServer/v7/docs/content/reference/services/profile_service.md
Outdated
Show resolved
Hide resolved
…ice.md Co-authored-by: Joe DeCock <[email protected]>
@josephdecock Agreed. |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://happy-ocean-048b8e60f-507.eastus2.azurestaticapps.net |
No description provided.