Skip re-authentication if we have a valid session

James Moger

2012-11-29 e5c7795dc9185272365ff340698c7d2f1e6f11ab

 src/com/gitblit/GitBlit.java

@@ -108,6 +108,7 @@
import com.gitblit.utils.ObjectCache;

import com.gitblit.utils.StringUtils;

import com.gitblit.utils.TimeUtils;

import com.gitblit.utils.X509Utils.X509Metadata;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.WicketUtils;



@@ -571,12 +572,15 @@
      UserModel model = HttpUtils.getUserModelFromCertificate(httpRequest, checkValidity, oids);

      if (model != null) {

         // grab real user model and preserve certificate serial number

         GitBlitWebSession session = GitBlitWebSession.get();

         session.authenticationType = AuthenticationType.CERTIFICATE;

         UserModel user = getUserModel(model.username);

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

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

         return user;

         if (user != null) {

            GitBlitWebSession session = GitBlitWebSession.get();

            session.authenticationType = AuthenticationType.CERTIFICATE;

            X509Metadata metadata = HttpUtils.getCertificateMetadata(httpRequest);

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

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

            return user;

         }

      }

      

      // try to authenticate by cookie

@@ -587,6 +591,8 @@
         if (user != null) {

            GitBlitWebSession session = GitBlitWebSession.get();

            session.authenticationType = AuthenticationType.COOKIE;

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

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

            return user;

         }

      }