Do not traverse unaccessible subdirectories (issue 51)

lemval

2012-01-31 1c30dad2115fc513791d8a5b292ad0f7d7b85749

 src/com/gitblit/wicket/GitBlitWebApp.java

@@ -22,26 +22,28 @@
import org.apache.wicket.Session;

import org.apache.wicket.markup.html.WebPage;

import org.apache.wicket.protocol.http.WebApplication;

import org.apache.wicket.request.target.coding.MixedParamUrlCodingStrategy;



import com.gitblit.GitBlit;

import com.gitblit.Keys;

import com.gitblit.wicket.pages.ActivityPage;

import com.gitblit.wicket.pages.BlamePage;

import com.gitblit.wicket.pages.BlobDiffPage;

import com.gitblit.wicket.pages.BlobPage;

import com.gitblit.wicket.pages.BranchesPage;

import com.gitblit.wicket.pages.CommitDiffPage;

import com.gitblit.wicket.pages.CommitPage;

import com.gitblit.wicket.pages.DocsPage;

import com.gitblit.wicket.pages.FederationRegistrationPage;

import com.gitblit.wicket.pages.GravatarProfilePage;

import com.gitblit.wicket.pages.HistoryPage;

import com.gitblit.wicket.pages.LogPage;

import com.gitblit.wicket.pages.LoginPage;

import com.gitblit.wicket.pages.LogoutPage;

import com.gitblit.wicket.pages.MarkdownPage;

import com.gitblit.wicket.pages.MetricsPage;

import com.gitblit.wicket.pages.PatchPage;

import com.gitblit.wicket.pages.RawPage;

import com.gitblit.wicket.pages.RepositoriesPage;

import com.gitblit.wicket.pages.ReviewProposalPage;

import com.gitblit.wicket.pages.SearchPage;

import com.gitblit.wicket.pages.StatsPage;

import com.gitblit.wicket.pages.SummaryPage;

import com.gitblit.wicket.pages.TagPage;

import com.gitblit.wicket.pages.TagsPage;

@@ -69,6 +71,11 @@
         getRequestCycleSettings().setGatherExtendedBrowserInfo(true);

      }



      // configure the resource cache duration to 90 days for deployment

      if (!GitBlit.isDebugMode()) {

         getResourceSettings().setDefaultCacheDuration(90 * 86400);

      }



      // setup the standard gitweb-ish urls

      mount("/summary", SummaryPage.class, "r");

      mount("/log", LogPage.class, "r", "h");

@@ -84,8 +91,9 @@
      mount("/patch", PatchPage.class, "r", "h", "f");

      mount("/history", HistoryPage.class, "r", "h", "f");

      mount("/search", SearchPage.class);

      mount("/stats", StatsPage.class, "r");

		
      mount("/metrics", MetricsPage.class, "r");

      mount("/blame", BlamePage.class, "r", "h", "f");



      // setup ticket urls

      mount("/tickets", TicketsPage.class, "r");

      mount("/ticket", TicketPage.class, "r", "h", "f");

@@ -94,18 +102,22 @@
      mount("/docs", DocsPage.class, "r");

      mount("/markdown", MarkdownPage.class, "r", "h", "f");



      // setup login/logout urls, if we are using authentication

      if (useAuthentication) {

         mount("/login", LoginPage.class);

         mount("/logout", LogoutPage.class);

      }

      // federation urls

      mount("/proposal", ReviewProposalPage.class, "t");

      mount("/registration", FederationRegistrationPage.class, "u", "n");



      mount("/activity", ActivityPage.class, "r", "h");

      mount("/gravatar", GravatarProfilePage.class, "h");

   }



   private void mount(String location, Class<? extends WebPage> clazz, String... parameters) {

      if (parameters == null) {

         parameters = new String[] {};

      }

      mount(new MixedParamUrlCodingStrategy(location, clazz, parameters));

      if (!GitBlit.getBoolean(Keys.web.mountParameters, true)) {

         parameters = new String[] {};

      }

      mount(new GitblitParamUrlCodingStrategy(location, clazz, parameters));

   }



   @Override

@@ -120,7 +132,7 @@


   @Override

   public final String getConfigurationType() {

      if (GitBlit.self().isDebugMode()) {

      if (GitBlit.isDebugMode()) {

         return Application.DEVELOPMENT;

      }

      return Application.DEPLOYMENT;