From db09ef1033ce4942ac1eb3cb684f232bf7e5e231 Mon Sep 17 00:00:00 2001 From: Dariusz Bywalec <dariusz.bywalec@trapezegroup.pl> Date: Mon, 04 Jan 2016 05:41:12 -0500 Subject: [PATCH] Lower log level of servlet authenticate when request is already authenticated --- src/main/java/com/gitblit/servlet/RawServlet.java | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/src/main/java/com/gitblit/servlet/RawServlet.java b/src/main/java/com/gitblit/servlet/RawServlet.java index 81793bc..897047d 100644 --- a/src/main/java/com/gitblit/servlet/RawServlet.java +++ b/src/main/java/com/gitblit/servlet/RawServlet.java @@ -233,7 +233,14 @@ try { String ext = StringUtils.getFileExtension(file).toLowerCase(); - String contentType = file.charAt(0) == '.' ? "text/plain" : quickContentTypes.get(ext); + // We can't parse out an extension for classic "dotfiles", so make a general assumption that + // they're text files to allow presenting them in browser instead of only for download. + // + // However, that only holds for files with no other extension included, for files that happen + // to start with a dot but also include an extension, process the extension normally. + // This logic covers .gitattributes, .gitignore, .zshrc, etc., but does not cover .mongorc.js, .zshrc.bak + boolean isExtensionlessDotfile = file.charAt(0) == '.' && (file.length() == 1 || file.indexOf('.', 1) < 0); + String contentType = isExtensionlessDotfile ? "text/plain" : quickContentTypes.get(ext); if (contentType == null) { List<String> exts = runtimeManager.getSettings().getStrings(Keys.web.prettyPrintExtensions); -- Gitblit v1.9.1