Commit a140136f authored by Tomas Madeja's avatar Tomas Madeja
Browse files

feat: add input checking

parent 769ada5a

service/src/main/java/cz/muni/fi/pa165/icehockeymanager/services/UserAuthServiceImpl.java

+12 −0
Original line number Diff line number Diff line
@@ -5,6 +5,7 @@ import cz.muni.fi.pa165.icehockeymanager.model.User; 
import org.springframework.beans.factory.annotation.Autowired;

import org.springframework.stereotype.Service;



import java.util.Objects;

import java.util.Optional;

import java.util.concurrent.atomic.AtomicBoolean;
@@ -25,6 +26,11 @@ public class UserAuthServiceImpl implements UserAuthService{ 
    }



    public boolean authenticateUser(String username, String password) {

        Objects.requireNonNull(username);

        Objects.requireNonNull(password);

        requireNonEmpty(username, "username");

        requireNonEmpty(password, "password");



        Optional<User> userOpt = userAuthDao.findByUsername(username);



        AtomicBoolean isCorrectPassword = new AtomicBoolean(false);
@@ -36,4 +42,10 @@ public class UserAuthServiceImpl implements UserAuthService{ 


        return isCorrectPassword.get();

    }



    private void requireNonEmpty(String argument, String paramName) {

        if (argument.isEmpty()) {

            throw new IllegalArgumentException(paramName);

        }

    }

}