Get name of currently logged in user

Question

I made a simple login using Identity. I logged in with signInManager.PasswordSignInAsync().

If the login was successful, I would like to get the currently logged in user's data using the userManager.

The problem is that I can only get an id from the GetUserAsync and don't know what to do with it.

var result = await mSignInManager.PasswordSignInAsync(username, password, true, false);

if (result.Succeeded)
{
    //var loggedUser = mUserManager.GetUserName(HttpContext.User);
    var userData = mUserManager.GetUserAsync(HttpContext.User);
    var user = await mUserManager.FindByIdAsync(userData.Id.ToString());
    return Redirect("/createpost");
}

userData is{ Id = 317, Status = RanToCompletion, Method = "{null}", Result = "" } , user is null — Andrea, May 29 '19 at 09:50
He asked you what is the type which means what type of object you are expecting to those variables? — Samaritan_Learner, May 29 '19 at 10:03
What all are the user details you are expecting, Please elaborate that so that others can help you on what exactly you need — Samaritan_Learner, May 29 '19 at 10:04
ApplicationUser, or some properties from it, like username,email — Andrea, May 29 '19 at 10:05
@Andrea you're calling async methods but never await the results. You should use `var userData = mUserManager.GetUserAsync(HttpContext.User);` — Panagiotis Kanavos, May 29 '19 at 11:16
The missing `await` is only part of the problem. The other part is covered by the possible duplicate. — Kirk Larkin, May 29 '19 at 11:22

Martin · Accepted Answer · 2019-05-29T11:42:43.800

0

To me it looks like you are logging the user in and immediately trying to access the user info. The httpContext has not had the chance to update and still reflects the time when there was no user logged in. Try moving your user info retrieval logic to "createpost" where it gets executed on the next http roundtrip.

    public async Task<IActionResult> createpost()
    {
        var userData = await mUserManager.GetUserAsync(HttpContext.User);
        Debug.WriteLine(userData.Id);
        Debug.WriteLine(userData.UserName);
        Debug.WriteLine(userData.Email);
        return View();
    }

edited May 29 '19 at 11:42

answered May 29 '19 at 11:33

Martin

87
1
5

If that does not work, make sure that your authentication system works. You could, for example, decorate the createpost() action with the [Authorize] attribute. If your current user can execute the createpost() action, you should be able to access the user data exactly as shown. – Martin May 30 '19 at 09:23
thanks for your answer, it helped me, mUserManager.GetUserName(HttpContext.User); after that I use the logged username to access from database – Andrea May 31 '19 at 21:33

Chris Pratt · Answer 2 · 2019-05-29T14:59:21.417

You've got two issues. First, GetUserAsync relies on the principal being set in HttpContext, which hasn't occurred yet, so that method will fail. There must be an intervening request before you can access the user that way.

What you can do is pull the user out directly using something like FindByIdAsync, which you're attempting to do already. However, you're trying to find the user based on the id of the user instance you tried to retrieve via GetUserAsync. Since that failed, there will be no user, and thus no id to use in this lookup, causing it to fail as well. It doesn't even make sense to do this, as if you had the user already to get the id from, there'd be no point in looking that user up by its id.

So, what you need to do is look the user up by information you actually do have, which based on your code, is username. Therefore, do:

var user = await mUserManager.FindByNameAsync(username);

score 0 · Answer 3 · answered May 30 '19 at 02:50

0

The reason is that the user will be available for next request, you could not get the user in this current request,let alone user's name.

answered May 30 '19 at 02:50

Ryan

19,118
10
37
53

Get name of currently logged in user

3 Answers3