From 1d78b8b372f15d89f10fd32cb0227a6a7966de3c Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 17 Apr 2014 23:08:07 -0400
Subject: [PATCH] [findbugs] Clarify class used for resource loading

---
 src/main/java/WEB-INF/web.xml |  117 ++++++++++++++++++++++++++++++++++++++++++++--------------
 1 files changed, 89 insertions(+), 28 deletions(-)

diff --git a/src/main/java/WEB-INF/web.xml b/src/main/java/WEB-INF/web.xml
index 5ee5ed2..77456d4 100644
--- a/src/main/java/WEB-INF/web.xml
+++ b/src/main/java/WEB-INF/web.xml
@@ -22,10 +22,12 @@
 		review the defaults for any new settings.  Settings are always versioned
 		with a SINCE x.y.z attribute and also noted in the release changelog.
 		-->
-	<context-param>
-		<param-name>baseFolder</param-name>
-		<param-value>${contextFolder}/WEB-INF/data</param-value>
-	</context-param>
+	<env-entry>
+		<description>The base folder is used to specify the root location of your Gitblit data.</description>
+		<env-entry-name>baseFolder</env-entry-name>
+		<env-entry-type>java.lang.String</env-entry-type>
+		<env-entry-value>${contextFolder}/WEB-INF/data</env-entry-value>
+	</env-entry>
 	
 	<!-- Gitblit Displayname -->
 	<display-name>
@@ -34,9 +36,10 @@
 
 	<!-- PARAMS --> 
 	 
-	<!-- Gitblit Context Listener --><!-- STRIP	 
+	 
+<!-- Gitblit Context Listener --><!-- STRIP	 
 	<listener>
- 		<listener-class>com.gitblit.GitBlit</listener-class>
+ 		<listener-class>com.gitblit.servlet.GitblitContext</listener-class>
  	</listener>STRIP --> 	
 	
 	
@@ -47,13 +50,31 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>GitServlet</servlet-name>
-		<servlet-class>com.gitblit.git.GitServlet</servlet-class>
+		<servlet-class>com.gitblit.servlet.GitServlet</servlet-class>
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>GitServlet</servlet-name>		
 		<url-pattern>/git/*</url-pattern>
 	</servlet-mapping>
+	<servlet-mapping>
+		<servlet-name>GitServlet</servlet-name>		
+		<url-pattern>/r/*</url-pattern>
+	</servlet-mapping>
+
 	
+	<!-- SparkleShare Invite Servlet
+		 <url-pattern> MUST match: 
+			* com.gitblit.Constants.SPARKLESHARE_INVITE_PATH
+			* Wicket Filter ignorePaths parameter -->
+	<servlet>
+		<servlet-name>SparkleShareInviteServlet</servlet-name>
+		<servlet-class>com.gitblit.servlet.SparkleShareInviteServlet</servlet-class>
+	</servlet>
+	<servlet-mapping>
+		<servlet-name>SparkleShareInviteServlet</servlet-name>		
+		<url-pattern>/sparkleshare/*</url-pattern>
+	</servlet-mapping>
+
 	
 	<!-- Syndication Servlet
 		 <url-pattern> MUST match: 
@@ -62,7 +83,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>SyndicationServlet</servlet-name>
-		<servlet-class>com.gitblit.SyndicationServlet</servlet-class>		
+		<servlet-class>com.gitblit.servlet.SyndicationServlet</servlet-class>		
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>SyndicationServlet</servlet-name>
@@ -77,7 +98,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>ZipServlet</servlet-name>
-		<servlet-class>com.gitblit.DownloadZipServlet</servlet-class>		
+		<servlet-class>com.gitblit.servlet.DownloadZipServlet</servlet-class>		
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>ZipServlet</servlet-name>
@@ -91,7 +112,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>FederationServlet</servlet-name>
-		<servlet-class>com.gitblit.FederationServlet</servlet-class>		
+		<servlet-class>com.gitblit.servlet.FederationServlet</servlet-class>		
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>FederationServlet</servlet-name>
@@ -105,7 +126,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>RpcServlet</servlet-name>
-		<servlet-class>com.gitblit.RpcServlet</servlet-class>		
+		<servlet-class>com.gitblit.servlet.RpcServlet</servlet-class>		
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>RpcServlet</servlet-name>
@@ -120,27 +141,64 @@
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>PagesServlet</servlet-name>
-		<servlet-class>com.gitblit.PagesServlet</servlet-class>
+		<servlet-class>com.gitblit.servlet.PagesServlet</servlet-class>
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>PagesServlet</servlet-name>		
 		<url-pattern>/pages/*</url-pattern>
 	</servlet-mapping>	
+
 	
+	<!-- Logo Servlet
+		 <url-pattern> MUST match: 
+			* Wicket Filter ignorePaths parameter -->
+	<servlet>
+		<servlet-name>LogoServlet</servlet-name>
+		<servlet-class>com.gitblit.servlet.LogoServlet</servlet-class>
+	</servlet>
+	<servlet-mapping>
+		<servlet-name>LogoServlet</servlet-name>		
+		<url-pattern>/logo.png</url-pattern>
+	</servlet-mapping>
+
+
+	<!-- PT Servlet
+		 <url-pattern> MUST match: 
+			* Wicket Filter ignorePaths parameter -->
+	<servlet>
+		<servlet-name>PtServlet</servlet-name>
+		<servlet-class>com.gitblit.servlet.PtServlet</servlet-class>
+	</servlet>
+	<servlet-mapping>
+		<servlet-name>PtServlet</servlet-name>		
+		<url-pattern>/pt</url-pattern>
+	</servlet-mapping>
+
+
+	<!-- Branch Graph Servlet
+		 <url-pattern> MUST match: 
+			* Wicket Filter ignorePaths parameter -->
+	<servlet>
+		<servlet-name>BranchGraphServlet</servlet-name>
+		<servlet-class>com.gitblit.servlet.BranchGraphServlet</servlet-class>
+	</servlet>
+	<servlet-mapping>
+		<servlet-name>BranchGraphServlet</servlet-name>		
+		<url-pattern>/graph/*</url-pattern>
+	</servlet-mapping>
 
 	<!-- Robots.txt Servlet
 		 <url-pattern> MUST match: 
 			* Wicket Filter ignorePaths parameter -->
 	<servlet>
 		<servlet-name>RobotsTxtServlet</servlet-name>
-		<servlet-class>com.gitblit.RobotsTxtServlet</servlet-class>
+		<servlet-class>com.gitblit.servlet.RobotsTxtServlet</servlet-class>
 	</servlet>
 	<servlet-mapping>
 		<servlet-name>RobotsTxtServlet</servlet-name>		
 		<url-pattern>/robots.txt</url-pattern>
 	</servlet-mapping>
 
-	
 	<!-- Git Access Restriction Filter
 		 <url-pattern> MUST match: 
 			* GitServlet
@@ -148,11 +206,15 @@
 			* Wicket Filter ignorePaths parameter -->
 	<filter>
 		<filter-name>GitFilter</filter-name>
-		<filter-class>com.gitblit.GitFilter</filter-class>
+		<filter-class>com.gitblit.servlet.GitFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>GitFilter</filter-name>
 		<url-pattern>/git/*</url-pattern>
+	</filter-mapping>
+	<filter-mapping>
+		<filter-name>GitFilter</filter-name>
+		<url-pattern>/r/*</url-pattern>
 	</filter-mapping>
 	
 	
@@ -163,7 +225,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<filter>
 		<filter-name>SyndicationFilter</filter-name>
-		<filter-class>com.gitblit.SyndicationFilter</filter-class>
+		<filter-class>com.gitblit.servlet.SyndicationFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>SyndicationFilter</filter-name>
@@ -178,7 +240,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<filter>
 		<filter-name>ZipFilter</filter-name>
-		<filter-class>com.gitblit.DownloadZipFilter</filter-class>
+		<filter-class>com.gitblit.servlet.DownloadZipFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>ZipFilter</filter-name>
@@ -193,7 +255,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<filter>
 		<filter-name>RpcFilter</filter-name>
-		<filter-class>com.gitblit.RpcFilter</filter-class>
+		<filter-class>com.gitblit.servlet.RpcFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>RpcFilter</filter-name>
@@ -208,7 +270,7 @@
 			* Wicket Filter ignorePaths parameter -->
 	<filter>
 		<filter-name>PagesFilter</filter-name>
-		<filter-class>com.gitblit.PagesFilter</filter-class>
+		<filter-class>com.gitblit.servlet.PagesFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>PagesFilter</filter-name>
@@ -217,7 +279,7 @@
 	
 	<filter>
 		<filter-name>EnforceAuthenticationFilter</filter-name>
-		<filter-class>com.gitblit.EnforceAuthenticationFilter</filter-class>
+		<filter-class>com.gitblit.servlet.EnforceAuthenticationFilter</filter-class>
 	</filter>
 	<filter-mapping>
         <filter-name>EnforceAuthenticationFilter</filter-name>
@@ -229,12 +291,8 @@
     <filter>
         <filter-name>wicketFilter</filter-name>
         <filter-class>
-            org.apache.wicket.protocol.http.WicketFilter
-        </filter-class>
-        <init-param>
-            <param-name>applicationClassName</param-name>
-            <param-value>com.gitblit.wicket.GitBlitWebApp</param-value>
-        </init-param>
+            com.gitblit.wicket.GitblitWicketFilter
+        </filter-class>        
         <init-param>
             <param-name>ignorePaths</param-name>
             <!-- Paths should match 
@@ -244,6 +302,8 @@
              	* GitFilter <url-pattern>
              	* GitServlet <url-pattern>
              	* com.gitblit.Constants.GIT_PATH
+             	* SparkleshareInviteServlet <url-pattern>
+             	* com.gitblit.Constants.SPARKLESHARE_INVITE_PATH
              	* Zipfilter <url-pattern>
              	* ZipServlet <url-pattern>
              	* com.gitblit.Constants.ZIP_PATH
@@ -253,11 +313,12 @@
              	* PagesFilter <url-pattern>
              	* PagesServlet <url-pattern>
              	* com.gitblit.Constants.PAGES_PATH -->
-            <param-value>git/,feed/,zip/,federation/,rpc/,pages/,robots.txt</param-value>
+            <param-value>r/,git/,pt,feed/,zip/,federation/,rpc/,pages/,robots.txt,logo.png,graph/,sparkleshare/</param-value>
         </init-param>
     </filter>
     <filter-mapping>
         <filter-name>wicketFilter</filter-name>
         <url-pattern>/*</url-pattern>
     </filter-mapping>
-</web-app>
\ No newline at end of file
+    
+</web-app>

--
Gitblit v1.9.1