Skip to main content
se añadieron 986 caracteres en el cuerpo
Origen Enlace
Diego
  • 5.6k
  • 1
  • 18
  • 38

El problema es que el método CreateRoles es asíncrono y lo estás invocando en el constructor el cual no puede ser asíncrono.

El flujo de la ejecución no va a "esperar" a que se termine de ejecutar el método y pasará a ejecutar la acción del controlador, una vez que la acción termina el framework desecha los recursos que cuyo ciclo de vida corresponde a solo la duración del request tal como UserManager. Es posible que en este punto, el método CreateRoles continue con su ejecución, teniendo referencia a objetos que han sido desechados (disposed).

Viendo el código asumo que quieres asegurar que existan roles antes que se ejecute el resto del programa. Lo que te recomiendo es que saques esa inicialización del constructor del controlador, el constructor no es un buen lugar para ponerlo. Podrías ponerlo en un Middleware antes de la ejecución del flujo regular de MVC.

Así creas un middleware:

public class MyMiddleware
{
    private readonly RequestDelegate _next;
    private readonly IServiceProvider _serviceProvider;

    public MyMiddleware(RequestDelegate next, IServiceProvider serviceProvider)
    {
        _next = next;
        _serviceProvider = serviceProvider;
    }

    public async Task Invoke(HttpContext httpContext)
    {
        // tu código

        await _next(httpContext);
    }
}

public static class MiddlewareExtensions
{
    public static IApplicationBuilder UseMiddleware(this IApplicationBuilder builder)
    {
        return builder.UseMiddleware<MyMiddleware>();
    }
}

y lo registras en el Startup:

app.UseAuthentication();
app.UseMyMiddleware();
app.UseMvc(routes => //...

el orden es importante aquí, asegúrate de que sea registrado luego de la autenticación y antes de MVC.

El problema es que el método CreateRoles es asíncrono y lo estás invocando en el constructor el cual no puede ser asíncrono.

El flujo de la ejecución no va a "esperar" a que se termine de ejecutar el método y pasará a ejecutar la acción del controlador, una vez que la acción termina el framework desecha los recursos que cuyo ciclo de vida corresponde a solo la duración del request tal como UserManager. Es posible que en este punto, el método CreateRoles continue con su ejecución, teniendo referencia a objetos que han sido desechados (disposed).

Viendo el código asumo que quieres asegurar que existan roles antes que se ejecute el resto del programa. Lo que te recomiendo es que saques esa inicialización del constructor del controlador, el constructor no es un buen lugar para ponerlo. Podrías ponerlo en un Middleware antes de la ejecución del flujo regular de MVC.

El problema es que el método CreateRoles es asíncrono y lo estás invocando en el constructor el cual no puede ser asíncrono.

El flujo de la ejecución no va a "esperar" a que se termine de ejecutar el método y pasará a ejecutar la acción del controlador, una vez que la acción termina el framework desecha los recursos que cuyo ciclo de vida corresponde a solo la duración del request tal como UserManager. Es posible que en este punto, el método CreateRoles continue con su ejecución, teniendo referencia a objetos que han sido desechados (disposed).

Viendo el código asumo que quieres asegurar que existan roles antes que se ejecute el resto del programa. Lo que te recomiendo es que saques esa inicialización del constructor del controlador, el constructor no es un buen lugar para ponerlo. Podrías ponerlo en un Middleware antes de la ejecución del flujo regular de MVC.

Así creas un middleware:

public class MyMiddleware
{
    private readonly RequestDelegate _next;
    private readonly IServiceProvider _serviceProvider;

    public MyMiddleware(RequestDelegate next, IServiceProvider serviceProvider)
    {
        _next = next;
        _serviceProvider = serviceProvider;
    }

    public async Task Invoke(HttpContext httpContext)
    {
        // tu código

        await _next(httpContext);
    }
}

public static class MiddlewareExtensions
{
    public static IApplicationBuilder UseMiddleware(this IApplicationBuilder builder)
    {
        return builder.UseMiddleware<MyMiddleware>();
    }
}

y lo registras en el Startup:

app.UseAuthentication();
app.UseMyMiddleware();
app.UseMvc(routes => //...

el orden es importante aquí, asegúrate de que sea registrado luego de la autenticación y antes de MVC.

Origen Enlace
Diego
  • 5.6k
  • 1
  • 18
  • 38

El problema es que el método CreateRoles es asíncrono y lo estás invocando en el constructor el cual no puede ser asíncrono.

El flujo de la ejecución no va a "esperar" a que se termine de ejecutar el método y pasará a ejecutar la acción del controlador, una vez que la acción termina el framework desecha los recursos que cuyo ciclo de vida corresponde a solo la duración del request tal como UserManager. Es posible que en este punto, el método CreateRoles continue con su ejecución, teniendo referencia a objetos que han sido desechados (disposed).

Viendo el código asumo que quieres asegurar que existan roles antes que se ejecute el resto del programa. Lo que te recomiendo es que saques esa inicialización del constructor del controlador, el constructor no es un buen lugar para ponerlo. Podrías ponerlo en un Middleware antes de la ejecución del flujo regular de MVC.