Disambiguate NetheriteFactory constructor and rev version

[davidmrdavid]

There are two NetheriteProviderFactory constructors:

durabletask-netherite/src/DurableTask.Netherite.AzureFunctions/NetheriteProviderFactory.cs

Lines 51 to 109 in f3e5f26

	/// <summary>
	/// This constructor should not be called directly. It is here only to avoid breaking changes.
	/// </summary>
	[Obsolete]
	public NetheriteProviderFactory(
	IOptions<DurableTaskOptions> extensionOptions,
	ILoggerFactory loggerFactory,
	#pragma warning disable CS0618 // Type or member is obsolete
	IConnectionStringResolver connectionStringResolver,
	#pragma warning restore CS0618 // Type or member is obsolete
	IHostIdProvider hostIdProvider,
	INameResolver nameResolver,
	#pragma warning disable CS0612 // Type or member is obsolete
	IPlatformInformation platformInfo)
	#pragma warning restore CS0612 // Type or member is obsolete
	: this(extensionOptions, loggerFactory, hostIdProvider, nameResolver, serviceProvider:null, ConnectionResolver.FromConnectionNameToConnectionStringResolver((s) => nameResolver.Resolve(s)), platformInfo)
	{
	}
	/// <summary>
	/// This constructor should not be called directly. The DI logic calls this when it constructs all the
	/// durability provider factories for use by <see cref="Microsoft.Azure.WebJobs.Extensions.DurableTask.DurableTaskExtension"/>.
	/// </summary>
	[ActivatorUtilitiesConstructor]
	public NetheriteProviderFactory(
	IOptions<DurableTaskOptions> extensionOptions,
	ILoggerFactory loggerFactory,
	IHostIdProvider hostIdProvider,
	INameResolver nameResolver,
	IServiceProvider serviceProvider,
	DurableTask.Netherite.ConnectionResolver connectionResolver,
	#pragma warning disable CS0612 // Type or member is obsolete
	IPlatformInformation platformInfo)
	#pragma warning restore CS0612 // Type or member is obsolete
	{
	this.options = extensionOptions?.Value ?? throw new ArgumentNullException(nameof(extensionOptions));
	this.loggerFactory = loggerFactory ?? throw new ArgumentNullException(nameof(loggerFactory));
	this.nameResolver = nameResolver ?? throw new ArgumentNullException(nameof(nameResolver));
	this.serviceProvider = serviceProvider;
	this.hostIdProvider = hostIdProvider;
	this.connectionResolver = connectionResolver;
	this.inConsumption = platformInfo.IsInConsumptionPlan();
	bool ReadBooleanSetting(string name) => this.options.StorageProvider.TryGetValue(name, out object objValue)
	&& objValue is string stringValue && bool.TryParse(stringValue, out bool boolValue) && boolValue;
	this.TraceToConsole = ReadBooleanSetting(nameof(this.TraceToConsole));
	this.TraceToBlob = ReadBooleanSetting(nameof(this.TraceToBlob));
	WorkerRuntimeType runtimeType = platformInfo.GetWorkerRuntimeType();
	if (runtimeType == WorkerRuntimeType.DotNetIsolated ||
	runtimeType == WorkerRuntimeType.Java ||
	runtimeType == WorkerRuntimeType.Custom)
	{
	this.usesNewPassthroughMiddlewareForEntities = true;
	}
	}

. One is obsolete, and the other is not.

We used the ActivatorUtilitiesConstructor to tell DI which constructor to choose. However, it seems that annotation isn't honored in .NET8: dotnet/runtime#95915.

Due to some combination of this .NET8 change, and the Host for Flex Consumption using a new DI library, DF apps in Flex Consumption using bundles are now failing with the following exception:

System.InvalidOperationException : Unable to activate type 'DurableTask.Netherite.AzureFunctions.NetheriteProviderFactory'. The following constructors are ambiguous:
Void .ctor(Microsoft.Extensions.Options.IOptions1[Microsoft.Azure.WebJobs.Extensions.DurableTask.DurableTaskOptions], Microsoft.Extensions.Logging.ILoggerFactory, Microsoft.Azure.WebJobs.Host.Executors.IHostIdProvider, Microsoft.Azure.WebJobs.INameResolver, System.IServiceProvider, DurableTask.Netherite.ConnectionResolver, Microsoft.Azure.WebJobs.Extensions.DurableTask.IPlatformInformation) Void .ctor(Microsoft.Extensions.Options.IOptions1[Microsoft.Azure.WebJobs.Extensions.DurableTask.DurableTaskOptions], Microsoft.Extensions.Logging.ILoggerFactory, Microsoft.Azure.WebJobs.Extensions.DurableTask.IConnectionStringResolver, Microsoft.Azure.WebJobs.Host.Executors.IHostIdProvider, Microsoft.Azure.WebJobs.INameResolver, Microsoft.Azure.WebJobs.Extensions.DurableTask.IPlatformInformation)

This PR disambiguates the constructors by creating a new internal wrapper class, UnambiguousNetheriteProviderFactory, that re-exposes only the expected constructor, and registering it as the service provider for IDurabilityProviderFactory. This is a workaround suggested by Fabio from the Host team.