Replace IActionContextAccessor with IHttpContextAccessor throughout the codebase

Co-authored-by: captainsafia <[email protected]>

Author: Copilot

src/Mvc/Mvc.Core/src/Infrastructure/ActionContextAccessor.cs

@@ -10,7 +10,6 @@ namespace Microsoft.AspNetCore.Mvc.Infrastructure;
 /// <summary>
 /// Type that provides access to an <see cref="ActionContext"/>.
 /// </summary>
-[Obsolete("ActionContextAccessor is obsolete. Use IHttpContextAccessor instead and access the endpoint information from HttpContext.GetEndpoint(). This type will be removed in a future version.")]
 public class ActionContextAccessor : IActionContextAccessor
 {
     internal static readonly IActionContextAccessor Null = new NullActionContextAccessor();

src/Mvc/Mvc.Core/src/Infrastructure/ControllerActionInvoker.cs

@@ -30,14 +30,11 @@ internal partial class ControllerActionInvoker : ResourceInvoker, IActionInvoker
     internal ControllerActionInvoker(
         ILogger logger,
         DiagnosticListener diagnosticListener,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor actionContextAccessor,
-#pragma warning restore CS0618 // Type or member is obsolete
         IActionResultTypeMapper mapper,
         ControllerContext controllerContext,
         ControllerActionInvokerCacheEntry cacheEntry,
         IFilterMetadata[] filters)
-        : base(diagnosticListener, logger, actionContextAccessor, mapper, controllerContext, filters, controllerContext.ValueProviderFactories)
+        : base(diagnosticListener, logger, mapper, controllerContext, filters, controllerContext.ValueProviderFactories)
     {
         ArgumentNullException.ThrowIfNull(cacheEntry);
 

src/Mvc/Mvc.Core/src/Infrastructure/ControllerActionInvokerProvider.cs

@@ -23,29 +23,13 @@ internal sealed class ControllerActionInvokerProvider : IActionInvokerProvider
     private readonly ILogger _logger;
     private readonly DiagnosticListener _diagnosticListener;
     private readonly IActionResultTypeMapper _mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-    private readonly IActionContextAccessor _actionContextAccessor;
-#pragma warning restore CS0618 // Type or member is obsolete
 
     public ControllerActionInvokerProvider(
         ControllerActionInvokerCache controllerActionInvokerCache,
         IOptions<MvcOptions> optionsAccessor,
         ILoggerFactory loggerFactory,
         DiagnosticListener diagnosticListener,
         IActionResultTypeMapper mapper)
-        : this(controllerActionInvokerCache, optionsAccessor, loggerFactory, diagnosticListener, mapper, null)
-    {
-    }
-
-    public ControllerActionInvokerProvider(
-        ControllerActionInvokerCache controllerActionInvokerCache,
-        IOptions<MvcOptions> optionsAccessor,
-        ILoggerFactory loggerFactory,
-        DiagnosticListener diagnosticListener,
-        IActionResultTypeMapper mapper,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor? actionContextAccessor)
-#pragma warning restore CS0618 // Type or member is obsolete
     {
         _controllerActionInvokerCache = controllerActionInvokerCache;
         _valueProviderFactories = optionsAccessor.Value.ValueProviderFactories.ToArray();
@@ -55,9 +39,6 @@ public ControllerActionInvokerProvider(
         _logger = loggerFactory.CreateLogger(typeof(ControllerActionInvoker));
         _diagnosticListener = diagnosticListener;
         _mapper = mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-        _actionContextAccessor = actionContextAccessor ?? ActionContextAccessor.Null;
-#pragma warning restore CS0618 // Type or member is obsolete
     }
 
     public int Order => -1000;
@@ -83,7 +64,6 @@ public void OnProvidersExecuting(ActionInvokerProviderContext context)
             var invoker = new ControllerActionInvoker(
                 _logger,
                 _diagnosticListener,
-                _actionContextAccessor,
                 _mapper,
                 controllerContext,
                 cacheEntry,

src/Mvc/Mvc.Core/src/Infrastructure/IActionContextAccessor.cs

@@ -10,7 +10,6 @@ namespace Microsoft.AspNetCore.Mvc.Infrastructure;
 /// <summary>
 /// Defines an interface for exposing an <see cref="ActionContext"/>.
 /// </summary>
-[Obsolete("IActionContextAccessor is obsolete. Use IHttpContextAccessor instead and access the endpoint information from HttpContext.GetEndpoint(). This type will be removed in a future version.")]
 public interface IActionContextAccessor
 {
     /// <summary>

src/Mvc/Mvc.Core/src/Infrastructure/ResourceInvoker.cs

@@ -15,9 +15,6 @@ internal abstract partial class ResourceInvoker
 {
     protected readonly DiagnosticListener _diagnosticListener;
     protected readonly ILogger _logger;
-#pragma warning disable CS0618 // Type or member is obsolete
-    protected readonly IActionContextAccessor _actionContextAccessor;
-#pragma warning restore CS0618 // Type or member is obsolete
     protected readonly IActionResultTypeMapper _mapper;
     protected readonly ActionContext _actionContext;
     protected readonly IFilterMetadata[] _filters;
@@ -39,17 +36,13 @@ internal abstract partial class ResourceInvoker
     public ResourceInvoker(
         DiagnosticListener diagnosticListener,
         ILogger logger,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor actionContextAccessor,
-#pragma warning restore CS0618 // Type or member is obsolete
         IActionResultTypeMapper mapper,
         ActionContext actionContext,
         IFilterMetadata[] filters,
         IList<IValueProviderFactory> valueProviderFactories)
     {
         _diagnosticListener = diagnosticListener ?? throw new ArgumentNullException(nameof(diagnosticListener));
         _logger = logger ?? throw new ArgumentNullException(nameof(logger));
-        _actionContextAccessor = actionContextAccessor ?? throw new ArgumentNullException(nameof(actionContextAccessor));
         _mapper = mapper ?? throw new ArgumentNullException(nameof(mapper));
         _actionContext = actionContext ?? throw new ArgumentNullException(nameof(actionContext));
 
@@ -65,7 +58,6 @@ public virtual Task InvokeAsync()
             return Logged(this);
         }
 
-        _actionContextAccessor.ActionContext = _actionContext;
         var scope = _logger.ActionScope(_actionContext.ActionDescriptor);
 
         Task task;
@@ -100,7 +92,6 @@ static async Task Awaited(ResourceInvoker invoker, Task task, IDisposable? scope
         static async Task Logged(ResourceInvoker invoker)
         {
             var actionContext = invoker._actionContext;
-            invoker._actionContextAccessor.ActionContext = actionContext;
             try
             {
                 var logger = invoker._logger;

src/Mvc/Mvc.Core/src/Routing/ControllerRequestDelegateFactory.cs

@@ -24,9 +24,6 @@ internal sealed class ControllerRequestDelegateFactory : IRequestDelegateFactory
     private readonly ILogger _logger;
     private readonly DiagnosticListener _diagnosticListener;
     private readonly IActionResultTypeMapper _mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-    private readonly IActionContextAccessor _actionContextAccessor;
-#pragma warning restore CS0618 // Type or member is obsolete
     private readonly bool _enableActionInvokers;
 
     public ControllerRequestDelegateFactory(
@@ -35,19 +32,6 @@ public ControllerRequestDelegateFactory(
         ILoggerFactory loggerFactory,
         DiagnosticListener diagnosticListener,
         IActionResultTypeMapper mapper)
-        : this(controllerActionInvokerCache, optionsAccessor, loggerFactory, diagnosticListener, mapper, null)
-    {
-    }
-
-    public ControllerRequestDelegateFactory(
-        ControllerActionInvokerCache controllerActionInvokerCache,
-        IOptions<MvcOptions> optionsAccessor,
-        ILoggerFactory loggerFactory,
-        DiagnosticListener diagnosticListener,
-        IActionResultTypeMapper mapper,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor? actionContextAccessor)
-#pragma warning restore CS0618 // Type or member is obsolete
     {
         _controllerActionInvokerCache = controllerActionInvokerCache;
         _valueProviderFactories = optionsAccessor.Value.ValueProviderFactories.ToArray();
@@ -58,9 +42,6 @@ public ControllerRequestDelegateFactory(
         _logger = loggerFactory.CreateLogger<ControllerActionInvoker>();
         _diagnosticListener = diagnosticListener;
         _mapper = mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-        _actionContextAccessor = actionContextAccessor ?? ActionContextAccessor.Null;
-#pragma warning restore CS0618 // Type or member is obsolete
     }
 
     public RequestDelegate? CreateRequestDelegate(ActionDescriptor actionDescriptor, RouteValueDictionary? dataTokens)
@@ -100,7 +81,6 @@ public ControllerRequestDelegateFactory(
             var invoker = new ControllerActionInvoker(
                 _logger,
                 _diagnosticListener,
-                _actionContextAccessor,
                 _mapper,
                 controllerContext,
                 cacheEntry,

src/Mvc/Mvc.Core/test/Filters/MiddlewareFilterTest.cs

@@ -398,9 +398,6 @@ public TestControllerActionInvoker(
             : base(
                   logger,
                   diagnosticListener,
-#pragma warning disable CS0618 // Type or member is obsolete
-                  ActionContextAccessor.Null,
-#pragma warning restore CS0618 // Type or member is obsolete
                   mapper,
                   CreateControllerContext(actionContext, valueProviderFactories, maxAllowedErrorsInModelState),
                   CreateCacheEntry((ControllerActionDescriptor)actionContext.ActionDescriptor, controllerFactory),

src/Mvc/Mvc.Core/test/Infrastructure/ControllerActionInvokerTest.cs

@@ -1432,9 +1432,6 @@ public async Task Invoke_UsesDefaultValuesIfNotBound()
         var invoker = new ControllerActionInvoker(
             new NullLoggerFactory().CreateLogger<ControllerActionInvoker>(),
             new DiagnosticListener("Microsoft.AspNetCore"),
-#pragma warning disable CS0618 // Type or member is obsolete
-            ActionContextAccessor.Null,
-#pragma warning restore CS0618 // Type or member is obsolete
             new ActionResultTypeMapper(),
             controllerContext,
             cacheEntry,
@@ -1771,9 +1768,6 @@ private ControllerActionInvoker CreateInvoker(
         var invoker = new ControllerActionInvoker(
             logger,
             diagnosticSource,
-#pragma warning disable CS0618 // Type or member is obsolete
-            ActionContextAccessor.Null,
-#pragma warning restore CS0618 // Type or member is obsolete
             new ActionResultTypeMapper(),
             controllerContext,
             cacheEntry,

src/Mvc/Mvc.RazorPages/src/Infrastructure/PageActionInvoker.cs

@@ -35,9 +35,6 @@ public PageActionInvoker(
         IPageHandlerMethodSelector handlerMethodSelector,
         DiagnosticListener diagnosticListener,
         ILogger logger,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor actionContextAccessor,
-#pragma warning restore CS0618 // Type or member is obsolete
         IActionResultTypeMapper mapper,
         PageContext pageContext,
         IFilterMetadata[] filterMetadata,
@@ -47,7 +44,6 @@ public PageActionInvoker(
         : base(
               diagnosticListener,
               logger,
-              actionContextAccessor,
               mapper,
               pageContext,
               filterMetadata,

src/Mvc/Mvc.RazorPages/src/Infrastructure/PageActionInvokerProvider.cs

@@ -25,9 +25,6 @@ internal sealed class PageActionInvokerProvider : IActionInvokerProvider
     private readonly DiagnosticListener _diagnosticListener;
     private readonly ILogger<PageActionInvoker> _logger;
     private readonly IActionResultTypeMapper _mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-    private readonly IActionContextAccessor _actionContextAccessor;
-#pragma warning restore CS0618 // Type or member is obsolete
 
     public PageActionInvokerProvider(
         PageLoader pageLoader,
@@ -39,10 +36,7 @@ public PageActionInvokerProvider(
         IPageHandlerMethodSelector selector,
         DiagnosticListener diagnosticListener,
         ILoggerFactory loggerFactory,
-        IActionResultTypeMapper mapper,
-#pragma warning disable CS0618 // Type or member is obsolete
-        IActionContextAccessor? actionContextAccessor = null)
-#pragma warning restore CS0618 // Type or member is obsolete
+        IActionResultTypeMapper mapper)
     {
         _pageLoader = pageLoader;
         _pageActionInvokerCache = pageActionInvokerCache;
@@ -54,9 +48,6 @@ public PageActionInvokerProvider(
         _diagnosticListener = diagnosticListener;
         _logger = loggerFactory.CreateLogger<PageActionInvoker>();
         _mapper = mapper;
-#pragma warning disable CS0618 // Type or member is obsolete
-        _actionContextAccessor = actionContextAccessor ?? ActionContextAccessor.Null;
-#pragma warning restore CS0618 // Type or member is obsolete
     }
 
     // For testing
@@ -96,7 +87,6 @@ public void OnProvidersExecuting(ActionInvokerProviderContext context)
             _selector,
             _diagnosticListener,
             _logger,
-            _actionContextAccessor,
             _mapper,
             pageContext,
             filters,