From 30f9d25d77ccb5cd978d4cf8fa389ec819e90e95 Mon Sep 17 00:00:00 2001 From: Philip L. McMahon <philip.l.mcmahon@gmail.com> Date: Fri, 27 Jan 2012 02:02:19 -0500 Subject: [PATCH] Correct update of HEAD symbolic reference when target is a tag. --- src/com/gitblit/GitServlet.java | 72 +++++++++++++++++++++++++---------- 1 files changed, 51 insertions(+), 21 deletions(-) diff --git a/src/com/gitblit/GitServlet.java b/src/com/gitblit/GitServlet.java index 23fb32a..3b60e9f 100644 --- a/src/com/gitblit/GitServlet.java +++ b/src/com/gitblit/GitServlet.java @@ -26,10 +26,12 @@ import java.io.OutputStreamWriter; import java.text.MessageFormat; import java.util.Collection; +import java.util.Enumeration; import java.util.LinkedHashSet; import java.util.Set; import javax.servlet.ServletConfig; +import javax.servlet.ServletContext; import javax.servlet.ServletException; import javax.servlet.http.HttpServletRequest; @@ -71,22 +73,9 @@ private File groovyDir; - /** - * Configure the servlet from Gitblit's configuration. - */ - @Override - public String getInitParameter(String name) { - if (name.equals("base-path")) { - return GitBlit.getRepositoriesFolder().getAbsolutePath(); - } else if (name.equals("export-all")) { - return "1"; - } - return super.getInitParameter(name); - } - @Override public void init(ServletConfig config) throws ServletException { - groovyDir = GitBlit.getGroovyScriptsFolder(); + groovyDir = GitBlit.getGroovyScriptsFolder(); try { gse = new GroovyScriptEngine(groovyDir.getAbsolutePath()); } catch (IOException e) { @@ -106,7 +95,48 @@ return rp; } }); - super.init(config); + super.init(new GitblitServletConfig(config)); + } + + /** + * Transitional wrapper class to configure the JGit 1.2 GitFilter. This + * GitServlet will probably be replaced by a GitFilter so that Gitblit can + * serve Git repositories on the root URL and not a /git sub-url. + * + * @author James Moger + * + */ + private class GitblitServletConfig implements ServletConfig { + final ServletConfig config; + + GitblitServletConfig(ServletConfig config) { + this.config = config; + } + + @Override + public String getServletName() { + return config.getServletName(); + } + + @Override + public ServletContext getServletContext() { + return config.getServletContext(); + } + + @Override + public String getInitParameter(String name) { + if (name.equals("base-path")) { + return GitBlit.getRepositoriesFolder().getAbsolutePath(); + } else if (name.equals("export-all")) { + return "1"; + } + return config.getInitParameter(name); + } + + @Override + public Enumeration<String> getInitParameterNames() { + return config.getInitParameterNames(); + } } /** @@ -130,9 +160,9 @@ */ @Override public void onPreReceive(ReceivePack rp, Collection<ReceiveCommand> commands) { - Set<String> scripts = new LinkedHashSet<String>(); - scripts.addAll(GitBlit.getStrings(Keys.groovy.preReceiveScripts)); RepositoryModel repository = getRepositoryModel(rp); + Set<String> scripts = new LinkedHashSet<String>(); + scripts.addAll(GitBlit.self().getPreReceiveScriptsInherited(repository)); scripts.addAll(repository.preReceiveScripts); UserModel user = getUserModel(rp); runGroovy(repository, user, commands, scripts); @@ -158,9 +188,9 @@ logger.info("skipping post-receive hooks, no refs created, updated, or removed"); return; } - Set<String> scripts = new LinkedHashSet<String>(); - scripts.addAll(GitBlit.getStrings(Keys.groovy.postReceiveScripts)); RepositoryModel repository = getRepositoryModel(rp); + Set<String> scripts = new LinkedHashSet<String>(); + scripts.addAll(GitBlit.self().getPostReceiveScriptsInherited(repository)); scripts.addAll(repository.postReceiveScripts); UserModel user = getUserModel(rp); runGroovy(repository, user, commands, scripts); @@ -178,8 +208,8 @@ protected RepositoryModel getRepositoryModel(ReceivePack rp) { Repository repository = rp.getRepository(); String rootPath = GitBlit.getRepositoriesFolder().getAbsolutePath(); - String repositoryName = repository.getDirectory().getAbsolutePath(); - repositoryName = repositoryName.substring(rootPath.length() + 1); + String repositoryName = StringUtils.getRelativePath(rootPath, repository.getDirectory() + .getAbsolutePath()); RepositoryModel model = GitBlit.self().getRepositoryModel(repositoryName); return model; } -- Gitblit v1.9.1