v9: Remove install endpoint as valid endpoint (#11710)

* Remove installer endpoint from valid candidates

* Removed unrelevant change

Author: Zeegaan

src/Umbraco.Web.BackOffice/Install/InstallController.cs

@@ -13,6 +13,7 @@
 using Umbraco.Cms.Core.Services;
 using Umbraco.Cms.Core.WebAssets;
 using Umbraco.Cms.Infrastructure.Install;
+using Umbraco.Cms.Web.Common.Attributes;
 using Umbraco.Cms.Web.Common.Filters;
 using Umbraco.Extensions;
 
@@ -101,6 +102,7 @@ public async Task<ActionResult> Index()
         /// </summary>
         /// <returns></returns>
         [HttpGet]
+        [IgnoreFromNotFoundSelectorPolicy]
         public ActionResult Redirect()
         {
             var uri = HttpContext.Request.GetEncodedUrl();

src/Umbraco.Web.Common/Attributes/IgnoreFromNotFoundSelectorPolicyAttribute.cs

@@ -0,0 +1,13 @@
+using System;
+
+namespace Umbraco.Cms.Web.Common.Attributes
+{
+    /// <summary>
+    /// When applied to an api controller it will be routed to the /Umbraco/BackOffice prefix route so we can determine if it
+    /// is a back office route or not.
+    /// </summary>
+    [AttributeUsage(AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
+    public sealed class IgnoreFromNotFoundSelectorPolicyAttribute : Attribute
+    {
+    }
+}

src/Umbraco.Web.Website/Routing/NotFoundSelectorPolicy.cs

@@ -8,6 +8,7 @@
 using Microsoft.AspNetCore.Routing;
 using Microsoft.AspNetCore.Routing.Matching;
 using Umbraco.Cms.Core.Routing;
+using Umbraco.Cms.Web.Common.Attributes;
 using Umbraco.Cms.Web.Common.Controllers;
 using Umbraco.Cms.Web.Common.Routing;
 
@@ -35,7 +36,7 @@ private Endpoint GetNotFoundEndpoint()
                 // return the endpoint for the RenderController.Index action.
                 ControllerActionDescriptor descriptor = x.Metadata?.GetMetadata<ControllerActionDescriptor>();
                 return descriptor?.ControllerTypeInfo == typeof(RenderController)
-                    && descriptor?.ActionName == nameof(RenderController.Index);
+                       && descriptor?.ActionName == nameof(RenderController.Index);
             });
             return e;
         }
@@ -45,7 +46,7 @@ private Endpoint GetNotFoundEndpoint()
         public bool AppliesToEndpoints(IReadOnlyList<Endpoint> endpoints)
         {
             // Don't apply this filter to any endpoint group that is a controller route
-            // i.e. only dynamic routes.            
+            // i.e. only dynamic routes.
             foreach (Endpoint endpoint in endpoints)
             {
                 ControllerAttribute controller = endpoint.Metadata?.GetMetadata<ControllerAttribute>();
@@ -80,11 +81,15 @@ private static bool AllInvalid(CandidateSet candidates)
         {
             for (int i = 0; i < candidates.Count; i++)
             {
-                if (candidates.IsValidCandidate(i))
+                // We have to check if candidates needs to be ignored here
+                // So we dont return false when all endpoints are invalid
+                if (candidates.IsValidCandidate(i) &&
+                    candidates[i].Endpoint.Metadata.GetMetadata<IgnoreFromNotFoundSelectorPolicyAttribute>() is null)
                 {
                     return false;
                 }
             }
+
             return true;
         }
     }