Author: markt
Date: Tue May  3 11:02:55 2016
New Revision: 1742093

URL: http://svn.apache.org/viewvc?rev=1742093&view=rev
Log:
Skip web-fragment.xml scan in WEB-INF/classes when scanning an unpacked WAR.
Move the test earlier:
- one test covers packed and unpacked WARs
- testing earlier is (marginally) more efficient

Modified:
    
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FragmentJarScannerCallback.java
    tomcat/trunk/java/org/apache/tomcat/util/scan/StandardJarScanner.java

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FragmentJarScannerCallback.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FragmentJarScannerCallback.java?rev=1742093&r1=1742092&r2=1742093&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FragmentJarScannerCallback.java
 (original)
+++ 
tomcat/trunk/java/org/apache/tomcat/util/descriptor/web/FragmentJarScannerCallback.java
 Tue May  3 11:02:55 2016
@@ -111,11 +111,6 @@ public class FragmentJarScannerCallback
     @Override
     public void scan(File file, String webappPath, boolean isWebapp) throws 
IOException {
 
-        // Fragments unpacked in WEB-INF/classes are not handled
-        if ("/WEB-INF/classes".equals(webappPath)) {
-            return;
-        }
-
         WebXml fragment = new WebXml();
         fragment.setWebappJar(isWebapp);
         fragment.setDelegate(delegate);

Modified: tomcat/trunk/java/org/apache/tomcat/util/scan/StandardJarScanner.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/scan/StandardJarScanner.java?rev=1742093&r1=1742092&r2=1742093&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/scan/StandardJarScanner.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/scan/StandardJarScanner.java Tue 
May  3 11:02:55 2016
@@ -261,6 +261,11 @@ public class StandardJarScanner implemen
                             }
                             String webappPath = null;
                             if (urls[i].equals(webInfURL)) {
+                                if (scanType == JarScanType.PLUGGABILITY) {
+                                    // WEB-INF/classes should not be scanned 
for
+                                    // web fragments.
+                                    continue;
+                                }
                                 webappPath = Constants.WEB_INF_CLASSES;
                             }
                             try {



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to