diff options
| author | Niels van Velzen <nielsvanvelzen@users.noreply.github.com> | 2025-11-27 16:31:05 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-11-27 16:31:05 +0100 |
| commit | 19c232809ed3b2f0a740239a863b66bebe41e32f (patch) | |
| tree | 2d349a8b7c408b9cf6b102d7af61b19c8393a623 /Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs | |
| parent | 301f65af483d349bcbc76866a5e6ad17588dbd76 (diff) | |
| parent | d43db230fa1ff6f371a9ab16063152f5b632e2e9 (diff) | |
Merge pull request #14950 from nielsvanvelzen/security-remove-has-password
Deprecate HasPassword property on UserDto
Diffstat (limited to 'Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs')
| -rw-r--r-- | Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs b/Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs index 35c43b176..446849b6f 100644 --- a/Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs +++ b/Jellyfin.Server.Implementations/Users/DefaultAuthenticationProvider.cs @@ -59,7 +59,7 @@ namespace Jellyfin.Server.Implementations.Users } // As long as jellyfin supports password-less users, we need this little block here to accommodate - if (!HasPassword(resolvedUser) && string.IsNullOrEmpty(password)) + if (string.IsNullOrEmpty(resolvedUser.Password) && string.IsNullOrEmpty(password)) { return Task.FromResult(new ProviderAuthenticationResult { @@ -94,10 +94,6 @@ namespace Jellyfin.Server.Implementations.Users } /// <inheritdoc /> - public bool HasPassword(User user) - => !string.IsNullOrEmpty(user?.Password); - - /// <inheritdoc /> public Task ChangePassword(User user, string newPassword) { if (string.IsNullOrEmpty(newPassword)) |