diff --git a/src/Umbraco.Web.BackOffice/Authorization/AdminUsersHandler.cs b/src/Umbraco.Web.BackOffice/Authorization/AdminUsersHandler.cs index 14f76c48dd..ff7ba13937 100644 --- a/src/Umbraco.Web.BackOffice/Authorization/AdminUsersHandler.cs +++ b/src/Umbraco.Web.BackOffice/Authorization/AdminUsersHandler.cs @@ -34,15 +34,15 @@ namespace Umbraco.Web.BackOffice.Authorization protected override Task HandleRequirementAsync(AuthorizationHandlerContext context, AdminUsersRequirement requirement) { - int[] userIds; - var queryString = _httpContextAcessor.HttpContext?.Request.Query[requirement.QueryStringName]; if (!queryString.HasValue) { - // don't set status since we cannot determine ourselves + // must succeed this requirement since we cannot process it + context.Succeed(requirement); return Task.CompletedTask; } + int[] userIds; if (int.TryParse(queryString, out var userId)) { userIds = new[] { userId }; @@ -52,7 +52,8 @@ namespace Umbraco.Web.BackOffice.Authorization var ids = _httpContextAcessor.HttpContext.Request.Query.Where(x => x.Key == requirement.QueryStringName).ToList(); if (ids.Count == 0) { - // don't set status since we cannot determine ourselves + // must succeed this requirement since we cannot process it + context.Succeed(requirement); return Task.CompletedTask; } userIds = ids.Select(x => x.Value.TryConvertTo()).Where(x => x.Success).Select(x => x.Result).ToArray(); @@ -60,7 +61,8 @@ namespace Umbraco.Web.BackOffice.Authorization if (userIds.Length == 0) { - // don't set status since we cannot determine ourselves + // must succeed this requirement since we cannot process it + context.Succeed(requirement); return Task.CompletedTask; } diff --git a/src/Umbraco.Web.BackOffice/Authorization/ContentPermissionsQueryStringHandler.cs b/src/Umbraco.Web.BackOffice/Authorization/ContentPermissionsQueryStringHandler.cs index 4d9e5baf1d..bba502e78f 100644 --- a/src/Umbraco.Web.BackOffice/Authorization/ContentPermissionsQueryStringHandler.cs +++ b/src/Umbraco.Web.BackOffice/Authorization/ContentPermissionsQueryStringHandler.cs @@ -41,7 +41,8 @@ namespace Umbraco.Web.BackOffice.Authorization { if (!_httpContextAccessor.HttpContext.Request.Query.TryGetValue(requirement.QueryStringName, out var routeVal)) { - // don't set status since we cannot determine ourselves + // must succeed this requirement since we cannot process it + context.Succeed(requirement); return Task.CompletedTask; } else diff --git a/src/Umbraco.Web.BackOffice/Authorization/MediaPermissionsQueryStringHandler.cs b/src/Umbraco.Web.BackOffice/Authorization/MediaPermissionsQueryStringHandler.cs index a4f8d32ee6..319d5969fd 100644 --- a/src/Umbraco.Web.BackOffice/Authorization/MediaPermissionsQueryStringHandler.cs +++ b/src/Umbraco.Web.BackOffice/Authorization/MediaPermissionsQueryStringHandler.cs @@ -33,7 +33,8 @@ namespace Umbraco.Web.BackOffice.Authorization { if (!_httpContextAccessor.HttpContext.Request.Query.TryGetValue(requirement.QueryStringName, out var routeVal)) { - // don't set status since we cannot determine ourselves + // must succeed this requirement since we cannot process it + context.Succeed(requirement); return Task.CompletedTask; }