Por qué es Usuario (como en User.Identity.Name) null en mi controlador base abstracto?

La respuesta a este problema es bastante simple. No puedo ejecutar el código desde dentro del constructor por razones señaladas por Raimond, pero puedo hacerlo fuera del constructor.

Así que lo que hice fue anular OnActionExecuting() en la clase controller base (Creé un atributo personalizado para ella, pero simplemente anular el método también debería funcionar) y luego hacer mi búsqueda de usuario desde allí.

Ahora funciona como se esperaba y no tengo código repetido.

La propiedad de usuario no se asigna hasta después de que se haya instanciado el Controlador, pero puede obtener acceso anticipado desde su constructor con:

System.Web.HttpContext.Current.User

Puedes agarrar esto usando algo como:

HttpContext currentContext = HttpContext.Current;
string userName = currentContext.User.Identity.Name;

¿O el HttpContext siempre está vacío??

¿Podría establecer el HttpContext a través del constructor de la clase abstracta? y usarlo de esta manera?

Gracias Raimond. Estaba demasiado cansado para ver lo obvio. @Keeney: Sí, el contexto siempre es nulo. Raimond señaló por qué. Gracias de todos modos, no vi por qué también: -)

Mi solución de trabajo actual (aunque no es lo que quería) es un Atributo que uso para decorar todas mis acciones de controlador. Aquí está la implementación:

public class MasterPageDataAttribute : ActionFilterAttribute
    {
        public override void OnActionExecuting(ActionExecutingContext filterContext)
        {
            base.OnActionExecuting(filterContext);
            IUserRepository _repUser = RepositoryFactory.getUserRepository();
            IPrincipal siteUser = filterContext.Controller.ControllerContext.HttpContext.User;
            User loggedInUser = null;

            if (siteUser == null || siteUser.Identity.Name == null)
            {
                //do nothing
            }
            else
            {
                loggedInUser = _repUser.findUserById(siteUser.Identity.Name);
            }
            filterContext.Controller.ViewData["LoggedInUser"] = loggedInUser ?? new User { Nickname = "Guest" };
        }
    }

Buscaré cómo ejecutar ese código de una manera que siga el principio DRY, ya que usar atributos para eso definitivamente significa repetir uno mismo. Tal vez algún tipo de interceptor (idea interesante) o gancho podría ayudar.

Salud por eso.

Estoy haciendo esto en una implementación de basecontroller y funciona como se esperaba.

public abstract class BaseController : Controller
{
    public bool LoggedOn
    {
        get { return User.Identity.IsAuthenticated; }
    }
}

Esto siempre devuelve verdadero o falso para mí así que User != null

A Masterfu: Hice algo similar con su ayuda, deseo que puede ayudar a los últimos visitantes. En mi caso, necesito crear reposicionamiento de controladores para diferentes usuarios, sin embargo, en el constructor de controladores, el Usuario (principal)no está listo. Así que creé un atributo para controladores:

[CreateRepositoryByUser]
public class MFCController : Controller
{
    protected MFCRepository _repository
    {
        get { return ViewData["repository"] as MFCRepository; }
    }
...

El _repository, de hecho, no es una variable privada del controlador, sino que algo crea por el atributo:

public class CreateRepositoryByUser : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        CreateRepository(filterContext);
    }

    public static void CreateRepository(ActionExecutingContext filterContext)
    {
        if (filterContext.Controller.ViewData["repository"] == null)
        {
            filterContext.Controller.ViewData["repository"] =
                MFCRepository.CreateMFCRepository(filterContext.Controller.ControllerContext.HttpContext.User);
        }
    }
}

Pongo los códigos de creación del repositorio en un método separado, en caso de es posible que otros atributos quieran usar (principal)User antes de que se active este atributo.

Llamar desde un constructor es demasiado pronto en la canalización MVC.

Al mover el código a OnAuthorization, se obtiene el usuario autorizado en un parámetro. ¡Funcionó para mí!

De tu ejemplo haría algo como esto:

public abstract class ApplicationController : Controller {
    private IUserRepository _repUser;

    protected override void OnAuthorization(AuthorizationContext filterContext)
    {
        _repUser = RepositoryFactory.getUserRepository();
        var loggedInUser = _repUser.FindById(filterContext.HttpContext.User.Identity.Name); //Problem!
        ViewData["LoggedInUser"] = loggedInUser;
    }


}