ServiceFabricModule is internal, visible to Autofac RegisterAssemblyModules, but cannot be constructed about autofac.servicefabric HOT 8 CLOSED

@wasabii Yes, making the module internal was intentional.

Do you have an exception that you can provide or a quick reproduction? I don't think RegisterAssemblyModules includes internal types by default.

from autofac.servicefabric.

It looks like internal types are considered. I guess what makes this different from regular cases is the non-default constructor. This is actually an issue for the core library more than this integration. I might open a separate issue in core for this.

from autofac.servicefabric.

Be careful if you change this in the core - had a similar issue about assembly type scanning and people have come to rely on the behavior, considering it a breaking change when I tried to make it public-only by default.

from autofac.servicefabric.

I was just about to link that that very issue. 😄

from autofac.servicefabric.

It is actually the internal constructor that is causing the problem in this case. The internal class is located in RegisterAssemblyTypes but the instance cannot be created because the DefaultConstructorFinder excludes the internal constructor.

From a backwards compatibility perspective changing anything in core around this will be difficult, especially since it sits on top of something that we have already decided is hard to change. A similar "public only" option is something that could be considered.

I would certainly recommend when assembly scanning to try and reduce what you include. Using a base type for your modules is a reasonable choice. I'll have a think about what else we might be able to do.

from autofac.servicefabric.

Internal type, public constructor?

from autofac.servicefabric.

That does prevent the exception but still includes the module in the scan. I actually added code in the RegisterServiceFabricSupport method to prevent the registrations in the module from being added twice. It might not matter that much, but it does highlight an issue with overreach in the module registration path.

from autofac.servicefabric.

I think I'll just convert the module code into a static helper method removing the module from the picture all together.

from autofac.servicefabric.