Skip to content

Commit

Permalink
KEYCLOAK-1280: i18n logging for org.keycloak.services.scheduled
Browse files Browse the repository at this point in the history
  • Loading branch information
ssilvert committed Jan 21, 2016
1 parent 446cd37 commit c6dd0d5
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -393,4 +393,12 @@ public interface ServicesLogger extends BasicLogger {
@LogMessage(level = ERROR)
@Message(id=88, value="Failed to send execute actions email")
void failedToSendActionsEmail(@Cause EmailException e);

@LogMessage(level = ERROR)
@Message(id=89, value="Failed to run scheduled task %s")
void failedToRunScheduledTask(@Cause Throwable t, String taskClass);

@LogMessage(level = ERROR)
@Message(id=90, value="Failed to close ProviderSession")
void failedToCloseProviderSession(@Cause Throwable t);
}
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
package org.keycloak.services.scheduled;

import org.jboss.logging.Logger;
import org.keycloak.models.KeycloakSession;
import org.keycloak.models.KeycloakSessionFactory;
import org.keycloak.services.ServicesLogger;

/**
* @author <a href="mailto:sthorger@redhat.com">Stian Thorgersen</a>
*/
public class ScheduledTaskRunner implements Runnable {

private static final Logger logger = Logger.getLogger(ScheduledTaskRunner.class);
private static final ServicesLogger logger = ServicesLogger.ROOT_LOGGER;

private final KeycloakSessionFactory sessionFactory;
private final ScheduledTask task;
Expand All @@ -29,14 +29,14 @@ public void run() {

logger.debug("Executed scheduled task " + task.getClass().getSimpleName());
} catch (Throwable t) {
logger.error("Failed to run scheduled task " + task.getClass().getSimpleName(), t);
logger.failedToRunScheduledTask(t, task.getClass().getSimpleName());

session.getTransaction().rollback();
} finally {
try {
session.close();
} catch (Throwable t) {
logger.error("Failed to close ProviderSession", t);
logger.failedToCloseProviderSession(t);
}
}
}
Expand Down

0 comments on commit c6dd0d5

Please sign in to comment.