From 87cc1ed60735a419a3ea23f20614fc0a3f9bab60 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Fri, 15 Apr 2011 17:18:51 -0400
Subject: [PATCH] Settings overhaul. Fixes to authentication.  Bind interface feature.

---
 src/com/gitblit/wicket/GitBlitWebApp.java |   17 ++++++++---------
 1 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/com/gitblit/wicket/GitBlitWebApp.java b/src/com/gitblit/wicket/GitBlitWebApp.java
index 8589b75..b70c95f 100644
--- a/src/com/gitblit/wicket/GitBlitWebApp.java
+++ b/src/com/gitblit/wicket/GitBlitWebApp.java
@@ -12,7 +12,6 @@
 
 import com.gitblit.GitBlit;
 import com.gitblit.Keys;
-import com.gitblit.StoredSettings;
 import com.gitblit.wicket.pages.BlobDiffPage;
 import com.gitblit.wicket.pages.BlobPage;
 import com.gitblit.wicket.pages.BranchesPage;
@@ -36,14 +35,14 @@
 		super.init();
 
 		// Setup page authorization mechanism
-		if (StoredSettings.getBoolean(Keys.web_authenticate, false)) {
+		if (GitBlit.self().settings().getBoolean(Keys.web.authenticate, false)) {
 			AuthorizationStrategy authStrategy = new AuthorizationStrategy();
 			getSecuritySettings().setAuthorizationStrategy(authStrategy);
 			getSecuritySettings().setUnauthorizedComponentInstantiationListener(authStrategy);
 		}
 
 		// Grab Browser info (like timezone, etc)
-		if (StoredSettings.getBoolean(Keys.web_useClientTimezone, false)) {
+		if (GitBlit.self().settings().getBoolean(Keys.web.useClientTimezone, false)) {
 			getRequestCycleSettings().setGatherExtendedBrowserInfo(true);
 		}
 
@@ -61,11 +60,15 @@
 		mount(new MixedParamUrlCodingStrategy("/commitdiff", CommitDiffPage.class, new String[] { "r", "h" }));
 		mount(new MixedParamUrlCodingStrategy("/patch", PatchPage.class, new String[] { "r", "h", "f" }));
 
-		// setup extended urls
+		// setup ticgit urls
 		mount(new MixedParamUrlCodingStrategy("/ticgit", TicGitPage.class, new String[] { "r" }));
 		mount(new MixedParamUrlCodingStrategy("/ticgittkt", TicGitTicketPage.class, new String[] { "r", "h", "f" }));
 
-		mount(new MixedParamUrlCodingStrategy("/login", LoginPage.class, new String[] {}));
+		// setup login/logout urls, if we are using authentication
+		if (GitBlit.self().settings().getBoolean(Keys.web.authenticate, true)) {
+			mount(new MixedParamUrlCodingStrategy("/login", LoginPage.class, new String[] {}));
+			mount(new MixedParamUrlCodingStrategy("/logout", LogoutPage.class, new String[] {}));
+		}
 	}
 
 	@Override
@@ -88,10 +91,6 @@
 		if (GitBlit.self().isDebugMode())
 			return Application.DEVELOPMENT;
 		return Application.DEPLOYMENT;
-	}
-
-	public String getCloneUrl(String repositoryName) {
-		return StoredSettings.getString(Keys.git_cloneUrl, "https://localhost/git/") + repositoryName;
 	}
 
 	public static GitBlitWebApp get() {

--
Gitblit v1.9.1