Issue #14 Chore: Logger type changed from error to debug

Sunbird-Lern · Dec 27, 2017 · deced78 · deced78
1 parent 1801a11
commit deced78
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/...ovider/src/main/java/org/sunbird/keycloak/login/phone/AbstractPhoneFormAuthenticator.java b/...ovider/src/main/java/org/sunbird/keycloak/login/phone/AbstractPhoneFormAuthenticator.java
@@ -42,7 +42,7 @@ public abstract class AbstractPhoneFormAuthenticator extends AbstractUsernameFor
 
     public boolean validateUserAndPassword(AuthenticationFlowContext context, MultivaluedMap<String, String> inputData) {
         String username = inputData.getFirst(AuthenticationManager.FORM_USERNAME);
-        logger.error("AbstractPhoneFormAuthenticator@validateUserAndPassword - Username -" + username);
+        logger.debug("AbstractPhoneFormAuthenticator@validateUserAndPassword - Username -" + username);
 
         if (username == null) {
             context.getEvent().error(Errors.USER_NOT_FOUND);
@@ -105,8 +105,6 @@ private UserModel getUser(AuthenticationFlowContext context, String username) {
             List<UserModel> userModels = session.users().searchForUserByUserAttribute(KeycloakSmsAuthenticatorConstants.ATTR_MOBILE, username, context.getRealm());
 
             if (userModels != null && userModels.size() > 0) {
-                logger.error("AbstractPhoneFormAuthenticator@getUser : Users List - " + userModels.size());
-                logger.error("AbstractPhoneFormAuthenticator@getUser : First User - " + userModels.get(0).getUsername());
                 return userModels.get(0);
             } else {
                 return KeycloakModelUtils.findUserByNameOrEmail(context.getSession(), context.getRealm(), username);

diff --git a/keycloak/sms-provider/src/main/java/org/sunbird/keycloak/login/phone/PhonePasswordForm.java b/keycloak/sms-provider/src/main/java/org/sunbird/keycloak/login/phone/PhonePasswordForm.java
@@ -44,7 +44,7 @@ public class PhonePasswordForm extends AbstractPhoneFormAuthenticator implements
 
     @Override
     public void action(AuthenticationFlowContext context) {
-        logger.error("PhonePasswordForm@action - called");
+        logger.debug("PhonePasswordForm@action - called");
         MultivaluedMap<String, String> formData = context.getHttpRequest().getDecodedFormParameters();
         if (formData.containsKey("cancel")) {
             context.cancelLogin();
@@ -57,7 +57,7 @@ public void action(AuthenticationFlowContext context) {
     }
 
     protected boolean validateForm(AuthenticationFlowContext context, MultivaluedMap<String, String> formData) {
-        logger.error("PhonePasswordForm@validateForm - called");
+        logger.debug("PhonePasswordForm@validateForm - called");
         return validateUserAndPassword(context, formData);
     }
 

diff --git a/...sms-provider/src/main/java/org/sunbird/keycloak/login/phone/PhonePasswordFormFactory.java b/...sms-provider/src/main/java/org/sunbird/keycloak/login/phone/PhonePasswordFormFactory.java
@@ -43,7 +43,7 @@ public class PhonePasswordFormFactory implements AuthenticatorFactory {
 
     @Override
     public Authenticator create(KeycloakSession session) {
-        logger.error("PhonePasswordFormFactory@create - PhonePasswordFormFactory is created");
+        logger.debug("PhonePasswordFormFactory@create - PhonePasswordFormFactory is created");
         return SINGLETON;
     }