{"action":"created","issue":{"url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/50","labels_url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/50/labels{/name}","comments_url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/50/comments","events_url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/50/events","html_url":"https://github.com/IdentityServer/IdentityServer3.AspNetIdentity/issues/50","id":104702962,"number":50,"title":"[Question] AspNetIdentityUserService - AuthenticateExternalAsync","user":{"login":"pawepaw","id":11609029,"avatar_url":"https://avatars.githubusercontent.com/u/11609029?v=3","gravatar_id":"","url":"https://api.github.com/users/pawepaw","html_url":"https://github.com/pawepaw","followers_url":"https://api.github.com/users/pawepaw/followers","following_url":"https://api.github.com/users/pawepaw/following{/other_user}","gists_url":"https://api.github.com/users/pawepaw/gists{/gist_id}","starred_url":"https://api.github.com/users/pawepaw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pawepaw/subscriptions","organizations_url":"https://api.github.com/users/pawepaw/orgs","repos_url":"https://api.github.com/users/pawepaw/repos","events_url":"https://api.github.com/users/pawepaw/events{/privacy}","received_events_url":"https://api.github.com/users/pawepaw/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-03T14:07:19Z","updated_at":"2015-11-20T13:46:28Z","closed_at":null,"body":"It's rather question than issue and I understand that AspNetIdentityUserService is just sample implementation which I can extend or replace but I want to understand better current implementation.\r\n\r\nI see that you save claims from external identity provider when user is for the first time in application and he doesn't exist in database but when user'll visit again you do not update, save new claims nor get old from database. Question is why do we save them?\r\n\r\nWhen user visit for the first time his account is created based on received claims. I understand that you save claims because some of them might be used later on user active session?\r\n\r\nI understand that every time user gets redirected from external identity provider into IdentityServer claims are attached into request (token) so it might be wise to update claims in database because they will change over the time?"},"comment":{"url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/comments/158407010","html_url":"https://github.com/IdentityServer/IdentityServer3.AspNetIdentity/issues/50#issuecomment-158407010","issue_url":"https://api.github.com/repos/IdentityServer/IdentityServer3.AspNetIdentity/issues/50","id":158407010,"user":{"login":"brockallen","id":649492,"avatar_url":"https://avatars.githubusercontent.com/u/649492?v=3","gravatar_id":"","url":"https://api.github.com/users/brockallen","html_url":"https://github.com/brockallen","followers_url":"https://api.github.com/users/brockallen/followers","following_url":"https://api.github.com/users/brockallen/following{/other_user}","gists_url":"https://api.github.com/users/brockallen/gists{/gist_id}","starred_url":"https://api.github.com/users/brockallen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brockallen/subscriptions","organizations_url":"https://api.github.com/users/brockallen/orgs","repos_url":"https://api.github.com/users/brockallen/repos","events_url":"https://api.github.com/users/brockallen/events{/privacy}","received_events_url":"https://api.github.com/users/brockallen/received_events","type":"User","site_admin":false},"created_at":"2015-11-20T13:46:28Z","updated_at":"2015-11-20T13:46:28Z","body":"Well, it's a tricky question -- do you always want those external claims? Some people do, some don't. So you have to decide and code the your solution the way you need it."}}