V13: The login screen does not display external login errors (#15715)

* show external login errors as a subheadline on the login screen * fix: migrate error handling from v12 for external login * cleanup unused client-side javascript
umbraco · Feb 14, 2024 · 1235521 · 1235521
1 parent 3d7e79c
commit 1235521
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 7 deletions.
diff --git a/src/Umbraco.Cms.StaticAssets/umbraco/UmbracoBackOffice/Default.cshtml b/src/Umbraco.Cms.StaticAssets/umbraco/UmbracoBackOffice/Default.cshtml
@@ -16,7 +16,6 @@
 @inject IOptions<GlobalSettings> globalSettings
 @inject IRuntimeMinifier runtimeMinifier
 @inject IProfilerHtml profilerHtml
-@inject IBackOfficeExternalLoginProviders externalLogins
 @{
     bool.TryParse(Context.Request.Query["umbDebug"], out bool isDebug);
     var backOfficePath = globalSettings.Value.GetBackOfficePath(hostingEnvironment);
@@ -114,8 +113,6 @@
 
     <script>
         document.angularReady = function(app) {
-            @await Html.AngularValueExternalLoginInfoScriptAsync(externalLogins, ViewData.GetExternalSignInProviderErrors()!)
-            @Html.AngularValueResetPasswordCodeInfoScript(ViewData[ViewDataExtensions.TokenPasswordResetCode]!)
             @await Html.AngularValueTinyMceAssetsAsync(runtimeMinifier)
             //required for the noscript trick
             document.getElementById("mainwrapper").style.display = "inherit";

diff --git a/src/Umbraco.Cms.StaticAssets/umbraco/UmbracoLogin/Index.cshtml b/src/Umbraco.Cms.StaticAssets/umbraco/UmbracoLogin/Index.cshtml
@@ -32,6 +32,7 @@
     var disableLocalLogin = ExternalLogins.HasDenyLocalLogin();
     var externalLoginsUrl = LinkGenerator.GetPathByAction(nameof(BackOfficeController.ExternalLogin), ControllerExtensions.GetControllerName<BackOfficeController>(), new { area = Constants.Web.Mvc.BackOfficeArea });
     var externalLoginProviders = await ExternalLogins.GetBackOfficeProvidersAsync();
+    var externalSignInErrors = ViewData.GetExternalSignInProviderErrors();
 }
 @addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers
 
@@ -106,6 +107,15 @@
                 custom-view="@provider.ExternalLoginProvider.Options.CustomBackOfficeView">
             </umb-external-login-provider>
         }
+        @if (externalSignInErrors?.Errors?.Count() > 0)
+        {
+            <div slot="subheadline" style="color:var(--uui-color-danger-standalone)">
+                @foreach (var error in externalSignInErrors.Errors)
+                {
+                    <div>@error</div>
+                }
+            </div>
+        }
     </umb-auth>
 </umb-backoffice-icon-registry>
 

diff --git a/src/Umbraco.Web.BackOffice/Controllers/BackOfficeController.cs b/src/Umbraco.Web.BackOffice/Controllers/BackOfficeController.cs
@@ -426,7 +426,8 @@ public async Task<IActionResult> ExternalLinkLoginCallback()
         if (user == null)
         {
             // ... this should really not happen
-            return RedirectToLogin(new { flow = "external-login", status = "localUserNotFound" });
+            TempData[ViewDataExtensions.TokenExternalSignInError] = new[] { "Local user does not exist" };
+            return RedirectToLogin(new { flow = "external-login", status = "localUserNotFound", logout = "true"});
         }
 
         ExternalLoginInfo? info =
@@ -435,7 +436,9 @@ public async Task<IActionResult> ExternalLinkLoginCallback()
         if (info == null)
         {
             // Add error and redirect for it to be displayed
-            return RedirectToLogin(new { flow = "external-login", status = "externalLoginInfoNotFound" });
+            TempData[ViewDataExtensions.TokenExternalSignInError] =
+                new[] { "An error occurred, could not get external login info" };
+            return RedirectToLogin(new { flow = "external-login", status = "externalLoginInfoNotFound", logout = "true"});
         }
 
         IdentityResult addLoginResult = await _userManager.AddLoginAsync(user, info);
@@ -448,7 +451,8 @@ public async Task<IActionResult> ExternalLinkLoginCallback()
         }
 
         // Add errors and redirect for it to be displayed
-        return RedirectToLogin(new { flow = "external-login", status = "failed" });
+        TempData[ViewDataExtensions.TokenExternalSignInError] = addLoginResult.Errors;
+        return RedirectToLogin(new { flow = "external-login", status = "failed", logout = "true" });
     }
 
     /// <summary>
@@ -469,8 +473,9 @@ public async Task<IActionResult> ExternalLinkLoginCallback()
                 _httpContextAccessor.HttpContext,
                 ViewDataExtensions.TokenExternalSignInError,
                 _jsonSerializer) ||
-            ViewData.FromTempData(TempData, ViewDataExtensions.TokenExternalSignInError) || ViewData.FromTempData(TempData, ViewDataExtensions.TokenPasswordResetCode))
+            ViewData.FromTempData(TempData, ViewDataExtensions.TokenExternalSignInError))
         {
+            // Return early to let the client side handle the messaging
             return defaultResponse();
         }
 

diff --git a/src/Umbraco.Web.Common/Extensions/ViewDataExtensions.cs b/src/Umbraco.Web.Common/Extensions/ViewDataExtensions.cs
@@ -122,9 +122,11 @@ public static bool FromTempData(this ViewDataDictionary viewData, ITempDataDicti
         this ViewDataDictionary viewData,
         BackOfficeExternalLoginProviderErrors errors) => viewData[TokenExternalSignInError] = errors;
 
+    [Obsolete("This is deprecated and will be removed in V15")]
     public static string? GetPasswordResetCode(this ViewDataDictionary viewData) =>
         (string?)viewData[TokenPasswordResetCode];
 
+    [Obsolete("This is deprecated and will be removed in V15")]
     public static void SetPasswordResetCode(this ViewDataDictionary viewData, string value) =>
         viewData[TokenPasswordResetCode] = value;