From 4e84166db5c5538e3984d9d2d6bb1f9902e65ee0 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Tue, 04 Nov 2014 17:38:17 -0500
Subject: [PATCH] Merged #217 "Exclude SSLv3 from Gitblit GO https protocols"

---
 src/main/java/com/gitblit/wicket/GitblitWicketFilter.java |   62 ++++++++++++++++++++++++++----
 1 files changed, 53 insertions(+), 9 deletions(-)

diff --git a/src/main/java/com/gitblit/wicket/GitblitWicketFilter.java b/src/main/java/com/gitblit/wicket/GitblitWicketFilter.java
index 0f639c7..68ad84a 100644
--- a/src/main/java/com/gitblit/wicket/GitblitWicketFilter.java
+++ b/src/main/java/com/gitblit/wicket/GitblitWicketFilter.java
@@ -17,15 +17,22 @@
 
 import java.util.Date;
 
+import com.google.inject.Inject;
+import com.google.inject.Singleton;
 import javax.servlet.http.HttpServletRequest;
 
+import org.apache.wicket.protocol.http.IWebApplicationFactory;
+import org.apache.wicket.protocol.http.WebApplication;
 import org.apache.wicket.protocol.http.WicketFilter;
 import org.apache.wicket.util.string.Strings;
 import org.eclipse.jgit.lib.Repository;
 import org.eclipse.jgit.revwalk.RevCommit;
 
-import com.gitblit.GitBlit;
+import com.gitblit.IStoredSettings;
 import com.gitblit.Keys;
+import com.gitblit.manager.IProjectManager;
+import com.gitblit.manager.IRepositoryManager;
+import com.gitblit.manager.IRuntimeManager;
 import com.gitblit.models.ProjectModel;
 import com.gitblit.models.RepositoryModel;
 import com.gitblit.utils.JGitUtils;
@@ -39,7 +46,43 @@
  * @author James Moger
  *
  */
+@Singleton
 public class GitblitWicketFilter extends WicketFilter {
+
+	private IStoredSettings settings;
+
+	private IRuntimeManager runtimeManager;
+
+	private IRepositoryManager repositoryManager;
+
+	private IProjectManager projectManager;
+
+	private GitBlitWebApp webapp;
+
+	@Inject
+	public GitblitWicketFilter(
+			IStoredSettings settings,
+			IRuntimeManager runtimeManager,
+			IRepositoryManager repositoryManager,
+			IProjectManager projectManager,
+			GitBlitWebApp webapp) {
+
+		this.settings = settings;
+		this.runtimeManager = runtimeManager;
+		this.repositoryManager = repositoryManager;
+		this.projectManager = projectManager;
+		this.webapp = webapp;
+	}
+
+	@Override
+	protected IWebApplicationFactory getApplicationFactory() {
+		return new IWebApplicationFactory() {
+			@Override
+			public WebApplication createApplication(WicketFilter filter) {
+				return webapp;
+			}
+		};
+	}
 
 	/**
 	 * Determines the last-modified date of the requested resource.
@@ -50,8 +93,9 @@
 	@Override
 	protected long getLastModified(final HttpServletRequest servletRequest)	{
 		final String pathInfo = getRelativePath(servletRequest);
-		if (Strings.isEmpty(pathInfo))
+		if (Strings.isEmpty(pathInfo)) {
 			return -1;
+		}
 		long lastModified = super.getLastModified(servletRequest);
 		if (lastModified > -1) {
 			return lastModified;
@@ -77,18 +121,18 @@
 			commitId = servletRequest.getParameter("h");
 		}
 
-		repo = repo.replace("%2f", "/").replace("%2F", "/").replace(GitBlit.getChar(Keys.web.forwardSlashCharacter, '/'), '/');
+		repo = repo.replace("%2f", "/").replace("%2F", "/").replace(settings.getChar(Keys.web.forwardSlashCharacter, '/'), '/');
 
 		GitBlitWebApp app = (GitBlitWebApp) getWebApplication();
-		int expires = GitBlit.getInteger(Keys.web.pageCacheExpires, 0);
+		int expires = settings.getInteger(Keys.web.pageCacheExpires, 0);
 		if (!StringUtils.isEmpty(page) && app.isCacheablePage(page) && expires > 0) {
 			// page can be cached by the browser
 			CacheControl cacheControl = app.getCacheControl(page);
-			Date bootDate = GitBlit.getBootDate();
+			Date bootDate = runtimeManager.getBootDate();
 			switch (cacheControl.value()) {
 			case ACTIVITY:
 				// returns the last activity date of the server
-				Date activityDate = GitBlit.getLastActivityDate();
+				Date activityDate = repositoryManager.getLastActivityDate();
 				if (activityDate != null) {
 					return activityDate.after(bootDate) ? activityDate.getTime() : bootDate.getTime();
 				}
@@ -98,7 +142,7 @@
 				return bootDate.getTime();
 			case PROJECT:
 				// return the latest change date for the project OR the boot date
-				ProjectModel project = GitBlit.self().getProjectModel(StringUtils.getRootPath(repo));
+				ProjectModel project = projectManager.getProjectModel(StringUtils.getRootPath(repo));
 				if (project != null) {
 					return project.lastChange.after(bootDate) ? project.lastChange.getTime() : bootDate.getTime();
 				}
@@ -106,7 +150,7 @@
 			case REPOSITORY:
 				// return the lastest change date for the repository OR the boot
 				// date, whichever is latest
-				RepositoryModel repository = GitBlit.self().getRepositoryModel(repo);
+				RepositoryModel repository = repositoryManager.getRepositoryModel(repo);
 				if (repository != null && repository.lastChange != null) {
 					return repository.lastChange.after(bootDate) ? repository.lastChange.getTime() : bootDate.getTime();
 				}
@@ -121,7 +165,7 @@
 					Repository r = null;
 					try {
 						// return the timestamp of the associated commit
-						r = GitBlit.self().getRepository(repo);
+						r = repositoryManager.getRepository(repo);
 						if (r != null) {
 							RevCommit commit = JGitUtils.getCommit(r, commitId);
 							if (commit != null) {

--
Gitblit v1.9.1