Merge pull request #22 from TechnologyEnhancedLearning/RC

Merge RC Changes to CI

Author: AnjuJose011

Auth/LearningHub.Nhs.Auth/Controllers/AccountController.cs

@@ -45,6 +45,7 @@ public class AccountController : IdentityServerController
         private readonly IAuthenticationSchemeProvider schemeProvider;
         private readonly LearningHubAuthConfig authConfig;
         private readonly WebSettings webSettings;
+        private readonly ILogger logger;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="AccountController"/> class.
@@ -55,6 +56,7 @@ public class AccountController : IdentityServerController
         /// <param name="events">events parameter.</param>
         /// <param name="userService">userService parameter.</param>
         /// <param name="webSettings">webSettings parameter.</param>
+        /// <param name="logger">ILogger instance.</param>
         /// <param name="authConfig">Auth service config.</param>
         /// <param name="cacheService">Cacje service config.</param>
         public AccountController(
@@ -64,6 +66,7 @@ public AccountController(
             IEventService events,
             IUserService userService,
             WebSettings webSettings,
+            ILogger<AccountController> logger,
             IOptions<LearningHubAuthConfig> authConfig,
             ICacheService cacheService)
             : base(userService, events, clientStore, webSettings, cacheService)
@@ -72,6 +75,7 @@ public AccountController(
             this.schemeProvider = schemeProvider;
             this.authConfig = authConfig?.Value;
             this.webSettings = webSettings;
+            this.logger = logger;
     }
 
         /// <summary>
@@ -336,7 +340,7 @@ private async Task<LoginViewModel> BuildLoginViewModelAsync(string returnUrl)
         {
             var context = await this.interaction.GetAuthorizationContextAsync(returnUrl);
             LoginClientTemplate loginClientTemplate = null;
-            if (context?.Client.ClientId != null && this.authConfig.IdsClients.ContainsKey(context.Client.ClientId))
+            if (context?.Client?.ClientId != null && this.authConfig.IdsClients.ContainsKey(context.Client.ClientId))
             {
                 loginClientTemplate = this.authConfig.IdsClients[context.Client.ClientId];
 
@@ -383,7 +387,7 @@ private async Task<LoginViewModel> BuildLoginViewModelAsync(string returnUrl)
 
             var allowLocal = true;
 
-            if (context?.Client.ClientId != null)
+            if (context?.Client?.ClientId != null)
             {
                 var client = await this.ClientStore.FindEnabledClientByIdAsync(context.Client.ClientId);
                 if (client != null)
@@ -396,6 +400,12 @@ private async Task<LoginViewModel> BuildLoginViewModelAsync(string returnUrl)
                 }
             }
 
+            if (context == null || loginClientTemplate == null)
+            {
+                string message = context == null ? "context" : loginClientTemplate == null ? "clientTemplate" : string.Empty;
+                this.logger.LogWarning($"return url has no {message} : {returnUrl}");
+            }
+
             return new LoginViewModel
             {
                 AllowRememberLogin = loginClientTemplate?.AllowRememberLogin ?? AccountOptions.AllowRememberLogin,
@@ -404,7 +414,7 @@ private async Task<LoginViewModel> BuildLoginViewModelAsync(string returnUrl)
                 Username = context?.LoginHint,
                 ExternalProviders = providers.ToArray(),
                 LoginClientTemplate = loginClientTemplate ?? new LoginClientTemplate(),
-                ClientId = context?.Client.ClientId,
+                ClientId = context?.Client?.ClientId,
             };
         }