From 86bea9e0016b2890db8ba83049dd4e89653a0a5e Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Fri, 16 Mar 2012 17:29:39 -0400
Subject: [PATCH] Ensure that the welcome message is interpreted as UTF-8 (issue 74)

---
 src/com/gitblit/FileSettings.java |   16 ++++++++++++++--
 1 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/com/gitblit/FileSettings.java b/src/com/gitblit/FileSettings.java
index 6110058..666bda0 100644
--- a/src/com/gitblit/FileSettings.java
+++ b/src/com/gitblit/FileSettings.java
@@ -20,7 +20,6 @@
 import java.io.FileNotFoundException;
 import java.util.Map;
 import java.util.Properties;
-import java.util.regex.Pattern;
 
 import com.gitblit.utils.FileUtils;
 
@@ -38,6 +37,8 @@
 	private final Properties properties = new Properties();
 
 	private volatile long lastModified;
+	
+	private volatile boolean forceReload;
 
 	public FileSettings(String file) {
 		super(FileSettings.class);
@@ -50,7 +51,7 @@
 	 */
 	@Override
 	protected synchronized Properties read() {
-		if (propertiesFile.exists() && (propertiesFile.lastModified() > lastModified)) {
+		if (propertiesFile.exists() && (forceReload || (propertiesFile.lastModified() > lastModified))) {
 			FileInputStream is = null;
 			try {
 				Properties props = new Properties();
@@ -61,6 +62,7 @@
 				properties.clear();
 				properties.putAll(props);
 				lastModified = propertiesFile.lastModified();
+				forceReload = false;
 			} catch (FileNotFoundException f) {
 				// IGNORE - won't happen because file.exists() check above
 			} catch (Throwable t) {
@@ -89,6 +91,9 @@
 			content = content.replaceAll(regex, setting.getKey() + " = " + setting.getValue());
 		}
 		FileUtils.writeContent(propertiesFile, content);
+		// manually set the forceReload flag because not all JVMs support real
+		// millisecond resolution of lastModified. (issue-55)		
+		forceReload = true;
 		return true;
 	}
 	
@@ -103,6 +108,13 @@
 		return lastModified;
 	}
 
+	/**
+	 * @return the state of the force reload flag
+	 */
+	protected boolean forceReload() {
+		return forceReload;
+	}
+
 	@Override
 	public String toString() {
 		return propertiesFile.getAbsolutePath();

--
Gitblit v1.9.1