Do not serialize the Logger instance

James Moger

2013-12-30 e92cef5abcbf6989e8ce58093b34b3ed909da4b8

 src/main/java/com/gitblit/wicket/pages/BasePage.java

@@ -65,20 +65,25 @@


public abstract class BasePage extends SessionPage {



   private final Logger logger;

   private transient Logger logger;



   private transient TimeUtils timeUtils;



   public BasePage() {

      super();

      logger = LoggerFactory.getLogger(getClass());

      customizeHeader();

   }



   public BasePage(PageParameters params) {

      super(params);

      logger = LoggerFactory.getLogger(getClass());

      customizeHeader();

   }



   protected Logger logger() {

      if (logger == null) {

         logger = LoggerFactory.getLogger(getClass());

      }

      return logger;

   }



   private void customizeHeader() {

@@ -171,7 +176,7 @@
         case NONE:

            break;

         default:

            logger.warn(getClass().getSimpleName() + ": unhandled LastModified type " + cacheControl.value());

            logger().warn(getClass().getSimpleName() + ": unhandled LastModified type " + cacheControl.value());

            break;

         }

      }

@@ -411,7 +416,7 @@
   }



   public void warn(String message, Throwable t) {

      logger.warn(message, t);

      logger().warn(message, t);

   }



   public void error(String message, boolean redirect) {

@@ -428,9 +433,9 @@


   public void error(String message, Throwable t, Class<? extends Page> toPage, PageParameters params) {

      if (t == null) {

         logger.error(message  + " for " + GitBlitWebSession.get().getUsername());

         logger().error(message  + " for " + GitBlitWebSession.get().getUsername());

      } else {

         logger.error(message  + " for " + GitBlitWebSession.get().getUsername(), t);

         logger().error(message  + " for " + GitBlitWebSession.get().getUsername(), t);

      }

      if (toPage != null) {

         GitBlitWebSession.get().cacheErrorMessage(message);

@@ -443,7 +448,7 @@
   }



   public void authenticationError(String message) {

      logger.error(getRequest().getURL() + " for " + GitBlitWebSession.get().getUsername());

      logger().error(getRequest().getURL() + " for " + GitBlitWebSession.get().getUsername());

      if (!GitBlitWebSession.get().isLoggedIn()) {

         // cache the request if we have not authenticated.

         // the request will continue after authentication.