Skip to content

Commit 0b4d090

Browse files
Refactor IdentityHostingStartup: replace hardcoded authentication keys with constants for improved maintainability and readability.
1 parent a36c12d commit 0b4d090

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

src/XtremeIdiots.Portal.Web/Areas/Identity/IdentityHostingStartup.cs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ namespace XtremeIdiots.Portal.Web.Areas.Identity;
1717

1818
public class IdentityHostingStartup : IHostingStartup
1919
{
20+
private const string AuthClientIdKey = "XtremeIdiots:Auth:ClientId";
21+
private const string AuthClientSecretKey = "XtremeIdiots:Auth:ClientSecret";
22+
2023
private const int SecurityStampValidationIntervalMinutes = 15;
2124
private const int CookieExpirationDays = 7;
2225
private const string ApplicationName = "portal";
@@ -40,8 +43,8 @@ private static void ValidateConfiguration(IConfiguration configuration)
4043
{
4144
var requiredKeys = new[]
4245
{
43-
"xtremeidiots_auth_client_id",
44-
"xtremeidiots_auth_client_secret",
46+
AuthClientIdKey,
47+
AuthClientSecretKey,
4548
"sql_connection_string"
4649
};
4750

@@ -103,8 +106,8 @@ private static void ConfigureAuthentication(IServiceCollection services, IConfig
103106
})
104107
.AddOAuth(OAuthSchemeName, options =>
105108
{
106-
options.ClientId = configuration["xtremeidiots_auth_client_id"] ?? throw new InvalidOperationException("OAuth client ID is required");
107-
options.ClientSecret = configuration["xtremeidiots_auth_client_secret"] ?? throw new InvalidOperationException("OAuth client secret is required");
109+
options.ClientId = configuration[AuthClientIdKey] ?? throw new InvalidOperationException("OAuth client ID is required");
110+
options.ClientSecret = configuration[AuthClientSecretKey] ?? throw new InvalidOperationException("OAuth client secret is required");
108111

109112
options.CallbackPath = new PathString("/signin-xtremeidiots");
110113

0 commit comments

Comments
 (0)