KEYCLOAK-1280: i18n logging for

org.keycloak.authentication.authenticators.resetcred
keycloak · Jan 21, 2016 · 7bc74aa · 7bc74aa
1 parent dfb92db
commit 7bc74aa
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/.../java/org/keycloak/authentication/authenticators/resetcred/ResetCredentialChooseUser.java b/.../java/org/keycloak/authentication/authenticators/resetcred/ResetCredentialChooseUser.java
@@ -1,6 +1,5 @@
 package org.keycloak.authentication.authenticators.resetcred;
 
-import org.jboss.logging.Logger;
 import org.keycloak.Config;
 import org.keycloak.authentication.AuthenticationFlowContext;
 import org.keycloak.authentication.AuthenticationFlowError;
@@ -17,6 +16,7 @@
 import org.keycloak.models.RealmModel;
 import org.keycloak.models.UserModel;
 import org.keycloak.provider.ProviderConfigProperty;
+import org.keycloak.services.ServicesLogger;
 import org.keycloak.services.messages.Messages;
 
 import javax.ws.rs.core.MultivaluedMap;
@@ -29,7 +29,7 @@
  */
 public class ResetCredentialChooseUser implements Authenticator, AuthenticatorFactory {
 
-    protected static Logger logger = Logger.getLogger(ResetCredentialChooseUser.class);
+    protected static ServicesLogger logger = ServicesLogger.ROOT_LOGGER;
 
     public static final String PROVIDER_ID = "reset-credentials-choose-user";
 

diff --git a/.../main/java/org/keycloak/authentication/authenticators/resetcred/ResetCredentialEmail.java b/.../main/java/org/keycloak/authentication/authenticators/resetcred/ResetCredentialEmail.java
@@ -1,6 +1,5 @@
 package org.keycloak.authentication.authenticators.resetcred;
 
-import org.jboss.logging.Logger;
 import org.keycloak.Config;
 import org.keycloak.authentication.AuthenticationFlowContext;
 import org.keycloak.authentication.AuthenticationFlowError;
@@ -22,6 +21,7 @@
 import org.keycloak.models.utils.FormMessage;
 import org.keycloak.models.utils.HmacOTP;
 import org.keycloak.provider.ProviderConfigProperty;
+import org.keycloak.services.ServicesLogger;
 import org.keycloak.services.messages.Messages;
 
 import javax.ws.rs.core.Response;
@@ -36,7 +36,7 @@
 public class ResetCredentialEmail implements Authenticator, AuthenticatorFactory {
     public static final String RESET_CREDENTIAL_SECRET = "RESET_CREDENTIAL_SECRET";
 
-    protected static Logger logger = Logger.getLogger(ResetCredentialEmail.class);
+    protected static ServicesLogger logger = ServicesLogger.ROOT_LOGGER;
 
     public static final String PROVIDER_ID = "reset-credential-email";
 
@@ -83,7 +83,7 @@ public void authenticate(AuthenticationFlowContext context) {
                     .detail(Details.USERNAME, username)
                     .user(user)
                     .error(Errors.EMAIL_SEND_FAILED);
-            logger.error("Failed to send password reset email", e);
+            logger.failedToSendPwdResetEmail(e);
             Response challenge = context.form()
                     .setError(Messages.EMAIL_SENT_ERROR)
                     .createErrorPage();

diff --git a/services/src/main/java/org/keycloak/services/ServicesLogger.java b/services/src/main/java/org/keycloak/services/ServicesLogger.java
@@ -27,6 +27,7 @@
 import static org.jboss.logging.Logger.Level.ERROR;
 import static org.jboss.logging.Logger.Level.INFO;
 import static org.jboss.logging.Logger.Level.WARN;
+import org.keycloak.email.EmailException;
 import org.keycloak.models.ModelDuplicateException;
 
 
@@ -139,4 +140,8 @@ public interface ServicesLogger extends BasicLogger {
     @LogMessage(level = ERROR)
     @Message(id=25, value="Error when validating client assertion")
     void errorValidatingAssertion(@Cause Throwable t);
+
+    @LogMessage(level = ERROR)
+    @Message(id=26, value="Failed to send password reset email")
+    void failedToSendPwdResetEmail(@Cause EmailException e);
 }