Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

Update UserController.cs #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarkSPowell
Copy link

Chaning CreateNewUser to use the samAccountName instead of the email address for the LoginId of a new user. This matches what the LoginController uses to look up a user. Previously, this would only work if a users samAccountName matched their email address- this is not allowed in highly regulated enviroments, as it discloses users samAccountName.

Chaning CreateNewUser to use the samAccountName instead of the email address for the LoginId of a new user. This matches what the LoginController uses to look up a user. Previously, this would only work if a users samAccountName matched their email address- this is not allowed in highly regulated enviroments, as it discloses users samAccountName.
@MarkSPowell
Copy link
Author

This commit fixes #14

@IonicEcko
Copy link

Tried your edit, if an admin creates an account trough /#/User the admin's username(who created the account) is inserted into the DB column LoginID.
Did the below instread, it does the thing.

List userlist = GetUserFromDifferentDomain(user.User.UserName);
var logonId = userlist[0].UserName;
//var logonId = User.Identity.Name;
//var Index = logonId.Split('');
//logonId = Index[1];

Am having the same issue, how did you resolve? Just those 2 lines error.

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

Successfully merging this pull request may close these issues.

4 participants