Merge pull request #238 from ministryofjustice/hotfix/username

allow additional characters in the username field to support genuine email addresses
ministryofjustice · Sep 11, 2024 · 86bff19 · 86bff19
2 parents caf48bb + 4a0b840
commit 86bff19
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/src/Infrastructure/Configurations/IdentitySettings.cs b/src/Infrastructure/Configurations/IdentitySettings.cs
@@ -51,4 +51,6 @@ public class IdentitySettings : IIdentitySettings
     /// </summary>
     public int MaxFailedAccessAttempts { get; set; } = 5;
 
+    public string AllowedUserNameCharacters { get; set; } = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-._@+'";
+
 }
diff --git a/src/Infrastructure/DependencyInjection.cs b/src/Infrastructure/DependencyInjection.cs
@@ -230,6 +230,7 @@ private static IServiceCollection AddAuthenticationService(this IServiceCollecti
 
             // User settings
             options.User.RequireUniqueEmail = true;
+            options.User.AllowedUserNameCharacters = identitySettings.AllowedUserNameCharacters;
             //options.Tokens.EmailConfirmationTokenProvider = "Email";
         });
 

diff --git a/src/Server.UI/appsettings.json b/src/Server.UI/appsettings.json
@@ -72,7 +72,8 @@
     "RequireLowerCase": true,
     "DefaultLockoutTimeSpan": 30,
     "MaxFailedAccessAttempts": 5,
-    "SecureCookies": true
+    "SecureCookies": true,
+    "AllowedUserNameCharacters": "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-._@+'"
   },
   "Notify": {
     "ApiKey": "",