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 of AutofacDependencyResolver.Current
has been updated to remove the need for an HttpContext
. If you are encountering this problem and found this answer, you can potentially easily solve things by updating to a later version of Autofac.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 an HttpContext
.
Walking through the code, AutofacDependencyResolver.Current
looks like this:
public static AutofacDependencyResolver Current
{
get
{
return DependencyResolver.Current.GetService<AutofacDependencyResolver>();
}
}
And, of course, if the current dependency resolver is an AutofacDependencyResolver
then it's going to try to do a resolution...
public object GetService(Type serviceType)
{
return RequestLifetimeScope.ResolveOptional(serviceType);
}
Which gets the lifetime scope from a RequestLifetimeScopeProvider
...
public ILifetimeScope GetLifetimeScope(Action<ContainerBuilder> configurationAction)
{
if (HttpContext.Current == null)
{
throw new InvalidOperationException("...");
}
// ...and your code is probably dying right there so I won't
// include the rest of the source.
}
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
requires HttpContext
.
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
and InstancePerHttpRequest
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 on HttpContext
and the other is based on IDependencyScope
. 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.
public static class ApplicationContainer
{
public static IContainer Container { get; set; }
}
// And then when you build your resolvers...
var container = builder.Build();
GlobalConfiguration.Configuration.DependencyResolver =
new AutofacWebApiDependencyResolver(container);
DependencyResolver.SetResolver(new AutofacDependencyResolver(container));
ApplicationContainer.Container = container;
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 access HttpContext
? Correct - null reference exception. Or in your case dependency resolution exception.
What you need to do:
User.Current
or HttpContext.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.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With