Add singleton IScopeAccessor service as an abstraction over HttpContext.

Author: dimarobert

DNN Platform/DotNetNuke.Web.Mvc/DnnMvcDependencyResolver.cs

@@ -5,9 +5,8 @@
 using Microsoft.Extensions.DependencyInjection;
 using System;
 using System.Collections.Generic;
-using System.Web;
 using System.Web.Mvc;
-using DotNetNuke.Common.Extensions;
+using DotNetNuke.Services.DependencyInjection;
 
 namespace DotNetNuke.Web.Mvc
 {
@@ -17,6 +16,12 @@ namespace DotNetNuke.Web.Mvc
     /// </summary>
     internal class DnnMvcDependencyResolver : IDependencyResolver
     {
+        private readonly IServiceProvider _serviceProvider;
+
+        public DnnMvcDependencyResolver(IServiceProvider serviceProvider)
+        {
+            _serviceProvider = serviceProvider ?? throw new ArgumentNullException(nameof(serviceProvider));
+        }
 
         /// <summary>
         /// Returns the specified service from the scope
@@ -29,7 +34,8 @@ internal class DnnMvcDependencyResolver : IDependencyResolver
         /// </returns>
         public object GetService(Type serviceType)
         {
-            var scope = HttpContext.Current?.GetScope();
+            var accessor = _serviceProvider.GetRequiredService<IScopeAccessor>();
+            var scope = accessor.GetScope();
             if (scope != null)
                 return scope.ServiceProvider.GetService(serviceType);
 
@@ -47,7 +53,8 @@ public object GetService(Type serviceType)
         /// </returns>
         public IEnumerable<object> GetServices(Type serviceType)
         {
-            var scope = HttpContext.Current?.GetScope();
+            var accessor = _serviceProvider.GetRequiredService<IScopeAccessor>();
+            var scope = accessor.GetScope();
             if (scope != null)
                 return scope.ServiceProvider.GetServices(serviceType);
 

DNN Platform/DotNetNuke.Web.Mvc/Startup.cs

@@ -6,6 +6,7 @@
 using Microsoft.Extensions.DependencyInjection;
 using DotNetNuke.Web.Mvc.Extensions;
 using System.Web.Mvc;
+using DotNetNuke.Common;
 
 namespace DotNetNuke.Web.Mvc
 {
@@ -18,7 +19,7 @@ public void ConfigureServices(IServiceCollection services)
 
             services.AddWebApiControllers();
 
-            DependencyResolver.SetResolver(new DnnMvcDependencyResolver());
+            DependencyResolver.SetResolver(new DnnMvcDependencyResolver(Globals.DependencyProvider));
         }
     }
 }

DNN Platform/DotNetNuke.Web/Api/Internal/DnnDependencyResolver.cs

@@ -7,6 +7,7 @@
 using System.Collections.Generic;
 using System.Web.Http.Dependencies;
 using DotNetNuke.Common.Extensions;
+using DotNetNuke.Services.DependencyInjection;
 
 namespace DotNetNuke.Web.Api.Internal
 {
@@ -37,7 +38,8 @@ public DnnDependencyResolver(IServiceProvider serviceProvider)
         /// </returns>
         public IDependencyScope BeginScope()
         {
-            var scope = System.Web.HttpContext.Current.GetScope();
+            var accessor = _serviceProvider.GetRequiredService<IScopeAccessor>();
+            var scope = accessor.GetScope();
             return new DnnDependencyResolver(scope.ServiceProvider);
         }
 

DNN Platform/DotNetNuke.Web/Common/DotNetNukeHttpApplication.cs

@@ -67,8 +67,9 @@ private void Application_Start(object sender, EventArgs eventArgs)
             var name = Config.GetSetting("ServerName");
             Globals.ServerName = String.IsNullOrEmpty(name) ? Dns.GetHostName() : name;
 
+            Globals.DependencyProvider = new LazyServiceProvider();
             var startup = new Startup();
-            Globals.DependencyProvider = startup.DependencyProvider;
+            (Globals.DependencyProvider as LazyServiceProvider).SetProvider(startup.DependencyProvider);
             ServiceRequestScopeModule.SetServiceProvider(Globals.DependencyProvider);
 
             ComponentFactory.Container = new SimpleContainer();

DNN Platform/DotNetNuke.Web/Common/LazyServiceProvider.cs

@@ -0,0 +1,26 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace DotNetNuke.Web.Common
+{
+    public class LazyServiceProvider : IServiceProvider
+    {
+        private IServiceProvider _serviceProvider;
+
+        public object GetService(Type serviceType)
+        {
+            if (_serviceProvider is null)
+                throw new Exception("Cannot resolve services until the service provider is built.");
+
+            return _serviceProvider.GetService(serviceType);
+        }
+
+        internal void SetProvider(IServiceProvider serviceProvider)
+        {
+            _serviceProvider = serviceProvider;
+        }
+    }
+}

DNN Platform/DotNetNuke.Web/DotNetNuke.Web.csproj

@@ -203,6 +203,7 @@
     <Compile Include="Common\DynamicSharedConstants.cs" />
     <Compile Include="Common\DotNetNukeHttpApplication.cs" />
     <Compile Include="Api\WebApiException.cs" />
+    <Compile Include="Common\LazyServiceProvider.cs" />
     <Compile Include="Components\Controllers\ControlBarController.cs" />
     <Compile Include="Components\Controllers\IControlBarController.cs" />
     <Compile Include="Components\Controllers\Models\MenuItemViewModel.cs" />

DNN Platform/DotNetNuke.Web/Startup.cs

@@ -5,6 +5,7 @@
 using DotNetNuke.DependencyInjection;
 using DotNetNuke.DependencyInjection.Extensions;
 using DotNetNuke.Instrumentation;
+using DotNetNuke.Services.DependencyInjection;
 using DotNetNuke.Web.Extensions;
 using Microsoft.Extensions.DependencyInjection;
 using System;
@@ -25,6 +26,7 @@ public Startup()
         private void Configure()
         {
             var services = new ServiceCollection();
+            services.AddSingleton<IScopeAccessor, ScopeAccessor>();
             ConfigureServices(services);
             DependencyProvider = services.BuildServiceProvider();
         }

DNN Platform/Library/Common/Extensions/HttpContextDependencyInjectionExtensions.cs

@@ -30,7 +30,7 @@ public static IServiceScope GetScope(this HttpContext httpContext)
             return GetScope(httpContext.Items);
         }
 
-        private static IServiceScope GetScope(System.Collections.IDictionary contextItems)
+        internal static IServiceScope GetScope(System.Collections.IDictionary contextItems)
         {
             if (!contextItems.Contains(typeof(IServiceScope)))
                 return null;

DNN Platform/Library/DotNetNuke.Library.csproj

@@ -701,6 +701,7 @@
     <Compile Include="Services\Cryptography\CryptographyProvider.cs" />
     <Compile Include="Services\Cryptography\ICryptographyProvider.cs" />
     <Compile Include="Services\Authentication\UserAuthenticationInfo.cs" />
+    <Compile Include="Services\DependencyInjection\IScopeAccessor.cs" />
     <Compile Include="Services\Exceptions\ExceptionExtensions.cs" />
     <Compile Include="Services\Exceptions\SearchIndexEmptyException.cs" />
     <Compile Include="Services\Assets\AssetManager.cs" />

DNN Platform/Library/Services/DependencyInjection/IScopeAccessor.cs

@@ -0,0 +1,35 @@
+using DotNetNuke.Common.Extensions;
+using Microsoft.Extensions.DependencyInjection;
+using System;
+using System.Collections;
+using System.Web;
+
+namespace DotNetNuke.Services.DependencyInjection
+{
+    public interface IScopeAccessor
+    {
+        IServiceScope GetScope();
+    }
+
+    public class ScopeAccessor : IScopeAccessor
+    {
+        private static Func<IDictionary> fallbackGetContextItems = () => HttpContext.Current?.Items;
+
+        private Func<IDictionary> _getContextItems;
+
+        public ScopeAccessor()
+        {
+            _getContextItems = fallbackGetContextItems;
+        }
+
+        public IServiceScope GetScope()
+        {
+            return HttpContextDependencyInjectionExtensions.GetScope(_getContextItems());
+        }
+
+        internal void SetContextItemsFunc(Func<IDictionary> getContextItems)
+        {
+            _getContextItems = getContextItems ?? fallbackGetContextItems;
+        }
+    }
+}