diff options
| author | Bond-009 <bond.009@outlook.com> | 2021-04-03 00:26:56 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-03 00:26:56 +0200 |
| commit | a92736baade6cb722e3a87d074b0c1766ddb5557 (patch) | |
| tree | 6e8c4fa3510f4db61847075969c5bf87e2ce67f0 /Jellyfin.Server.Implementations/Users/UserManager.cs | |
| parent | fd66ca25cd07db9f85cc24c8df2909426866b468 (diff) | |
| parent | 03cc6b1d782067e58975dbea4b61e42a3c04e1aa (diff) | |
Merge pull request #5174 from barronpm/authenticateuser-cleanup
Diffstat (limited to 'Jellyfin.Server.Implementations/Users/UserManager.cs')
| -rw-r--r-- | Jellyfin.Server.Implementations/Users/UserManager.cs | 21 |
1 files changed, 6 insertions, 15 deletions
diff --git a/Jellyfin.Server.Implementations/Users/UserManager.cs b/Jellyfin.Server.Implementations/Users/UserManager.cs index 50d7612f2..a2c11cb8a 100644 --- a/Jellyfin.Server.Implementations/Users/UserManager.cs +++ b/Jellyfin.Server.Implementations/Users/UserManager.cs @@ -407,27 +407,18 @@ namespace Jellyfin.Server.Implementations.Users } var user = Users.FirstOrDefault(i => string.Equals(username, i.Username, StringComparison.OrdinalIgnoreCase)); - bool success; - IAuthenticationProvider? authenticationProvider; + var authResult = await AuthenticateLocalUser(username, password, user, remoteEndPoint) + .ConfigureAwait(false); + var authenticationProvider = authResult.authenticationProvider; + var success = authResult.success; - if (user != null) - { - var authResult = await AuthenticateLocalUser(username, password, user, remoteEndPoint) - .ConfigureAwait(false); - authenticationProvider = authResult.authenticationProvider; - success = authResult.success; - } - else + if (user == null) { - var authResult = await AuthenticateLocalUser(username, password, null, remoteEndPoint) - .ConfigureAwait(false); - authenticationProvider = authResult.authenticationProvider; string updatedUsername = authResult.username; - success = authResult.success; if (success && authenticationProvider != null - && !(authenticationProvider is DefaultAuthenticationProvider)) + && authenticationProvider is not DefaultAuthenticationProvider) { // Trust the username returned by the authentication provider username = updatedUsername; |
