]> source.dussan.org Git - gitblit.git/commitdiff
Remove /org/ filtering 48/248/1
authorJames Moger <james.moger@gitblit.com>
Fri, 22 May 2015 01:36:48 +0000 (21:36 -0400)
committerJames Moger <james.moger@gitblit.com>
Fri, 22 May 2015 01:36:48 +0000 (21:36 -0400)
src/main/java/com/gitblit/guice/WebModule.java
src/main/java/com/gitblit/servlet/GitblitContext.java

index 4a0cfcd6c5731c5a48e080ad6627028cdfd6044d..c6172c3dec5ba57217ef532af026b107b7e322af 100644 (file)
@@ -53,11 +53,6 @@ import com.google.inject.servlet.ServletModule;
 public class WebModule extends ServletModule {
 
        final static String ALL = "/*";
-       private boolean isGO;
-
-       public WebModule(boolean isGO) {
-               this.isGO=isGO;
-       }
 
        @Override
        protected void configureServlets() {
@@ -75,20 +70,18 @@ public class WebModule extends ServletModule {
                serve(Constants.PT_PATH).with(PtServlet.class);
                serve("/robots.txt").with(RobotsTxtServlet.class);
                serve("/logo.png").with(LogoServlet.class);
-               if(isGO)
-               {
-                       /* Prevent accidental access to 'resources' such as GitBlit java classes
-                        *
-                        * In the GO setup the JAR containing the application and the WAR injected
-                        * into Jetty are the same file. However Jetty expects to serve the entire WAR
-                        * contents, except the WEB-INF folder. Thus, all java binary classes in the
-                        * JAR are served by default as is they were legitimate resources.
-                        *
-                        * The below servlet mappings prevent that behavior
-                        */
-                       serve(fuzzy("/com/")).with(AccessDeniedServlet.class);
-                       serve(fuzzy("/org/")).with(AccessDeniedServlet.class);
-               }
+
+               /* Prevent accidental access to 'resources' such as GitBlit java classes
+                *
+                * In the GO setup the JAR containing the application and the WAR injected
+                * into Jetty are the same file. However Jetty expects to serve the entire WAR
+                * contents, except the WEB-INF folder. Thus, all java binary classes in the
+                * JAR are served by default as is they were legitimate resources.
+                *
+                * The below servlet mappings prevent that behavior
+                */
+               serve(fuzzy("/com/")).with(AccessDeniedServlet.class);
+
                // global filters
                filter(ALL).through(ProxyFilter.class);
                filter(ALL).through(EnforceAuthenticationFilter.class);
index d447dacafb4fad410e167bd0af31732e517d4fbc..077624c2e5db638e67892b421f8cb18546efc5a9 100644 (file)
@@ -129,7 +129,7 @@ public class GitblitContext extends GuiceServletContextListener {
         * Returns Gitblit's Guice injection modules.
         */
        protected AbstractModule [] getModules() {
-               return new AbstractModule [] { new CoreModule(), new WebModule(null!=goBaseFolder) };
+               return new AbstractModule [] { new CoreModule(), new WebModule() };
        }
 
        /**