Skip to content
This repository has been archived by the owner on Sep 18, 2021. It is now read-only.

Add interface to validate incoming identity #2148

Closed
brockallen opened this issue Nov 9, 2015 · 8 comments
Closed

Add interface to validate incoming identity #2148

brockallen opened this issue Nov 9, 2015 · 8 comments
Assignees
Milestone

Comments

@brockallen
Copy link
Member

To be used from the CookieAuthenticationProvider on our main cookie middleware to reject incoming cookie.

@brockallen
Copy link
Member Author

Would it be acceptable to use the user service's IsActiveAsync for this? Or does that conflate it too much?

// @leastprivilege @snothub

@brockallen
Copy link
Member Author

Actually, I think I've already come to a conclusion -- this is conflating IsActive, which is supposed to mean "is user allowed to get tokens at all", which is different than "is the current authentication session valid". The latter seems to be what the semantics of this feature should be.

@snothub
Copy link

snothub commented Jan 17, 2016

Completely agree with the last one.

søn. 17. jan. 2016, 22.51 skrev Brock Allen notifications@github.com:

Actually, I think I've already come to a conclusion -- this is conflating
IsActive, which is supposed to mean "is user allowed to get tokens at
all", which is different than "is the current authentication session
valid". The latter seems to be what the semantics of this feature should be.


Reply to this email directly or view it on GitHub
#2148 (comment)
.

@brockallen
Copy link
Member Author

Added IAuthenticationSessionValidator to the identity server services factory on dev branch.

@leastprivilege
Copy link
Member

also open a doc issue.

@snothub
Copy link

snothub commented Jan 28, 2016

So how is this supposed to work then? When will this code hit and should I be looking for (or set) any specific claim in the principle?

@brockallen
Copy link
Member Author

When you validate the user's identity in the user service you can emit any claims you want into the cookie for IdSvr. These claims will be present in this check. You so whatever you want in this API to say yes or no.

@snothub
Copy link

snothub commented Jan 28, 2016

Awesome!

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

No branches or pull requests

3 participants