Merge pull request #154 from takuabonn/feature/153

Fixes crash on Create User / Forgot User Flow with empty password.
RaythaHQ · Dec 4, 2023 · 982ddec · 982ddec
2 parents c2a576f + 88179ca
commit 982ddec
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Raytha.Application/Login/Commands/CompleteForgotPassword.cs b/src/Raytha.Application/Login/Commands/CompleteForgotPassword.cs
@@ -30,7 +30,7 @@ public Validator(IRaythaDbContext db)
             RuleFor(x => x.Id).NotEmpty();
             RuleFor(x => x).Custom((request, context) =>
             {
-                if (request.NewPassword.Length < PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH)
+                if (string.IsNullOrEmpty(request.NewPassword) || request.NewPassword.Length < PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH)
                 {
                     context.AddFailure("NewPassword", $"Password must be at least {PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH} characters.");
                     return;

diff --git a/src/Raytha.Application/Login/Commands/CreateUser.cs b/src/Raytha.Application/Login/Commands/CreateUser.cs
@@ -36,7 +36,7 @@ public Validator(IRaythaDbContext db)
             RuleFor(x => x.EmailAddress).NotEmpty().EmailAddress();
             RuleFor(x => x).Custom((request, context) =>
             {
-                if (request.Password.Length < PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH)
+                if (string.IsNullOrEmpty(request.Password) || request.Password.Length < PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH)
                 {
                     context.AddFailure("Password", $"Password must be at least {PasswordUtility.PASSWORD_MIN_CHARACTER_LENGTH} characters.");
                     return;