Browse Source

Merge pull request #3727 from K900/patch-1

Fix #3624

(cherry picked from commit 06db5f8bca0b27663714ae7f8dc39084749a191e)
Signed-off-by: Joshua M. Boniface <joshua@boniface.me>
Anthony Lavado 4 years ago
parent
commit
e276532f2f
1 changed files with 1 additions and 6 deletions
  1. 1 6
      Jellyfin.Server.Implementations/Users/UserManager.cs

+ 1 - 6
Jellyfin.Server.Implementations/Users/UserManager.cs

@@ -592,18 +592,13 @@ namespace Jellyfin.Server.Implementations.Users
             }
 
             var defaultName = Environment.UserName;
-            if (string.IsNullOrWhiteSpace(defaultName))
+            if (string.IsNullOrWhiteSpace(defaultName) || !IsValidUsername(defaultName))
             {
                 defaultName = "MyJellyfinUser";
             }
 
             _logger.LogWarning("No users, creating one with username {UserName}", defaultName);
 
-            if (!IsValidUsername(defaultName))
-            {
-                throw new ArgumentException("Provided username is not valid!", defaultName);
-            }
-
             var newUser = CreateUser(defaultName);
             newUser.SetPermission(PermissionKind.IsAdministrator, true);
             newUser.SetPermission(PermissionKind.EnableContentDeletion, true);