Skip to content
This repository has been archived by the owner on Mar 19, 2019. It is now read-only.

AuthenticationHandler should use WindowsPrincipal instead of ClaimsPrincipal #306

Closed
kevinchalet opened this issue Feb 2, 2017 · 0 comments
Assignees

Comments

@kevinchalet
Copy link

Unlike IIS's authentication handler, HttpSysServer doesn't use WindowsPrincipal and directly relies on the ClaimsPrincipal base class to represent non-empty principals. This inconsistency causes weird issues when using IsInRole(...) with HttpSysServer, as ClaimsPrincipal.IsInRole(...) doesn't have the additional logic implemented by WindowsPrincipal.IsInRole(...).

This line should be changed to use WindowsPrincipal instead of ClaimsPrincipal.

/cc @Tratcher

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