Do not log empty messages in the receive pack

James Moger

2014-02-28 da99418dc677061610a1ccef5b2207d065600da8

 src/main/java/com/gitblit/git/GitblitReceivePack.java

@@ -87,9 +87,9 @@


   protected GroovyScriptEngine gse;



   private final IStoredSettings settings;

   protected final IStoredSettings settings;



   private final IGitblit gitblit;

   protected final IGitblit gitblit;



   public GitblitReceivePack(

         IGitblit gitblit,

@@ -224,13 +224,13 @@
         }

      }



      // reset branch commit cache on REWIND and DELETE

      for (ReceiveCommand cmd : commands) {

         String ref = cmd.getRefName();

         if (ref.startsWith(Constants.R_HEADS)) {

            switch (cmd.getType()) {

            case UPDATE_NONFASTFORWARD:

            case DELETE:

               // reset branch commit cache on REWIND and DELETE

               CommitCache.instance().clear(repository.name, ref);

               break;

            default:

@@ -400,14 +400,14 @@
   }



   protected void sendHeader(String msg, Object... objects) {

      sendMessage("--->", msg, objects);

      sendInfo("--> ", msg, objects);

   }



   protected void sendMessage(String msg, Object... objects) {

      sendMessage("    ", msg, objects);

   protected void sendInfo(String msg, Object... objects) {

      sendInfo("    ", msg, objects);

   }



   protected void sendMessage(String prefix, String msg, Object... objects) {

   protected void sendInfo(String prefix, String msg, Object... objects) {

      String text;

      if (ArrayUtils.isEmpty(objects)) {

         text = msg;

@@ -416,7 +416,9 @@
         text = MessageFormat.format(msg, objects);

         super.sendMessage(prefix + text);

      }

      LOGGER.info(text + " (" + user.username + ")");

      if (!StringUtils.isEmpty(msg)) {

         LOGGER.info(text + " (" + user.username + ")");

      }

   }



   protected void sendError(String msg, Object... objects) {

@@ -428,7 +430,9 @@
         text = MessageFormat.format(msg, objects);

         super.sendError(text);

      }

      LOGGER.error(text + " (" + user.username + ")");

      if (!StringUtils.isEmpty(msg)) {

         LOGGER.error(text + " (" + user.username + ")");

      }

   }



   /**