Merge pull request #144 from Code-4-Community/kjung/fix

Save emails as lowercase on signup
Code-4-Community · Apr 10, 2024 · 9cb8e38 · 9cb8e38
2 parents 2111030 + 84ffc70
commit 9cb8e38
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/service/src/main/java/com/codeforcommunity/dataaccess/AuthDatabaseOperations.java b/service/src/main/java/com/codeforcommunity/dataaccess/AuthDatabaseOperations.java
@@ -131,8 +131,9 @@ public boolean isValidLogin(String email, String pass) {
    *     table.
    */
   public UsersRecord createNewUser(NewUserRequest request) {
-    String email = request.getEmail();
-    boolean emailUsed = db.fetchExists(db.selectFrom(USERS).where(USERS.EMAIL.equalIgnoreCase(email)));
+    String email = request.getEmail().toLowerCase();
+    boolean emailUsed =
+        db.fetchExists(db.selectFrom(USERS).where(USERS.EMAIL.equalIgnoreCase(email)));
     if (emailUsed) {
       throw new EmailAlreadyInUseException(email);
     }
@@ -143,7 +144,7 @@ public UsersRecord createNewUser(NewUserRequest request) {
 
     UsersRecord newUser = db.newRecord(USERS);
     addAddressDataToUserRecord(newUser, request.getLocation());
-    newUser.setEmail(request.getEmail());
+    newUser.setEmail(email);
     newUser.setPassHash(Passwords.createHash(request.getPassword()));
     newUser.setPrivilegeLevel(PrivilegeLevel.STANDARD);
     newUser.setPhotoRelease(request.getPhotoRelease());