Re: Maven snapshot builds

2016-04-01 Thread Martin Grigorov
On Fri, Apr 1, 2016 at 10:07 PM, Martin Grigorov 
wrote:

> Hi,
>
> Does anyone know what is the reason for the missing -SNAPSHOT builds at
> https://repository.apache.org/content/repositories/snapshots/org/apache/tomcat/tomcat-catalina/
> (RAO) ?
>
> https://tomcat.apache.org/tomcat-8.5-doc/maven-jars.html says that the
> snapshots should be at
> http://people.apache.org/repo/m2-snapshot-repository/org/apache/tomcat/
> but this gives 404.
>
> The last 8.0-SNAPSHOT at RAO is from Feb 23th.
> I actually need 8.5-SNAPSHOT with the fix for ClassCastException when
> trying to get the ApplicationPushBuilder.
>
> Another option for me is to build and install the Maven jars locally. Any
> hints for the Ant commands to do this ?
>

I didn't find a way to install them just locally so I've uploaded
8.5-SNAPSHOT at
https://repository.apache.org/content/repositories/snapshots/org/apache/tomcat/tomcat-catalina/8.5-SNAPSHOT/
by following the steps at res/maven/README.txt.


>
> Thank you!
>
> Martin Grigorov
> Wicket Training and Consulting
> https://twitter.com/mtgrigorov
>


Maven snapshot builds

2016-04-01 Thread Martin Grigorov
Hi,

Does anyone know what is the reason for the missing -SNAPSHOT builds at
https://repository.apache.org/content/repositories/snapshots/org/apache/tomcat/tomcat-catalina/
(RAO) ?

https://tomcat.apache.org/tomcat-8.5-doc/maven-jars.html says that the
snapshots should be at
http://people.apache.org/repo/m2-snapshot-repository/org/apache/tomcat/ but
this gives 404.

The last 8.0-SNAPSHOT at RAO is from Feb 23th.
I actually need 8.5-SNAPSHOT with the fix for ClassCastException when
trying to get the ApplicationPushBuilder.

Another option for me is to build and install the Maven jars locally. Any
hints for the Ant commands to do this ?

Thank you!

Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov


svn commit: r1737428 - /tomcat/site/trunk/docs/tomcat-8.5-doc/building.html

2016-04-01 Thread mgrigorov
Author: mgrigorov
Date: Fri Apr  1 20:01:53 2016
New Revision: 1737428

URL: http://svn.apache.org/viewvc?rev=1737428&view=rev
Log:
s/8.0/8.5/ in links text


Modified:
tomcat/site/trunk/docs/tomcat-8.5-doc/building.html

Modified: tomcat/site/trunk/docs/tomcat-8.5-doc/building.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/tomcat-8.5-doc/building.html?rev=1737428&r1=1737427&r2=1737428&view=diff
==
--- tomcat/site/trunk/docs/tomcat-8.5-doc/building.html (original)
+++ tomcat/site/trunk/docs/tomcat-8.5-doc/building.html Fri Apr  1 20:01:53 2016
@@ -82,11 +82,11 @@ available, which will be used to actuall
 
   
   Tomcat SVN repository URL:
-  http://svn.apache.org/repos/asf/tomcat/tc8.5.x/trunk/";>http://svn.apache.org/repos/asf/tomcat/tc8.0.x/trunk/
+  http://svn.apache.org/repos/asf/tomcat/tc8.5.x/trunk/";>http://svn.apache.org/repos/asf/tomcat/tc8.5.x/trunk/
   
   
   Tomcat source packages:
-  http://tomcat.apache.org/download-85.cgi";>http://tomcat.apache.org/download-80.cgi.
+  http://tomcat.apache.org/download-85.cgi";>http://tomcat.apache.org/download-85.cgi.
   
 
   



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



Re: Project Jigsaw: The module system was integrated into JDK 9 and is now available for testing in early-access, build 111.

2016-04-01 Thread Mark Thomas
I ran the Tomcat 9 unit tests and there are a whole bunch of failures
(everything passed with the previous ea release). It looks like some of
our memory leak prevention tricks no longer work. That probably means we
either need to find a way around the new Java 9 restrictions or we need
to raise bugs against the JRE to get them fixed once and for all.

This is likely to be the next block of work that I look at.

Mark

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



[Bug 56126] 8.0.1 Regression with Spring with AspectJ Load-Time Weaving

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56126

--- Comment #7 from mda...@emergencycallworx.com  
---
We are using spring-instrumentation on load using
spring-instrument-tomcat-4.2.4

Based on what I'm reading, TomcatInstrumentableClassLoader is no longer
required in context.xml.

However up until 8.033, the following worked in context.xml: 


TomcatInstrumentableClassLoader does not work as of 8.033.

This is more of a note that something has changed between 8.032 and 8.033 that
impacts spring load time weaving.

For our application, TomcatInstrumentableClassLoader is required to work
correctly in 8.032, TomcatInstrumentableClassLoader needs to be pulled to work
correctly in 8.033.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59253] IOExceptions in websockets

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59253

Remy Maucherat  changed:

   What|Removed |Added

Summary|Read/Write errors   |IOExceptions in websockets

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59253] Read/Write errors

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59253

Remy Maucherat  changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Severity|normal  |enhancement

--- Comment #3 from Remy Maucherat  ---
The exception catch strategy is different in websockets (the servlet container
is more complex and catches and routes all these exceptions). Nothing seems
abnormal to me at this point, so it could be an enhancement.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59253] Read/Write errors

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59253

--- Comment #2 from jfclere  ---
32, 104 and 113 are errno on linux
#define EPIPE   32  /* Broken pipe */
#define ECONNRESET  104 /* Connection reset by peer */
#define EHOSTUNREACH113 /* No route to host */

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59261] Request getAsyncContext should throw IllegalStateException if async is not started

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59261

--- Comment #2 from Remy Maucherat  ---
Maybe that is what the specification says, but using null in this situation is
considerably better than using an ISE which should be reserved for some
invalid/meaningless situations.

Bad design ! [being part of that EG, I include myself in that since I missed it
then ...]

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59261] Request getAsyncContext should throw IllegalStateException if async is not started

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59261

--- Comment #1 from Rob Winch  ---
I messed up the description some. Sorry about that. It should read:

If ServletRequest.getAsyncContext() is invoked and has not been put into
asynchronous mode an IllegalStateException should be thrown.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[Bug 59261] New: Request getAsyncContext should throw IllegalStateException if async is not started

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59261

Bug ID: 59261
   Summary: Request getAsyncContext should throw
IllegalStateException if async is not started
   Product: Tomcat 8
   Version: 8.0.33
  Hardware: PC
OS: Mac OS X 10.1
Status: NEW
  Severity: normal
  Priority: P2
 Component: Catalina
  Assignee: dev@tomcat.apache.org
  Reporter: rwi...@gmail.com

If ServletRequest.getAsyncContext() IllegalStateException is invoked and has
not been put into asynchronous mode an . From the javadoc [1]:

> Throws: IllegalStateException - if this request has not been put into
> asynchronous mode, i.e., if neither startAsync() nor 
> startAsync(ServletRequest,ServletResponse) has been called

For implementations of HttpServletRequestWrapper that override this method, the
fact that result can be null can cause problems [2]. It appears there are parts
of tomcat that check if getAsyncContext() is null rather than checking
isAsycStarted(). For example, ApplicationDispatcher checks if getAsyncContext()
is null.


[1]
http://docs.oracle.com/javaee/6/api/javax/servlet/ServletRequest.html#getAsyncContext()
[2] https://github.com/spring-cloud/spring-cloud-netflix/issues/868

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



[GitHub] tomcat-maven-plugin pull request: Bind Tomcat to an address via co...

2016-04-01 Thread seanking
Github user seanking closed the pull request at:

https://github.com/apache/tomcat-maven-plugin/pull/15


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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



[GitHub] tomcat-maven-plugin pull request: add "clientAuth" parameter for t...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/tomcat-maven-plugin/pull/12


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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



svn commit: r1737348 - in /tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner: Tomcat7Runner.java Tomcat7RunnerCli.java

2016-04-01 Thread olamy
Author: olamy
Date: Fri Apr  1 11:28:18 2016
New Revision: 1737348

URL: http://svn.apache.org/viewvc?rev=1737348&view=rev
Log:
executable war: Bind Tomcat to an address via command-line option #15

Merged pr https://github.com/apache/tomcat-maven-plugin/pull/15

Signed-off-by: olivier lamy 

Modified:

tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7Runner.java

tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7RunnerCli.java

Modified: 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7Runner.java
URL: 
http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7Runner.java?rev=1737348&r1=1737347&r2=1737348&view=diff
==
--- 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7Runner.java
 (original)
+++ 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7Runner.java
 Fri Apr  1 11:28:18 2016
@@ -87,6 +87,8 @@ public class Tomcat7Runner
 public static final String HTTP_PORT_KEY = "httpPort";
 
 
+public String httpAddress;
+
 public int httpPort;
 
 public int httpsPort;
@@ -305,15 +307,24 @@ public class Tomcat7Runner
 
 debugMessage( "use connectorHttpProtocol:" + connectorHttpProtocol 
);
 
-if ( httpPort > 0 )
+if ( httpPort > 0  || httpAddress != null)
 {
 Connector connector = new Connector( connectorHttpProtocol );
-connector.setPort( httpPort );
 connector.setMaxPostSize( maxPostSize );
 
-if ( httpsPort > 0 )
+if(httpPort > 0)
+{
+connector.setPort( httpPort );
+
+if ( httpsPort > 0 )
+{
+connector.setRedirectPort( httpsPort );
+}
+}
+
+if( httpAddress != null)
 {
-connector.setRedirectPort( httpsPort );
+connector.setProperty("address", httpAddress);
 }
 connector.setURIEncoding( uriEncoding );
 

Modified: 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7RunnerCli.java
URL: 
http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7RunnerCli.java?rev=1737348&r1=1737347&r2=1737348&view=diff
==
--- 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7RunnerCli.java
 (original)
+++ 
tomcat/maven-plugin/trunk/tomcat7-war-runner/src/main/java/org/apache/tomcat/maven/runner/Tomcat7RunnerCli.java
 Fri Apr  1 11:28:18 2016
@@ -42,6 +42,9 @@ public class Tomcat7RunnerCli
 
 public static final String STAND_ALONE_PROPERTIES_FILENAME = 
"tomcat.standalone.properties";
 
+static Option httpAddress  =
+OptionBuilder.withArgName( "httpAddress" ).hasArg().withDescription( 
"http address to use" ).create( "httpAddress" );
+
 static Option httpPort =
 OptionBuilder.withArgName( "httpPort" ).hasArg().withDescription( 
"http port to use" ).create( "httpPort" );
 
@@ -102,7 +105,8 @@ public class Tomcat7RunnerCli
 
 static
 {
-options.addOption( httpPort ) //
+options.addOption( httpAddress ) //
+.addOption( httpPort ) //
 .addOption( httpsPort ) //
 .addOption( ajpPort ) //
 .addOption( serverXmlPath ) //
@@ -160,6 +164,11 @@ public class Tomcat7RunnerCli
 tomcat7Runner.serverXmlPath = line.getOptionValue( 
serverXmlPath.getOpt() );
 }
 
+if ( line.hasOption( httpAddress.getOpt() ) )
+{
+tomcat7Runner.httpAddress = line.getOptionValue( 
httpAddress.getOpt() );
+}
+
 String port = tomcat7Runner.runtimeProperties.getProperty( 
Tomcat7Runner.HTTP_PORT_KEY );
 if ( port != null )
 {



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



svn commit: r1737347 - /tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java

2016-04-01 Thread olamy
Author: olamy
Date: Fri Apr  1 11:28:13 2016
New Revision: 1737347

URL: http://svn.apache.org/viewvc?rev=1737347&view=rev
Log:
fix https://github.com/apache/tomcat-maven-plugin/pull/12

Signed-off-by: olivier lamy 

Modified:

tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java

Modified: 
tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
URL: 
http://svn.apache.org/viewvc/tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java?rev=1737347&r1=1737346&r2=1737347&view=diff
==
--- 
tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
 (original)
+++ 
tomcat/maven-plugin/trunk/tomcat6-maven-plugin/src/main/java/org/apache/tomcat/maven/plugin/tomcat6/AbstractRunMojo.java
 Fri Apr  1 11:28:13 2016
@@ -287,6 +287,15 @@ public abstract class AbstractRunMojo
 private boolean ignorePackaging;
 
 /**
+ * enable client authentication for https (if configured)
+ * see http://tomcat.apache.org/tomcat-6.0-doc/config/http.html#SSL_Support";>http://tomcat.apache.org/tomcat-6.0-doc/config/http.html#SSL_Support
+ *
+ * @since 2.3
+ */
+@Parameter(property = "maven.tomcat.https.clientAuth", defaultValue = 
"false")
+private String clientAuth;
+
+/**
  * Override the default keystoreFile for the HTTPS connector (if enabled)
  *
  * @since 1.1
@@ -978,6 +987,7 @@ public abstract class AbstractRunMojo
 httpsConnector.setAttribute( "truststoreProvider", 
truststoreProvider );
 }
 
+httpsConnector.setAttribute("clientAuth", clientAuth);
 httpsConnector.setUseBodyEncodingForURI( 
this.useBodyEncodingForURI );
 
 if ( address != null )



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



[Bug 59256] SLF4J in default jarsToSkip value in catalina.properties too broad

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59256

Violeta Georgieva  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Violeta Georgieva  ---
Hi,

The fix will be available from 9.0.0.M5, 8.5.1 and 8.0.34 onwards.

Regards,
Violeta

-- 
You are receiving this mail because:
You are the assignee for the bug.

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



svn commit: r1737341 - in /tomcat/tc8.0.x/trunk: ./ conf/catalina.properties webapps/docs/changelog.xml

2016-04-01 Thread violetagg
Author: violetagg
Date: Fri Apr  1 10:25:04 2016
New Revision: 1737341

URL: http://svn.apache.org/viewvc?rev=1737341&view=rev
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59256
Merged revision 1737339 from tomcat/trunk:
slf4j-taglib*.jar should not be excluded from the standard JAR scanning by 
default.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/conf/catalina.properties
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Apr  1 10:25:04 2016
@@ -1,2 +1,2 @@
 /tomcat/tc8.5.x/trunk:1735042
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,1637890,1637892,1638720-1638725,1639653,1640010,1640083-1640084,1640088,1640275,1640322,1640347,1640361,1640365,1640403,1640410,1640652,1640655-1640658,1640688,1640700-1640883,1640903,1640976,1640978,1641000,1641026,1641038-1641039,1641051-1641052,1641058,1641064,1641300,1641369,1641374,1641380,1641486,1641634,1641656-1641692,1641704,1641707-1641718,1641720-1641722,1641735,1641981,1642233,1642280,1642554,1642564,1642595,1642606,1642668,1642679,1642697,1642699,1642766,1643002,1643045,1643054-1643055,1643066,1643121,1643128,1643206,1643209-1643210,1643216,1643249,1643270,1643283,1643309-1643310,1643323,1643365-1643366,1643370-1643371,1643465,1643474,1643536,1643570,1643634,1643649,1643651,1643654,1643675,1643731,1643733-1643734,1643761,1643766,1643814,1643937,1643963,1644017,1644169,1644201-1644203,1644321,1644323,1644516,1644523,1644529,1644535,1644730,1644768,1644784-1644785,1644790,1644793,1644815,1644884,1644886
 
,1644890,1644892,1644910,1644924,1644929-1644930,1644935,1644989,1645011,1645247,1645355,1645357-1645358,1645455,1645465,1645469,1645471,1645473,1645475,1645486-1645488,1645626,1645641,1645685,1645743,1645763,1645951-1645953,1645955,1645993,1646098-1646106,1646178,1646220,1646302,1646304,1646420,1646470-1646471,1646476,1646559,1646717-1646723,1646773,1647026,1647042,1647530,1647655,1648304,1648815,1648907,1649973,1650081,1650365,1651116,1651120,1651280,1651470,1652938,1652970,1653041,1653471,1653550,1653574,1653797,1653815-1653816,1653819,1653840,1653857,1653888,1653972,1654013,1654030,1654050,1654123,1654148,1654159,1654513,1654515,1654517,1654522,1654524,1654725,1654735,1654766,1654785,1654851-1654852,1654978,1655122-1655124,1655126-1655127,1655129-1655130,1655132-1655133,1655312,1655351,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657
 
592,1657607,1657609,1657682,1657907,1658207,1658734,1658781,1658790,1658799,1658802,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659174,1659184,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661770,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662696,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085,1665292,1665559,1665653,1665661,1665672,1665694,1665697,1665736,1665779,1665976-1665977,1665980-1665981,1665985-1665986,1665989,1665998,1666004,1666008,1666013,1666017,1666024,1666116,1666386-1
 
666387,1666494,1666496,1666552,1666569,1666579,137,149,1666757,1666966,1666972,1666985,1666995,1666997,1667292,1667402,1667406,1667546,1667615,1667630,1667636,1667688,1667764,1667871,1668026,1668135,1668193,1668593,1668596,1668630,1668639,1668843,1669353,1669370,1669451,1669800,1669838,1669876,1669882,1670394,1670433,1670591,1670598-1670600,1670610,1670631,1670719,1670724,1670726,1670730,1670940,1671112,1672272,1672284,1673754,1674294,1675461,1675486,1675594,1675830,1676231,1676250-1676251,1676364,1676381,1676393,1676479,1676525,1676552,1676615,1676630,1676634,1676721,1676926,1676943,1677140,1677802,1678011,1678162,1678174,1678339,1678426-1678427,1678694,1678701,1679534,1679708,1679710,1679716,1680034,1680246,1681056,1681123,1681138,1681280,1681283,1681286,1681450,1681697,1681699,1681701,1681729,1681770,1681779,1681793,1681807,1681837-1681838,1681854,1681862,1681958,1682028,1682033,1682311,1682315,1682317,1682320,1682324,1682330,1682842,1684172,1684366,1684383,1684526-168452
 
7,1684549-1684550,1685556,1685591,1685739,1685744,1685772,1685816,1685826,1685891,1687242,1687261,1687268,1687340,1687551,1688563,1688841,1688878,165,1688896,1688901,1689345-1689346,1689357,1689656,1689675-1689677,1689679,1689687,1689825,1689856,1689918,1690011,1690021,1690054,1690080,1

svn commit: r1737340 - in /tomcat/tc8.5.x/trunk: ./ conf/catalina.properties webapps/docs/changelog.xml

2016-04-01 Thread violetagg
Author: violetagg
Date: Fri Apr  1 10:23:25 2016
New Revision: 1737340

URL: http://svn.apache.org/viewvc?rev=1737340&view=rev
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59256
Merged revision 1737339 from tomcat/trunk:
slf4j-taglib*.jar should not be excluded from the standard JAR scanning by 
default.

Modified:
tomcat/tc8.5.x/trunk/   (props changed)
tomcat/tc8.5.x/trunk/conf/catalina.properties
tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.5.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Apr  1 10:23:25 2016
@@ -1 +1 @@
-/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280
+/tomcat/trunk:1734785,1734799,1734845,1734928,1735041,1735044,1735480,1735577,1735597,1735599-1735600,1735615,1736145,1736162,1736209,1736280,1736297,1736299,1736489,1736646,1736703,1736836,1736849,1737104-1737105,1737112,1737117,1737119-1737120,1737155,1737157,1737192,1737280,1737339

Modified: tomcat/tc8.5.x/trunk/conf/catalina.properties
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/conf/catalina.properties?rev=1737340&r1=1737339&r2=1737340&view=diff
==
--- tomcat/tc8.5.x/trunk/conf/catalina.properties (original)
+++ tomcat/tc8.5.x/trunk/conf/catalina.properties Fri Apr  1 10:23:25 2016
@@ -137,7 +137,8 @@ xom-*.jar
 # has been excluded by a broad file name pattern in the jarsToSkip list.
 # The list of JARs to scan may be over-ridden at a Context level for individual
 # scan types by configuring a JarScanner with a nested JarScanFilter.
-tomcat.util.scan.StandardJarScanFilter.jarsToScan=log4j-core*.jar,log4j-taglib*.jar,log4javascript*.jar
+tomcat.util.scan.StandardJarScanFilter.jarsToScan=\
+log4j-core*.jar,log4j-taglib*.jar,log4javascript*.jar,slf4j-taglib*.jar
 
 # String cache configuration.
 tomcat.util.buf.StringCache.byte.enabled=true

Modified: tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml?rev=1737340&r1=1737339&r2=1737340&view=diff
==
--- tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.5.x/trunk/webapps/docs/changelog.xml Fri Apr  1 10:23:25 2016
@@ -77,6 +77,10 @@
   
 59255: Fix possible NPE in mapper. (kkolinko/remm)
   
+  
+59256: slf4j-taglib*.jar should not be excluded
+from the standard JAR scanning by default. (violetagg)
+  
 
   
   



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



svn commit: r1737339 - in /tomcat/trunk: conf/catalina.properties webapps/docs/changelog.xml

2016-04-01 Thread violetagg
Author: violetagg
Date: Fri Apr  1 10:21:15 2016
New Revision: 1737339

URL: http://svn.apache.org/viewvc?rev=1737339&view=rev
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=59256
slf4j-taglib*.jar should not be excluded from the standard JAR scanning by 
default.

Modified:
tomcat/trunk/conf/catalina.properties
tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/conf/catalina.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/conf/catalina.properties?rev=1737339&r1=1737338&r2=1737339&view=diff
==
--- tomcat/trunk/conf/catalina.properties (original)
+++ tomcat/trunk/conf/catalina.properties Fri Apr  1 10:21:15 2016
@@ -137,7 +137,8 @@ xom-*.jar
 # has been excluded by a broad file name pattern in the jarsToSkip list.
 # The list of JARs to scan may be over-ridden at a Context level for individual
 # scan types by configuring a JarScanner with a nested JarScanFilter.
-tomcat.util.scan.StandardJarScanFilter.jarsToScan=log4j-core*.jar,log4j-taglib*.jar,log4javascript*.jar
+tomcat.util.scan.StandardJarScanFilter.jarsToScan=\
+log4j-core*.jar,log4j-taglib*.jar,log4javascript*.jar,slf4j-taglib*.jar
 
 # String cache configuration.
 tomcat.util.buf.StringCache.byte.enabled=true

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1737339&r1=1737338&r2=1737339&view=diff
==
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Fri Apr  1 10:21:15 2016
@@ -87,6 +87,10 @@
   
 59255: Fix possible NPE in mapper. (kkolinko/remm)
   
+  
+59256: slf4j-taglib*.jar should not be excluded
+from the standard JAR scanning by default. (violetagg)
+  
 
   
   



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



[Bug 59256] SLF4J in default jarsToSkip value in catalina.properties too broad

2016-04-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=59256

--- Comment #2 from David Scourfield  ---
Sure, something like: slf4j-taglib-*.jar

-- 
You are receiving this mail because:
You are the assignee for the bug.

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