Browse Source

Fix 500 error on login with email not in DB (#212)

* Fix 500 error on login with email not in DB

* Fix type declarations in ServerInfoDto
Jaime Baez 3 years ago
parent
commit
e3449f9c8f

+ 6 - 2
server/apps/immich/src/api-v1/auth/auth.service.ts

@@ -17,7 +17,7 @@ export class AuthService {
     private immichJwtService: ImmichJwtService,
   ) {}
 
-  private async validateUser(loginCredential: LoginCredentialDto): Promise<UserEntity> {
+  private async validateUser(loginCredential: LoginCredentialDto): Promise<UserEntity | null> {
     const user = await this.userRepository.findOne(
       { email: loginCredential.email },
       {
@@ -35,9 +35,13 @@ export class AuthService {
       },
     );
 
+    if (!user) {
+      return null;
+    }
+
     const isAuthenticated = await this.validatePassword(user.password, loginCredential.password, user.salt);
 
-    if (user && isAuthenticated) {
+    if (isAuthenticated) {
       return user;
     }
 

+ 3 - 3
server/apps/immich/src/api-v1/server-info/dto/server-info.dto.ts

@@ -1,7 +1,7 @@
 export class ServerInfoDto {
-  diskSize: String;
-  diskUse: String;
-  diskAvailable: String;
+  diskSize: string;
+  diskUse: string;
+  diskAvailable: string;
   diskSizeRaw: number;
   diskUseRaw: number;
   diskAvailableRaw: number;