Is the dispose of the lifetime made too soon? about autofac.owin HOT 18 CLOSED

Is this an OWIN-only app or a mix of OWIN and classic pipelines? For example, ASP.NET MVC has OWIN hooks but doesn't actually run OWIN-only - it still uses the classic ASP.NET pipeline.

from autofac.owin.

The end scenario is an ASP.NET MVC application.
Pretty similar of the default samples of ASP.NET Katana.

But I'm able to reproduce this without any MVC reference and only using OWIN (the inicial sample I posted on the issue).

from autofac.owin.

From what I remember, in ASP.NET host (i.e., IIS), OnSendingRequestHeaders occurs far after the end of OWIN pipeline (even after Request_End). Autofac.OWIN disposes of the lifetime scope at the end of the pipeline (as OWIN doesn't offer an opportunity to dispose something at the end of request). One can add a check whether the call is done in the context of ASP.NET pipeline and bind the disposal to ASP.NET request (this, actually, is possible), but I think this would heavily complicate things (and add unneeded dependencies).

from autofac.owin.

Indeed.. I also filed an issue on aspnet/AspNetKatana#85 becuase I don't think that's a good place to put that logic in...
But I'm not optimistic..

from autofac.owin.

@tillig, actually, I think it's possible to provide a fairly easy-to-use WebHost integration package on top of my #10, which had arisen from, I think, the same problem as @joaomatossilva has - of course, if I am not mistaken and they are running in ASP.NET pipeline.

from autofac.owin.

I'm open to suggestions or PRs, but if it needs a separate WebHost integration package just to support this, I'd be very open to someone else owning and distributing that package. We have quite a few integration packages now and it's pretty difficult to keep on top of everything. It's one of the reasons we don't have time to address some of the longer-running issues in Autofac core.

from autofac.owin.

@joaomatossilva, are you hosting your OWIN pipeline in ASP.NET/IIS? Is that your target production environment?

from autofac.owin.

from autofac.owin.

Well, then you may try to get advantage of the approach I've vouched in #10 (for similar reasons), and which is now possible with Autofac.Owin 4.1.0:

1. Create your lifetime scope manually in Application_BeginRequest (it happens earlier that any OWIN pipeline) and save it to HttpContext. In the same event handler register your lifetime scope for disposal with HttpContext.DisposeOnPipelineCompleted
2. In OWIN pipeline, use UseAutofacLifetimeInjector with factory (instead of container) and in that factory get your lifetime from HttpContext (which is available from IOwinContext with the following: context.Get<HttpContextBase>(typeof(HttpContextBase).FullName), or you can simply use HttpContext.Current, if you don't mind about testability there).

I will migrate my own project to this solution in a few days and get back with exact code samples.

from autofac.owin.

And I see that 4.1.0 is already out there. I'll give it a try as well

from autofac.owin.

One question, this will mean that I need to change the lifetime configuration to InstancePerLifetimeScope instead of InstancePerRequest, right?

from autofac.owin.

This is my working sample so far..

On Global.asax

public class MvcApplication : System.Web.HttpApplication
{
    public const string ScopeKey = "CustomRequestScope";


    protected void Application_Start()
    {
        AreaRegistration.RegisterAllAreas();
        RouteConfig.RegisterRoutes(RouteTable.Routes);
    }

    protected void Application_BeginRequest()
    {
        var context = HttpContext.Current;
        ConfigureAutofac(context);
    }

    private IContainer BuildContainer()
    {
        var builder = new ContainerBuilder();

        //setup the registry of the PluggableComponent
        builder.RegisterType<PluggableComponent>().AsSelf().InstancePerLifetimeScope();

        //setup the middleware
        builder.RegisterType<SimpleMiddleware>().AsSelf();

        //Register the Controllers
        builder.RegisterControllers(typeof(MvcApplication).Assembly);

        var container = builder.Build();
        return container;
    }

    private void ConfigureAutofac(HttpContext context)
    {
        //configure the container and start a lifetimescope
        var container = BuildContainer();
        var scope = container.BeginLifetimeScope(ScopeKey);

        //Save the scope on the context and register it to be disposed on the end of the context
        context.Items[ScopeKey] = scope;
        context.DisposeOnPipelineCompleted(scope);

        DependencyResolver.SetResolver(new AutofacDependencyResolver(container));
    }
}

And the owin startup:

public partial class Startup
{
    public void Configuration(IAppBuilder app)
    {
        app.UseAutofacLifetimeScopeInjector(context =>
        {
            var httpContext = context.Get<HttpContextBase>(typeof(HttpContextBase).FullName);
            var scope = (ILifetimeScope) httpContext.Items[MvcApplication.ScopeKey];
            return scope;
        });

        app.Use<SimpleMiddleware>();
        app.UseAutofacMvc();
    }
}

Full sample here: https://github.com/joaomatossilva/AutofacCustomLifetimeScope

from autofac.owin.

If you use Autofac.Core.Lifetime.MatchingScopeLifetimeTags.RequestLifetimeScopeTag as the request lifetime scope tag instead of a custom tag then you can use InstancePerRequest.

from autofac.owin.

@joaomatossilva, you should create your container in Application_Start, not Application_BeginRequest.

from autofac.owin.

Ah.. Of course.. it was a copy past from the Owin startup that only runs once..

from autofac.owin.

I digged a bit further, and found that the scope is not being disposed by the context.DisposeOnPipelineCompleted but from the Autofac.Integration.Mvc.RequestLifetimeHttpModule.OnEndRequest

And that is due to the DependencyResolver.SetResolver(new AutofacDependencyResolver(container));

Does this means that in order to make this work I'll need to create a custom IDependencyResolver?

from autofac.owin.

I've update the sample now.

I did not write an entire IDependencyResolver, I've created a ILifetimeScopeProvider and passed it on the AutofacDependencyResolver.

here's mine:

public class LifeTimeScopeProvider : ILifetimeScopeProvider
{
    public LifeTimeScopeProvider(ILifetimeScope container)
    {
        ApplicationContainer = container;
    }

    public ILifetimeScope GetLifetimeScope(Action<ContainerBuilder> configurationAction)
    {
        if (HttpContext.Current == null)
        {
            throw new InvalidOperationException("Http Context Not Available");
        }
        return (ILifetimeScope)HttpContext.Current.Items[MvcApplication.ScopeKey];
    }

    public void EndLifetimeScope()
    {
    }

    public ILifetimeScope ApplicationContainer { get; }
}

then I register it like this on Application_Start:

        Container = BuildContainer();
        var lifeTimeScopeProvider = new LifeTimeScopeProvider(Container);
        DependencyResolver.SetResolver(new AutofacDependencyResolver(Container, lifeTimeScopeProvider));

from autofac.owin.

It sounds like with the recent changes and the ability to provide your own lifetime scope/scope provider you've figured out how to get things working for your unique app needs. I'm going to go ahead and close this issue.

from autofac.owin.