On 26/03/2015 13:27, Pravallika Peddi wrote:

<snip/>

> 1)     As mentioned in bug, in Tomcat7.0 catalina.properties file first 4
> default jars(except jdom.jar) are missing:
>   commons-discovery-*.jar,commons-el-*.jar,commons-net-*.jar
> 
>          We can add them under
> “tomcat.util.scan.DefaultJarScanner.jarsToSkip”

jdom.jar isn't covered by the existing filters.

<snip/>

> 3)       3)      So I am planning to provide a fix as below for Tomcat 7.0
> TLD scan:

Looks OK so far.

<snip/>

> But in Tomcat 7.0, StandardJarScanner.scan  method only called from both
> TldConfig.java and ContextConfig.java. So I am afraid I cannot add the
> Summary message in Tomcat7.0. When User enables FINE  level then only they
> can see the noTldInJar messages

Look again. It certainly is possible to add the summary message for TLDs
in Tomcat 7.

> 4) Tomcat7.0 ContextConfig scan is taking less time, hence I feel we can
> ignore it for now.

I disagree. The pluggability scanning can add a significant start-up
delay to web applications. The reason that the bug report was opened in
the first place is the time taken for the pluggability scans.

> Please let me know your opinion.

I think you have some more research to do.

Mark


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

Reply via email to