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 option to request renewing user data #855

Closed
valdian opened this issue Sep 15, 2020 · 4 comments
Closed

Add option to request renewing user data #855

valdian opened this issue Sep 15, 2020 · 4 comments

Comments

@valdian
Copy link
Contributor

valdian commented Sep 15, 2020

Is your feature request related to a problem? Please describe.
In my application, a user can update his profile. After he submits the form, his claims in the oidc provider (IdentityServer4) database are updated and the user data in the browser storage becomes outdated.

Therefore, I need to be able to request renewing user data (i.e. call userinfo endpoint of the oidc provider) after the profile update. In version 10, there was a method called getUserinfo(), which I used for it. In version 11 (before 11.1.4), I use forceRefreshSession() to renew tokens as well as user data. Since #812, there is no way to do it.

Describe the solution you'd like
Add a parameter renewUserInfoAfterTokenRenew to OpenIdConfiguration and if it is set to true, then renew user data after a token renew.

Describe alternatives you've considered

  1. Refresh user data always after token renew, as it was done in version 11.1.3.
  2. Add a boolean parameter shouldRefreshUserData to the forceRefreshSession() method in OidcSecurityService.

Additional context
I will create a PR to demonstrate my idea but I am open also to other ways of solving my problem.

@damienbod
Copy link
Owner

damienbod commented Sep 17, 2020

I think this is good, thanks!

@valdian
Copy link
Contributor Author

valdian commented Oct 1, 2020

Thanks for approving.
@damienbod When do you plan the next release?

@emresalum1993
Copy link

Any eta for next release?

@valdian valdian closed this as completed Oct 27, 2020
@damienbod
Copy link
Owner

Hi @emresalum1993 This has been released in version 11.2.1

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

No branches or pull requests

3 participants