summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJames Moger <james.moger@gitblit.com>2014-02-28 08:22:44 -0600
committerJames Moger <james.moger@gitblit.com>2014-02-28 08:22:44 -0600
commit3d096fee8d8f7dae9ba4801480f775285227d9f6 (patch)
treeac876213c39c830a7d3ecff90dc9dc4e7ba8eca1
parentf6a8cf5a4c0f573790cd5b23a38e69f26df90f3f (diff)
parent2f3c342b5751f0ea41b0711b6d8757471572a79c (diff)
downloadgitblit-3d096fee8d8f7dae9ba4801480f775285227d9f6.tar.gz
gitblit-3d096fee8d8f7dae9ba4801480f775285227d9f6.zip
Merged ticket #14 patchset 1
-rw-r--r--src/main/java/com/gitblit/servlet/GitblitContext.java31
1 files changed, 20 insertions, 11 deletions
diff --git a/src/main/java/com/gitblit/servlet/GitblitContext.java b/src/main/java/com/gitblit/servlet/GitblitContext.java
index 682b590c..d4ec9671 100644
--- a/src/main/java/com/gitblit/servlet/GitblitContext.java
+++ b/src/main/java/com/gitblit/servlet/GitblitContext.java
@@ -149,7 +149,11 @@ public class GitblitContext extends DaggerContext {
String contextRealPath = context.getRealPath("/");
File contextFolder = (contextRealPath != null) ? new File(contextRealPath) : null;
- if (!StringUtils.isEmpty(System.getenv("OPENSHIFT_DATA_DIR"))) {
+ // if the base folder dosen't match the default assume they don't want to use express,
+ // this allows for other containers to customise the basefolder per context.
+ String defaultBase = Constants.contextFolder$ + "/WEB-INF/data";
+ String base = lookupBaseFolderFromJndi();
+ if (!StringUtils.isEmpty(System.getenv("OPENSHIFT_DATA_DIR")) && defaultBase.equals(base)) {
// RedHat OpenShift
baseFolder = configureExpress(context, webxmlSettings, contextFolder, runtimeSettings);
} else {
@@ -184,6 +188,18 @@ public class GitblitContext extends DaggerContext {
logger.info("");
}
+ private String lookupBaseFolderFromJndi() {
+ try {
+ // try to lookup JNDI env-entry for the baseFolder
+ InitialContext ic = new InitialContext();
+ Context env = (Context) ic.lookup("java:comp/env");
+ return (String) env.lookup("baseFolder");
+ } catch (NamingException n) {
+ logger.error("Failed to get JNDI env-entry: " + n.getExplanation());
+ }
+ return null;
+ }
+
protected <X extends IManager> X startManager(ObjectGraph injector, Class<X> clazz) {
logManager(clazz);
X x = injector.get(clazz);
@@ -268,16 +284,9 @@ public class GitblitContext extends DaggerContext {
logger.error("");
}
- try {
- // try to lookup JNDI env-entry for the baseFolder
- InitialContext ic = new InitialContext();
- Context env = (Context) ic.lookup("java:comp/env");
- String val = (String) env.lookup("baseFolder");
- if (!StringUtils.isEmpty(val)) {
- path = val;
- }
- } catch (NamingException n) {
- logger.error("Failed to get JNDI env-entry: " + n.getExplanation());
+ String baseFromJndi = lookupBaseFolderFromJndi();
+ if (!StringUtils.isEmpty(baseFromJndi)) {
+ path = baseFromJndi;
}
File base = com.gitblit.utils.FileUtils.resolveParameter(Constants.contextFolder$, contextFolder, path);