HAM/Berufsschule_HAM
1
0
Fork 1
mirror of https://github.com/LD-Reborn/Berufsschule_HAM.git synced 2025-12-20 06:51:55 +00:00
Code Issues Packages Projects 1 Releases Wiki Activity

Fixed uncaught exception when user not found

Browse Source
This commit is contained in:
LD-Reborn
2025-10-03 16:40:16 +02:00
parent 0bb5f650b6
commit a5a08e5876
1 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/Services/LdapService.cs
View File

@@ -142,6 +142,8 @@ public partial class LdapService : IDisposable
public async Task<UserAuthenticationResult> AuthenticateUser(string username, string password) public async Task<UserAuthenticationResult> AuthenticateUser(string username, string password)
{ {
await ConnectAndBind(); await ConnectAndBind();
try
{
try try
{ {
UserModel user = await GetUserByUidAsync(username); UserModel user = await GetUserByUidAsync(username);
@@ -151,10 +153,15 @@ public partial class LdapService : IDisposable
} }
if (CompareStringToSha256(password, user.UserPassword)) if (CompareStringToSha256(password, user.UserPassword))
{ {
return new() { Success = true}; return new() { Success = true };
} }
return new() { Success = false, AuthenticationState = UserNotAuthenticatedReason.InvalidCredentials }; return new() { Success = false, AuthenticationState = UserNotAuthenticatedReason.InvalidCredentials };
} }
catch (InvalidOperationException)
{
return new() { Success = false, AuthenticationState = UserNotAuthenticatedReason.InvalidCredentials };
}
}
catch (LdapException) catch (LdapException)
{ {
return new() { Success = false, AuthenticationState = UserNotAuthenticatedReason.InvalidCredentials }; return new() { Success = false, AuthenticationState = UserNotAuthenticatedReason.InvalidCredentials };