Use standard ServletRequestWrapper instead of custom wrapper (issue 224)

James Moger

2013-03-28 1f596c9875687561548ac893d4f105da49d700b5

 src/main/java/com/gitblit/GitBlit.java

@@ -613,7 +613,7 @@
         X509Metadata metadata = HttpUtils.getCertificateMetadata(httpRequest);

         if (user != null) {

            flagWicketSession(AuthenticationType.CERTIFICATE);

            logger.info(MessageFormat.format("{0} authenticated by client certificate {1} from {2}",

            logger.debug(MessageFormat.format("{0} authenticated by client certificate {1} from {2}",

                  user.username, metadata.serialNumber, httpRequest.getRemoteAddr()));

            return user;

         } else {

@@ -633,7 +633,7 @@
         UserModel user = getUserModel(principal.getName());

         if (user != null) {

            flagWicketSession(AuthenticationType.CONTAINER);

            logger.info(MessageFormat.format("{0} authenticated by servlet container principal from {1}",

            logger.debug(MessageFormat.format("{0} authenticated by servlet container principal from {1}",

                  user.username, httpRequest.getRemoteAddr()));

            return user;

         } else {

@@ -647,7 +647,7 @@
         UserModel user = authenticate(httpRequest.getCookies());

         if (user != null) {

            flagWicketSession(AuthenticationType.COOKIE);

            logger.info(MessageFormat.format("{0} authenticated by cookie from {1}",

            logger.debug(MessageFormat.format("{0} authenticated by cookie from {1}",

                  user.username, httpRequest.getRemoteAddr()));

            return user;

         }

@@ -669,7 +669,7 @@
            UserModel user = authenticate(username, password);

            if (user != null) {

               flagWicketSession(AuthenticationType.CREDENTIALS);

               logger.info(MessageFormat.format("{0} authenticated by BASIC request header from {1}",

               logger.debug(MessageFormat.format("{0} authenticated by BASIC request header from {1}",

                     user.username, httpRequest.getRemoteAddr()));

               return user;

            } else {

@@ -1827,6 +1827,9 @@
      } else {

         // not caching

         ProjectModel project = getProjectModel(userProject);

         if (project == null) {

            return null;

         }

         for (String repository : project.repositories) {

            if (repository.startsWith(userProject)) {

               RepositoryModel model = getRepositoryModel(repository);

@@ -2900,6 +2903,10 @@
    * @param toAddresses

    */

   public void sendMail(String subject, String message, String... toAddresses) {

      if (toAddresses == null || toAddresses.length == 0) {

         logger.debug(MessageFormat.format("Dropping message {0} because there are no recipients", subject));

         return;

      }

      try {

         Message mail = mailExecutor.createMessage(toAddresses);

         if (mail != null) {

@@ -2931,6 +2938,10 @@
    * @param toAddresses

    */

   public void sendHtmlMail(String subject, String message, String... toAddresses) {

      if (toAddresses == null || toAddresses.length == 0) {

         logger.debug(MessageFormat.format("Dropping message {0} because there are no recipients", subject));

         return;

      }

      try {

         Message mail = mailExecutor.createMessage(toAddresses);

         if (mail != null) {