Short Question: Same as this unanswered problem
Long Question:
I just ported some code over from an MVC 4 + Web Api solution that was using Autofac into my new solution which is also using Autofac but only with Web Api 2 (no MVC 5.1 project, just a web api).
In my previous solution I had MVC4 and Web Api so I had 2 Bootstrapper.cs files, one for each. I copied over just the Web Api bootstrapper for the new project.
Now I have 2 other projects in the new solution that need to pull a dependency. Lets just assume I have to use DependencyResolver.Current.GetService<T>()
despite it being an anti-pattern.
At first this was not working until I set the MVC Dependency Resolver to the same container:
GlobalConfiguration.Configuration.DependencyResolver =
new AutofacWebApiDependencyResolver(container);
//I had to pull in Autofac.Mvc and Mvc 5.1 integration but this line fixed it
DependencyResolver.SetResolver(new AutofacDependencyResolver(container));
The strange part is, doing that only fixed it in ONE of those projects! Here's the situation:
Solution.Web project
Bootstrapper.cs that registers both dependency resolvers for web api and mvc.
Solution.ClassLib project
var userRepo = DependencyResolver.Current.GetService<IUserRepo>(); //Good! :)
Solution.WindowsWorkflow project
var userRepo = DependencyResolver.Current.GetService<IUserRepo>(); //Throws exception :(
The exception is: The request lifetime scope cannot be created because the HttpContext is not available.
Now before we start blaming the workflow, just know I had this exact set up working just fine in another solution the workflow was able to use DependencyResolver just fine. So I suspect this had to do with using a newer version of Autofac and the fact that the workflow runs asynchronously (just like the question i linked to regarding async code)
I tried switching all the registration code to use InstancePerLifetimeScope()
instead of InstancePerHttpRequest()
and trying to create a scope:
using (var c= AutofacDependencyResolver.Current
.ApplicationContainer.BeginLifetimeScope("AutofacWebRequest"))
{
var userRepo = DependencyResolver.Current.GetServices<IUserRepo>();
}
But it didnt change the exception. Breaking the code down even further here's the exact culprit:
var adr = AutofacDependencyResolver.Current; //Throws that exception
Really need to get past this spent too much time stuck. Will reward existing answer with bounty in 2 days
UPDATE Nov. 20, 2014: In releases of
Autofac.Mvc5
since this question was released, the implementation ofAutofacDependencyResolver.Current
has been updated to remove the need for anHttpContext
. If you are encountering this problem and found this answer, you can potentially easily solve things by updating to a later version ofAutofac.Mvc5
. However, I will leave the original answer intact for folks to understand why the original question asker was having issues.Original answer follows:
AutofacDependencyResolver.Current
requires anHttpContext
.Walking through the code,
AutofacDependencyResolver.Current
looks like this:And, of course, if the current dependency resolver is an
AutofacDependencyResolver
then it's going to try to do a resolution...Which gets the lifetime scope from a
RequestLifetimeScopeProvider
...It has to work like that to support tools like Glimpse that dynamically wrap/proxy the dependency resolver in order to instrument it. That's why you can't just cast
DependencyResolver.Current as AutofacDependencyResolver
.Pretty much anything using the
Autofac.Integration.Mvc.AutofacDependencyResolver
requiresHttpContext
.That's why you keep getting this error. It doesn't matter if you have no dependencies that are registered
InstancePerHttpRequest
-AutofacDependencyResolver
will still require a web context.I'm guessing the other workflow app you had where this wasn't an issue was an MVC app or something where there was always a web context.
Here's what I'd recommend:
Autofac.Integration.WebApi.AutofacWebApiDependencyResolver
.AutofacHostFactory.Container
and that host factory implementation to resolve dependencies. (WCF is a little weird with its singleton host potential, etc. so "per request" isn't quite as straightforward.)CommonServiceLocator
implementation for Autofac. It doesn't create request lifetimes, but it may solve some problems.If you keep those things straight and don't try to use the various resolvers outside their native habitats, as it were, then you shouldn't run into issues.
You can fairly safely use
InstancePerApiRequest
andInstancePerHttpRequest
interchangeably in service registrations. Both of these extensions use the same lifetime scope tag so the notion of an MVC web request and a web API request can be treated similarly even if the underlying lifetime scope in one case is based onHttpContext
and the other is based onIDependencyScope
. So you could hypothetically share a registration module across apps/app types and it should do the right thing.If you need the original Autofac container, store your own reference to it. Rather than assuming Autofac will return that container somehow, you may need to store a reference to your application container if you need to get it later for whatever reason.
That will save you a lot of trouble down the road.
My assumptions:
IUserRepo
is dependent on HttpContextIf my assumption correct, Workflow project would have no idea about
HttpContext.Current
.WindowsWorkflow project runs all the time (If I understand it correctly - did not actually work with this tech). Where as MVC is based on HTTP requests.
HttpContext.Current
is populated only when there is a request coming in. If no request - this variable is null. What happens if there is no request, but Workflow instance is trying to accessHttpContext
? Correct - null reference exception. Or in your case dependency resolution exception.What you need to do:
User.Current
orHttpContext.Current
. And Workflow module (if required) with all Workflow specific implementations.IUserRepo
create implementation that is not dependent on HttpContext. This probably will be the most problematic to do.I've done something similar for Quartz.Net execution in Azure. See my blog post about this: http://tech.trailmax.info/2013/07/quartz-net-in-azure-with-autofac-smoothness/. This post will not help you directly, but explains my reasoning for splitting autofac modules.
Update as per comment: WebApi clarifies a lot of things here. WebApi request don't go through the same pipeline as your MVC requests. And WebApi controllers don't have access to HttpContext. See this answer.
Now, depending on what you are doing in your wepApi controller, you might want to change the
IUserRepo
implementation to be able to work with both MVC and WebApi.We're currently in a situation where we have tests that suffer from the 'missing httpcontext' issue but cannot yet use the excellent recommendations above due to version constraints.
The way we solved it was to create a 'mock' http context in our test setup: see: Mock HttpContext.Current in Test Init Method