svn commit: r1658734 - in /tomcat/trunk/java/org/apache/coyote/http11/upgrade: LocalStrings.properties UpgradeProcessor.java

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 15:43:46 2015
New Revision: 1658734

URL: http://svn.apache.org/r1658734
Log:
Ensure that a dropped connection does not leave references to the 
UpgradeProcessor

Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java

Modified: 
tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties?rev=1658734r1=1658733r2=1658734view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties 
Tue Feb 10 15:43:46 2015
@@ -14,6 +14,8 @@
 # limitations under the License.
 
 upgradeProcessor.isCloseFail=Failed to close input stream associated with 
upgraded connection
+upgradeProcessor.onDataAvailableFail=Failed to process data available event
+upgradeProcessor.onWritePossibleFail=Failed to process write possible event
 upgradeProcessor.osCloseFail=Failed to close output stream associated with 
upgraded connection
 
 upgrade.sis.isFinished.ise=It is illegal to call isFinished() when the 
ServletInputStream is not in non-blocking mode (i.e. setReadListener() must be 
called first)

Modified: 
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java?rev=1658734r1=1658733r2=1658734view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
Tue Feb 10 15:43:46 2015
@@ -98,10 +98,24 @@ public class UpgradeProcessor implements
 @Override
 public final SocketState upgradeDispatch(SocketStatus status) throws 
IOException {
 if (status == SocketStatus.OPEN_READ) {
-upgradeServletInputStream.onDataAvailable();
-upgradeServletOutputStream.checkWriteDispatch();
+try {
+upgradeServletInputStream.onDataAvailable();
+upgradeServletOutputStream.checkWriteDispatch();
+} catch (IOException ioe) {
+// The error handling within the ServletInputStream should have
+// marked the stream for closure which will get picked up 
below,
+// triggering the clean-up of this processor.
+
log.debug(sm.getString(upgradeProcessor.onDataAvailableFail), ioe);
+}
 } else if (status == SocketStatus.OPEN_WRITE) {
-upgradeServletOutputStream.onWritePossible();
+try {
+upgradeServletOutputStream.onWritePossible();
+} catch (IOException ioe) {
+// The error handling within the ServletOutputStream should 
have
+// marked the stream for closure which will get picked up 
below,
+// triggering the clean-up of this processor.
+
log.debug(sm.getString(upgradeProcessor.onWritePossibleFail), ioe);
+}
 } else if (status == SocketStatus.STOP) {
 try {
 upgradeServletInputStream.close();



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



svn commit: r1658738 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/AbstractProcessor.java java/org/apache/coyote/http11/upgrade/LocalStrings.properties webapps/docs/changelog.xm

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 15:54:05 2015
New Revision: 1658738

URL: http://svn.apache.org/r1658738
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57546
Ensure that a dropped connection does not leave references to the 
UpgradeProcessor

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java

tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 15:54:05 2015
@@ -1,2 +1,2 @@
-/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1647030,1647043,1648816,1651420-1651422,1651844,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1655109,1655112,1655114-1655115,1655117,1655120,1655445,1655467,1656300,1657049,1657356,1657360,1657370,1657460,1657908,1658208
-/tomcat/trunk:1156115-1157160,1157162-1157859,1157862-1157942,1157945-1160347,1160349-1163716,1163718-1166689,1166691-1174340,1174342-1175596,1175598-1175611,1175613-1175932,1175934-1177783,1177785-1177980,1178006-1180720,1180722-1183094,1183096-1187753,1187755,1187775,1187801,1187806,1187809,1187826-1188312,1188314-1188401,1188646-1188840,1188842-1190176,1190178-1195223,1195225-1195953,1195955,1195957-1201238,1201240-1203345,1203347-1206623,1206625-1208046,1208073,1208096,1208114,1208145,1208772,1209194-1212125,1212127-1220291,1220293,1220295-1221321,1221323-1222329,1222332-1222401,1222405-1222795,1222850-1222950,1222969-1225326,1225328-1225463,1225465,1225627,1225629-1226534,1226536-1228908,1228911-1228923,1228927-1229532,1229534-1230766,1230768-1231625,1231627-1233414,1233419-1235207,1235209-1237425,1237427,1237429-1237977,1237981,1237985,1237995,1238070,1238073,1239024-1239048,1239050-1239062,1239135,1239256,1239258-1239485,1239785-1240046,1240101,1240106,1240109,1240112,1240114
 
,1240116,1240118,1240121,1240329,1240474-1240850,1240857,1241087,1241160,1241408-1241822,1241908-1241909,1241912-1242110,1242371-1292130,1292134-1292458,1292464-1292670,1292672-1292776,1292780-1293392,1293397-1297017,1297019-1297963,1297965-1299820,1300108,1300111-1300460,1300520-1300948,1300997,1301006,1301280,1302332,1302348,1302608-1302610,1302649,1302837,1303138,1303163,1303338,1303521,1303587,1303698,1303803,1303852,1304011,1304035,1304037,1304135,1304249,1304253,1304260,1304271,1304275,1304468,1304895,1304930-1304932,1305194,1305943,1305965,1306556,1306579-1306580,1307084,1307310,1307511-1307512,1307579,1307591,1307597,1310636,1310639-1310640,1310642,1310701,1311212,1311995,1327617,1327670,1331766,1333161,1333173,1333827,1334787,1335026,1335257,1335547,1335692,1335711,1335731,1336515,1336813,1336864,1336868,1336884,1337419,1337426,1337546,1337572,1337591-1337595,1337643,1337707,1337719,1337734,1337741,1337745,1338151-1338154,1338178,1342027,1342029,1342315,1342320,1342476,1342
 
498,1342503,1342717,1342795,1342805,1343044-1343046,1343335,1343394,1343400,1343629,1343708,1343718,1343895,1344063,1344068,1344250,1344266,1344515,1344528,1344612,1344629,1344725,1344868,1344890,1344893,1344896,1344901,1345020,1345029,1345039,1345287-1345290,1345294,1345309,1345325,1345357,1345367,1345579-1345580,1345582,1345688,1345699,1345704,1345731-1345732,1345737,1345744,1345752,1345754,1345779,1345781,1345846,1346107,1346365,1346376,1346404,1346510,1346514,1346519,1346581,1346635,1346644,1346683,1346794,1346885,1346932,1347034,1347047,1347087,1347108-1347109,1347583,1347737,1348105,1348357,1348398,1348425,1348461-1348495,1348498,1348752,1348762,1348772,1348776,1348859,1348968,1348973,1348989,1349007,1349237,1349298,1349317,1349410,1349473,1349539,1349879,1349887,1349893,1349922,1349984,1350124,1350241,1350243,1350294-1350295,1350299,1350864,1350900,1351010,1351054,1351056,1351068,1351134-1351135,1351148,1351259,1351604,1351636-1351640,1351991,1351993,1352011,1352056,1352059,1
 

[Bug 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

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

--- Comment #5 from Mark Thomas ma...@apache.org ---
Thanks for the report and the pointers to reproduce this.

I have fixed the problem in trunk, 8.0.x (for 8.0.19 onwards) and 7.0.x (for
7.0.60 onwards).

-- 
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 57559] New: Decoded Request URI used for Asynchronous dispatch

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57559

Bug ID: 57559
   Summary: Decoded Request URI used for Asynchronous dispatch
   Product: Tomcat 7
   Version: 7.0.53
  Hardware: PC
Status: NEW
  Severity: normal
  Priority: P2
 Component: Catalina
  Assignee: dev@tomcat.apache.org
  Reporter: cg.throwa...@mailinator.com

Overview:

In a scenario where an original request contains part of the URI
percent-encoded (e.g. /context/foo/%24/bar), when the request is put into
asynchronous mode using ServletRequest.startAsync() and then dispatched using
the no-argument AsyncContext.dispatch() method, the dispatch is incorrectly
made to a percent-decoded URI (e.g. /context/foo/$/bar), not the original
request URI.

Steps to Reproduce:

Consider the following servlet:
https://gist.github.com/anonymous/957122b45ae656a5e8f1
It puts a request into asynchronous mode using ServletRequest.startAsync() and
immediately dispatches it using the no-argument AsyncContext.dispatch() method.
When the asynchronous dispatch takes place, the request URI is written to the
response.

1) Compile and deploy the servlet to http://localhost:8080/context
2) Visit http://localhost:8080/context/foo/%24/bar (or any similar
percent-encoded URI) in your browser

Actual Results:

The printed-out request URI is /context/foo/$/bar (i.e. the request has been
dispatched to /context/foo/$/bar, a percent-decoded version of the original
URI)

Expected Results:

The printed-out request URI is /context/foo/%24/bar (i.e. the request should be
dispatched to /context/foo/%24/bar, the original URI)

Additional Information:

The Java Servlet 3.0 Specification says: The dispatch method takes no
argument. It uses the original URI as the path. If the AsyncContext was
initialized via the startAsync(ServletRequest, ServletResponse) and the request
passed is an instance of HttpServletRequest, then the dispatch is to the URI
returned by HttpServletRequest.getRequestURI(). Otherwise the dispatch is to
the URI of the request when it was last dispatched by the container.

It seems clear that the dispatch should be to the original request URI as
obtained from HttpServletRequest.getRequestURI(), and the Javadoc of
HttpServletRequest.getRequestURI() states that The web container does not
decode this String. Therefore, it seems the asynchronous dispatch should not
be made to a percent-decoded version of the original URI, but to the original
URI as-is.

-- 
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: r1658724 - in /tomcat/native/trunk/native: include/ssl_private.h src/sslutils.c

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 14:40:37 2015
New Revision: 1658724

URL: http://svn.apache.org/r1658724
Log:
Replace access to OpenSSL SSL internals
by accessor function. Internals are no longer
available for OpenSSL master.

In openssl/ssl.h the macro is defined as
#define SSL_CTX_clear_extra_chain_certs(ctx) \
SSL_CTX_ctrl(ctx,SSL_CTRL_CLEAR_EXTRA_CHAIN_CERTS,0,NULL)

The implementation of SSL_CTRL_CLEAR_EXTRA_CHAIN_CERTS does NULL check and
does call sk_X509_pop_free(), so there no more need to do it here.
http://osxr.org/openssl/source/ssl/s3_lib.c#3637

Not compiled or tested, waiting for GUMP.

Modified:
tomcat/native/trunk/native/include/ssl_private.h
tomcat/native/trunk/native/src/sslutils.c

Modified: tomcat/native/trunk/native/include/ssl_private.h
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/include/ssl_private.h?rev=1658724r1=1658723r2=1658724view=diff
==
--- tomcat/native/trunk/native/include/ssl_private.h (original)
+++ tomcat/native/trunk/native/include/ssl_private.h Tue Feb 10 14:40:37 2015
@@ -298,12 +298,6 @@ typedef struct {
 } tcn_ssl_conn_t;
 
 
-#define SSL_CTX_get_extra_certs(ctx)((ctx)-extra_certs)
-#define SSL_CTX_set_extra_certs(ctx, value) \
-TCN_BEGIN_MACRO \
-(ctx)-extra_certs = (value);   \
-TCN_END_MACRO
-
 /*
  *  Additional Functions
  */

Modified: tomcat/native/trunk/native/src/sslutils.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/src/sslutils.c?rev=1658724r1=1658723r2=1658724view=diff
==
--- tomcat/native/trunk/native/src/sslutils.c (original)
+++ tomcat/native/trunk/native/src/sslutils.c Tue Feb 10 14:40:37 2015
@@ -405,7 +405,6 @@ int SSL_CTX_use_certificate_chain(SSL_CT
 X509 *x509;
 unsigned long err;
 int n;
-STACK_OF(X509) *extra_certs;
 
 if ((bio = BIO_new(BIO_s_file_internal())) == NULL)
 return -1;
@@ -421,12 +420,10 @@ int SSL_CTX_use_certificate_chain(SSL_CT
 }
 X509_free(x509);
 }
+
 /* free a perhaps already configured extra chain */
-extra_certs = SSL_CTX_get_extra_certs(ctx);
-if (extra_certs != NULL) {
-sk_X509_pop_free(extra_certs, X509_free);
-SSL_CTX_set_extra_certs(ctx,NULL);
-}
+SSL_CTX_clear_extra_chain_certs(ctx);
+
 /* create new extra chain by loading the certs */
 n = 0;
 while ((x509 = PEM_read_bio_X509(bio, NULL, NULL, NULL)) != NULL) {



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



svn commit: r1658737 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/AbstractProcessor.java java/org/apache/coyote/http11/upgrade/LocalStrings.properties webapps/docs/changelog.xm

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 15:53:06 2015
New Revision: 1658737

URL: http://svn.apache.org/r1658737
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57546
Ensure that a dropped connection does not leave references to the 
UpgradeProcessor

Modified:
tomcat/tc8.0.x/trunk/   (props changed)

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 15:53:06 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java
URL: 

Re: svn commit: r1658734 - in /tomcat/trunk/java/org/apache/coyote/http11/upgrade: LocalStrings.properties UpgradeProcessor.java

2015-02-10 Thread Rémy Maucherat
2015-02-10 16:43 GMT+01:00 ma...@apache.org:

 Author: markt
 Date: Tue Feb 10 15:43:46 2015
 New Revision: 1658734

 URL: http://svn.apache.org/r1658734
 Log:
 Ensure that a dropped connection does not leave references to the
 UpgradeProcessor

 Good find, but what happens if onDataAvailable or onWritePossible throw a
runtime exception (like a NPE), since this is user code. It could also leak
then ? Shouldn't the code catch everything, also call onError on the
listener and close ?

Rémy


svn commit: r1658728 - in /tomcat/native/branches/1.1.x: ./ native/include/ssl_private.h native/src/sslutils.c xdocs/miscellaneous/changelog.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 14:45:19 2015
New Revision: 1658728

URL: http://svn.apache.org/r1658728
Log:
Replace access to OpenSSL SSL internals
by accessor function. Internals are no longer
available for OpenSSL master.

In openssl/ssl.h the macro is defined as
#define SSL_CTX_clear_extra_chain_certs(ctx) \
SSL_CTX_ctrl(ctx,SSL_CTRL_CLEAR_EXTRA_CHAIN_CERTS,0,NULL)

The implementation of SSL_CTRL_CLEAR_EXTRA_CHAIN_CERTS does NULL check and
does call sk_X509_pop_free(), so there no more need to do it here.
http://osxr.org/openssl/source/ssl/s3_lib.c#3637

Not compiled or tested, waiting for GUMP.

It is merge of r1658724 from tomcat/native/trunk.

Modified:
tomcat/native/branches/1.1.x/   (props changed)
tomcat/native/branches/1.1.x/native/include/ssl_private.h
tomcat/native/branches/1.1.x/native/src/sslutils.c
tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml

Propchange: tomcat/native/branches/1.1.x/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 14:45:19 2015
@@ -1,3 +1,3 @@
-/tomcat/native/trunk:815411,1342003,1342008,1342013,1342020,1342024,1394258,1394342,1424947,1424971,1430753,1437081,1438342,1439337,1441884,1441886,1442579,1442581,1445972,1507113,1532577,1532590,1539594,1555184,1559180,1588195,1607262,1607267,1607278,1607291,1607477,1648821,1650119,1658557,1658641-1658642
+/tomcat/native/trunk:815411,1342003,1342008,1342013,1342020,1342024,1394258,1394342,1424947,1424971,1430753,1437081,1438342,1439337,1441884,1441886,1442579,1442581,1445972,1507113,1532577,1532590,1539594,1555184,1559180,1588195,1607262,1607267,1607278,1607291,1607477,1648821,1650119,1658557,1658641-1658642,1658724
 /tomcat/tc7.0.x/trunk:1199985,1200164,1349932,1434887,1435769
 
/tomcat/trunk:815418,832198,1001939,1033916,1043103,1044729,1078522,1145209,1145285,1149092,1241356,1241406-1241407,1242254,1292671,1299980,1300102,1434905,1437083

Modified: tomcat/native/branches/1.1.x/native/include/ssl_private.h
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/include/ssl_private.h?rev=1658728r1=1658727r2=1658728view=diff
==
--- tomcat/native/branches/1.1.x/native/include/ssl_private.h (original)
+++ tomcat/native/branches/1.1.x/native/include/ssl_private.h Tue Feb 10 
14:45:19 2015
@@ -295,12 +295,6 @@ typedef struct {
 } tcn_ssl_conn_t;
 
 
-#define SSL_CTX_get_extra_certs(ctx)((ctx)-extra_certs)
-#define SSL_CTX_set_extra_certs(ctx, value) \
-TCN_BEGIN_MACRO \
-(ctx)-extra_certs = (value);   \
-TCN_END_MACRO
-
 /*
  *  Additional Functions
  */

Modified: tomcat/native/branches/1.1.x/native/src/sslutils.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/src/sslutils.c?rev=1658728r1=1658727r2=1658728view=diff
==
--- tomcat/native/branches/1.1.x/native/src/sslutils.c (original)
+++ tomcat/native/branches/1.1.x/native/src/sslutils.c Tue Feb 10 14:45:19 2015
@@ -420,7 +420,6 @@ int SSL_CTX_use_certificate_chain(SSL_CT
 X509 *x509;
 unsigned long err;
 int n;
-STACK_OF(X509) *extra_certs;
 
 if ((bio = BIO_new(BIO_s_file_internal())) == NULL)
 return -1;
@@ -436,12 +435,10 @@ int SSL_CTX_use_certificate_chain(SSL_CT
 }
 X509_free(x509);
 }
+
 /* free a perhaps already configured extra chain */
-extra_certs = SSL_CTX_get_extra_certs(ctx);
-if (extra_certs != NULL) {
-sk_X509_pop_free(extra_certs, X509_free);
-SSL_CTX_set_extra_certs(ctx,NULL);
-}
+SSL_CTX_clear_extra_chain_certs(ctx);
+
 /* create new extra chain by loading the certs */
 n = 0;
 while ((x509 = PEM_read_bio_X509(bio, NULL, NULL, NULL)) != NULL) {

Modified: tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml?rev=1658728r1=1658727r2=1658728view=diff
==
--- tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/native/branches/1.1.x/xdocs/miscellaneous/changelog.xml Tue Feb 10 
14:45:19 2015
@@ -40,7 +40,7 @@
   changelog
 fix
   Fix compilation failures with master branch of OpenSSL. Replace access
-  to OpenSSL session internals by accessor function. (rjung)
+  to OpenSSL internals by accessor functions. (rjung/kkolinko)
 /fix
   /changelog
 /section



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



svn commit: r1658854 - in /tomcat/site/trunk: docs/ci.html xdocs/ci.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Wed Feb 11 01:32:58 2015
New Revision: 1658854

URL: http://svn.apache.org/r1658854
Log:
Add links to Gump results for Tomcat-Native

Modified:
tomcat/site/trunk/docs/ci.html
tomcat/site/trunk/xdocs/ci.xml

Modified: tomcat/site/trunk/docs/ci.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/ci.html?rev=1658854r1=1658853r2=1658854view=diff
==
--- tomcat/site/trunk/docs/ci.html (original)
+++ tomcat/site/trunk/docs/ci.html Wed Feb 11 01:32:58 2015
@@ -679,6 +679,46 @@ prepared and published by ASF Buildbot,
 
 tr style=vertical-align:top
   
+tda 
href=http://vmgump.apache.org/gump/public/tomcat-native/;tomcat-native/a/td
+  td
+
+ul
+  
+liSource path: code/tomcat/native/branches/1.1.x/code
+/li
+  
+li
+strongProjects:/strong
+
+ul
+  
+li
+a 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-buildconf/;tomcat-native-buildconf/a
+/li
+  
+li
+a 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-configure/;tomcat-native-configure/a
+/li
+  
+li
+a 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-make/;tomcat-native-make/a
 - performs the actual build/li
+  
+li
+a 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-make-install/;tomcat-native-make-install/a
+/li
+
+/ul
+  
+/li
+
+/ul
+  
+/td
+
+/tr
+
+tr style=vertical-align:top
+  
 tda 
href=http://vmgump.apache.org/gump/public/tomcat-taglibs/;tomcat-taglibs/a/td
   td
 

Modified: tomcat/site/trunk/xdocs/ci.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/ci.xml?rev=1658854r1=1658853r2=1658854view=diff
==
--- tomcat/site/trunk/xdocs/ci.xml (original)
+++ tomcat/site/trunk/xdocs/ci.xml Wed Feb 11 01:32:58 2015
@@ -204,6 +204,22 @@ prepared and published by ASF Buildbot,
   /td
 /tr
 tr style=vertical-align:top
+  tda 
href=http://vmgump.apache.org/gump/public/tomcat-native/;tomcat-native/a/td
+  td
+ul
+  liSource path: code/tomcat/native/branches/1.1.x/code/li
+  listrongProjects:/strong
+ul
+  lia 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-buildconf/;tomcat-native-buildconf/a/li
+  lia 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-configure/;tomcat-native-configure/a/li
+  lia 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-make/;tomcat-native-make/a
 - performs the actual build/li
+  lia 
href=http://vmgump.apache.org/gump/public/tomcat-native/tomcat-native-make-install/;tomcat-native-make-install/a/li
+/ul
+  /li
+/ul
+  /td
+/tr
+tr style=vertical-align:top
   tda 
href=http://vmgump.apache.org/gump/public/tomcat-taglibs/;tomcat-taglibs/a/td
   td
 ul



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



[GUMP@vmgump]: Project tomcat-tc7.0.x-test-bio (in module tomcat-7.0.x) failed

2015-02-10 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-tc7.0.x-test-bio has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 5 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-tc7.0.x-test-bio :  Tomcat 7.x, a web server implementing Java 
Servlet 3.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-bio/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on tomcat-tc7.0.x-dbcp exists, no need to add for property 
tomcat-dbcp-src.jar.
 -DEBUG- Dependency on commons-daemon exists, no need to add for property 
commons-daemon.native.src.tgz.
 -DEBUG- Dependency on commons-daemon exists, no need to add for property 
tomcat-native.tar.gz.
 -DEBUG- Dependency on tomcat-tc7.0.x-dbcp exists, no need to add for property 
tomcat-dbcp.home.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-7.0.x/output/logs-BIO
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-7.0.x/output/test-tmp-BIO/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-bio/gump_work/build_tomcat-7.0.x_tomcat-tc7.0.x-test-bio.html
Work Name: build_tomcat-7.0.x_tomcat-tc7.0.x-test-bio (Type: Build)
Work ended in a state of : Failed
Elapsed: 17 mins 46 secs
Command Line: /usr/lib/jvm/java-8-oracle/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Djunit.jar=/srv/gump/public/workspace/junit/target/junit-4.13-SNAPSHOT.jar 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150211-native-src.tar.gz
 -Dtest.reports=output/logs-BIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150211-native-src.tar.gz
 -Dexamples.sources.skip=true 
-Dtomcat-dbcp.home=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps 
-Djdt.jar=/srv/gump/packages/eclipse/plugins/R-4.4-201406061215/ecj-4.4.jar 
-Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20150211.jar
 
-Dtomcat-dbcp-src.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-src.jar
 -Dexecute.test.nio=false -Dtest.accesslog=true 
-Dcommons-pool.home=/srv/gump/public/wo
 rkspace/commons-pool-1.x 
-Dcommons-dbcp.home=/srv/gump/public/workspace/commons-dbcp-1.x 
-Dexecute.test.apr=false -Dexecute.test.bio=true 
-Dtomcat-dbcp.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-20150211.jar
 -Dtest.temp=output/test-tmp-BIO 
-Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-7.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-7.0.x/output/testclasses:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit4.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/servlet-api.ja
 

[VOTE] Release Apache Taglibs 1.2.3

2015-02-10 Thread Jeremy Boynes
The proposed Apache Taglibs 1.2.3 release is now available for voting.

It can be obtained from:
 https://dist.apache.org/repos/dist/dev/tomcat/taglibs/taglibs-standard-1.2.3/

The Maven staging repo is:
 https://repository.apache.org/content/repositories/orgapachetomcat-1034/

The SVN tag is:
 
http://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/

The proposed 1.2.3 release is:
[ ] Broken - do not release
[ ] Stable - go ahead and release as 1.2.3 Stable

Thanks
Jeremy


signature.asc
Description: Message signed with OpenPGP using GPGMail


[GUMP@vmgump]: Project tomcat-tc8.0.x-validate (in module tomcat-8.0.x) failed

2015-02-10 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-tc8.0.x-validate has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 10 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-tc8.0.x-validate :  Tomcat 8.x, a web server implementing the Java 
Servlet 3.1,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-validate/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on checkstyle exists, no need to add for property 
checkstyle.jar.
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-validate/gump_work/build_tomcat-8.0.x_tomcat-tc8.0.x-validate.html
Work Name: build_tomcat-8.0.x_tomcat-tc8.0.x-validate (Type: Build)
Work ended in a state of : Failed
Elapsed: 31 secs
Command Line: /usr/lib/jvm/java-8-oracle/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dcheckstyle.jar=/srv/gump/public/workspace/checkstyle/target/checkstyle-6.4-SNAPSHOT.jar
 -Dexecute.validate=true validate 
[Working Directory: /srv/gump/public/workspace/tomcat-8.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit4.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/packages/antlr/antlr-3.1.3.jar:/srv/gump/public/workspace/checkstyle/target/checkstyle-6.4-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/beanutils/dist/commons-beanutils-20150211.jar:/srv/gump/public/workspace/apache-commons/cli/target/commons-cli-1.3-SNAPSHOT.jar:/srv/gump/packages/commons-collections3/commons-collections-3.2.1.jar:/srv/gump/public/workspace/apache-commons/exec/target/commons-exec-1.3.1-SNAPSHOT.j
 
ar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-20150211.jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-api-20150211.jar:/srv/gump/public/workspace/apache-commons/validator/dist/commons-validator-20150211.jar:/srv/gump/packages/guava/guava-18.0.jar
-
Buildfile: /srv/gump/public/workspace/tomcat-8.0.x/build.xml

build-prepare:
   [delete] Deleting directory 
/srv/gump/public/workspace/tomcat-8.0.x/output/build/temp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-8.0.x/output/build/temp

compile-prepare:

download-validate:

testexist:
 [echo] Testing  for 
/srv/gump/public/workspace/checkstyle/target/checkstyle-6.4-SNAPSHOT.jar

setproxy:

downloadfile:

validate:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-8.0.x/output/res/checkstyle
[checkstyle] Running Checkstyle 6.4-SNAPSHOT on 2939 files
[checkstyle] 
/srv/gump/public/workspace/tomcat-8.0.x/webapps/docs/changelog.xml:71: Line 
matches the illegal pattern '\s+$'.

BUILD FAILED
/srv/gump/public/workspace/tomcat-8.0.x/build.xml:545: Got 1 errors and 0 
warnings.

Total time: 31 seconds
-

To subscribe to this information via syndicated feeds:
- RSS: 
http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-validate/rss.xml
- Atom: 
http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-validate/atom.xml

== Gump Tracking Only ===
Produced by Apache Gump(TM) version 2.3.
Gump Run 2015021104, vmgump.apache.org:vmgump:2015021104
Gump E-mail Identifier (unique within run) #3.

--
Apache Gump
http://gump.apache.org/ [Instance: vmgump]

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



[Bug 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

--- Comment #8 from Paul Gaylie paul.gay...@gmail.com ---
Hi Mark,

Great that you found the bug.

Exactly what code needed to be updated? Just curious where the proper fix
needed to be applied - was driving me nuts.

-- 
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



[GUMP@vmgump]: Project tomcat-tc7.0.x-test-nio (in module tomcat-7.0.x) failed

2015-02-10 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-tc7.0.x-test-nio has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 5 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-tc7.0.x-test-nio :  Tomcat 7.x, a web server implementing Java 
Servlet 3.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-nio/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on tomcat-tc7.0.x-dbcp exists, no need to add for property 
tomcat-dbcp-src.jar.
 -DEBUG- Dependency on commons-daemon exists, no need to add for property 
commons-daemon.native.src.tgz.
 -DEBUG- Dependency on commons-daemon exists, no need to add for property 
tomcat-native.tar.gz.
 -DEBUG- Dependency on tomcat-tc7.0.x-dbcp exists, no need to add for property 
tomcat-dbcp.home.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-7.0.x/output/logs-NIO
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-7.0.x/output/test-tmp-NIO/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-nio/gump_work/build_tomcat-7.0.x_tomcat-tc7.0.x-test-nio.html
Work Name: build_tomcat-7.0.x_tomcat-tc7.0.x-test-nio (Type: Build)
Work ended in a state of : Failed
Elapsed: 18 mins 34 secs
Command Line: /usr/lib/jvm/java-8-oracle/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Djunit.jar=/srv/gump/public/workspace/junit/target/junit-4.13-SNAPSHOT.jar 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150211-native-src.tar.gz
 -Dtest.reports=output/logs-NIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150211-native-src.tar.gz
 -Dexamples.sources.skip=true 
-Dtomcat-dbcp.home=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps 
-Djdt.jar=/srv/gump/packages/eclipse/plugins/R-4.4-201406061215/ecj-4.4.jar 
-Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20150211.jar
 
-Dtomcat-dbcp-src.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-src.jar
 -Dexecute.test.nio=true -Dtest.accesslog=true 
-Dcommons-pool.home=/srv/gump/public/wor
 kspace/commons-pool-1.x 
-Dcommons-dbcp.home=/srv/gump/public/workspace/commons-dbcp-1.x 
-Dexecute.test.apr=false -Dexecute.test.bio=false 
-Dtomcat-dbcp.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-20150211.jar
 -Dtest.temp=output/test-tmp-NIO 
-Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-7.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-7.0.x/output/testclasses:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit4.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/servlet-api.ja
 

svn commit: r7986 - /dev/tomcat/taglibs/taglibs-standard-1.2.3/

2015-02-10 Thread jboynes
Author: jboynes
Date: Wed Feb 11 04:37:27 2015
New Revision: 7986

Log:
Stage files for taglibs 1.2.3 release vote

Added:
dev/tomcat/taglibs/taglibs-standard-1.2.3/
dev/tomcat/taglibs/taglibs-standard-1.2.3/NOTICE
dev/tomcat/taglibs/taglibs-standard-1.2.3/README_bin.txt
dev/tomcat/taglibs/taglibs-standard-1.2.3/README_src.txt

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-1.2.3-source-release.zip
   (with props)

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-1.2.3-source-release.zip.asc

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-1.2.3-source-release.zip.md5

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-1.2.3-source-release.zip.sha1
dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-compat-1.2.3.jar 
  (with props)

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-compat-1.2.3.jar.asc

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-compat-1.2.3.jar.md5

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-compat-1.2.3.jar.sha1
dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-impl-1.2.3.jar   
(with props)

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-impl-1.2.3.jar.asc

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-impl-1.2.3.jar.md5

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-impl-1.2.3.jar.sha1
dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-jstlel-1.2.3.jar 
  (with props)

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-jstlel-1.2.3.jar.asc

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-jstlel-1.2.3.jar.md5

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-jstlel-1.2.3.jar.sha1
dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-spec-1.2.3.jar   
(with props)

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-spec-1.2.3.jar.asc

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-spec-1.2.3.jar.md5

dev/tomcat/taglibs/taglibs-standard-1.2.3/taglibs-standard-spec-1.2.3.jar.sha1

Added: dev/tomcat/taglibs/taglibs-standard-1.2.3/NOTICE
==
--- dev/tomcat/taglibs/taglibs-standard-1.2.3/NOTICE (added)
+++ dev/tomcat/taglibs/taglibs-standard-1.2.3/NOTICE Wed Feb 11 04:37:27 2015
@@ -0,0 +1,5 @@
+Apache Standard Taglib
+Copyright 2001-2015 The Apache Software Foundation
+
+This product includes software developed at
+The Apache Software Foundation (http://www.apache.org/).

Added: dev/tomcat/taglibs/taglibs-standard-1.2.3/README_bin.txt
==
--- dev/tomcat/taglibs/taglibs-standard-1.2.3/README_bin.txt (added)
+++ dev/tomcat/taglibs/taglibs-standard-1.2.3/README_bin.txt Wed Feb 11 
04:37:27 2015
@@ -0,0 +1,111 @@
+---
+Apache Standard Tag Library 1.2.3 -- BINARY DISTRIBUTION
+---
+Thanks for downloading this release of the Standard tag library, 
+an implementation of the JavaServer Pages(tm)(JSP) 
+Standard Tag Library (JSTL).
+
+This code is licensed to you by the Apache Software Foundation and its
+contributors under the terms of the Apache License V2.0;
+please see the included NOTICE and LICENSE files for details.
+
+JSTL is an effort of the Java Community Process (JCP) and
+comes out of the JSR-052 expert group. For more information on JSTL,
+please go to http://jcp.org/en/jsr/detail?id=52 .
+
+---
+LIBRARY DEPENDENCIES
+
+This version of the Standard Tag Library has the following runtime
+dependencies:
+
+   1. Dependencies provided by a JSP 2.1 container:
+  - Java 1.5 or later
+  - Servlet 2.5 or later
+  - JSP 2.1 or later
+
+   2. Additional dependencies
+  - The XML tag library requires Apache Xalan 2.7.1 or later
+
+---
+Apache Xalan 2.7.1
+
+To address performance issues with XSLT processing, this version relies on
+implementation specific functionality from Apache Xalan. The following
+libraries should be included in the classpath for your application:
+   - xalan-2.7.1.jar
+   - serializer-2.7.1.jar
+
+---
+ADD DEPENDENCIES TO A WEB APPLICATION
+
+To use this distribution with your own web applications, add the following JAR
+files to the '/WEB-INF/lib' directory of your application:
+   - taglibs-standard-spec-1.2.3.jar
+   - taglibs-standard-impl-1.2.3.jar
+   - taglibs-standard-jstlel-1.2.3.jar
+   - xalan-2.7.1.jar
+   - serializer-2.7.1.jar
+
+If you do not use JSTL 1.0 tags then the taglibs-standard-jstlel JAR may be
+omitted. If you do not use the XML library, then the Apache Xalan dependencies
+may also be omitted.
+
+If you build you 

svn commit: r1658862 - /tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Wed Feb 11 01:51:30 2015
New Revision: 1658862

URL: http://svn.apache.org/r1658862
Log:
Fix trailing whitespace

Modified:
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml?rev=1658862r1=1658861r2=1658862view=diff
==
--- tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Wed Feb 11 01:51:30 2015
@@ -68,7 +68,7 @@
   fix
 bug57556/bug: Align codegetRealPath()/code behaviour with that
 of earlier versions and include a trailing separator if the real path
-refers to a directory. (markt) 
+refers to a directory. (markt)
   /fix
 /changelog
   /subsection



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



svn commit: r1658865 - in /tomcat/site/trunk: docs/sitemap-main.xml xdocs/sitemap-main.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Wed Feb 11 02:00:42 2015
New Revision: 1658865

URL: http://svn.apache.org/r1658865
Log:
Add oldnews-2014.html to sitemap file.

Modified:
tomcat/site/trunk/docs/sitemap-main.xml
tomcat/site/trunk/xdocs/sitemap-main.xml

Modified: tomcat/site/trunk/docs/sitemap-main.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/sitemap-main.xml?rev=1658865r1=1658864r2=1658865view=diff
==
--- tomcat/site/trunk/docs/sitemap-main.xml (original)
+++ tomcat/site/trunk/docs/sitemap-main.xml Wed Feb 11 02:00:42 2015
@@ -112,6 +112,11 @@
 priority0.5/priority
   /url
   url
+lochttp://tomcat.apache.org/oldnews-2014.html/loc
+changefreqmonthly/changefreq
+priority0.4/priority
+  /url
+  url
 lochttp://tomcat.apache.org/oldnews-2013.html/loc
 changefreqmonthly/changefreq
 priority0.4/priority

Modified: tomcat/site/trunk/xdocs/sitemap-main.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/sitemap-main.xml?rev=1658865r1=1658864r2=1658865view=diff
==
--- tomcat/site/trunk/xdocs/sitemap-main.xml (original)
+++ tomcat/site/trunk/xdocs/sitemap-main.xml Wed Feb 11 02:00:42 2015
@@ -112,6 +112,11 @@
 priority0.5/priority
   /url
   url
+lochttp://tomcat.apache.org/oldnews-2014.html/loc
+changefreqmonthly/changefreq
+priority0.4/priority
+  /url
+  url
 lochttp://tomcat.apache.org/oldnews-2013.html/loc
 changefreqmonthly/changefreq
 priority0.4/priority



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



[Bug 57564] New: Make JspC amenable to subclassing

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57564

Bug ID: 57564
   Summary: Make JspC amenable to subclassing
   Product: Tomcat 8
   Version: trunk
  Hardware: All
OS: All
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Jasper
  Assignee: dev@tomcat.apache.org
  Reporter: j...@mortbay.com

Created attachment 32451
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=32451action=edit
JspC proposed patch

In a similar vein to the changes to the JasperInitializer to facilitate
subclassing for different usage scenarios (eg use in different containers), it
would be great if there were a few minor tweaks to JspC that would allow it to
support these same usages.

Attached is a patch with the changes proposed.

regards
Jan

-- 
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: r1658853 - in /tomcat/site/trunk: docs/ci.html xdocs/ci.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Wed Feb 11 01:20:52 2015
New Revision: 1658853

URL: http://svn.apache.org/r1658853
Log:
I configured APR connector tests for Tomcat 7 at Gump. Add a link to them.

Modified:
tomcat/site/trunk/docs/ci.html
tomcat/site/trunk/xdocs/ci.xml

Modified: tomcat/site/trunk/docs/ci.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/ci.html?rev=1658853r1=1658852r2=1658853view=diff
==
--- tomcat/site/trunk/docs/ci.html (original)
+++ tomcat/site/trunk/docs/ci.html Wed Feb 11 01:20:52 2015
@@ -616,6 +616,10 @@ prepared and published by ASF Buildbot,
 /li
   
 li
+a 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-apr/;tomcat-tc7.0.x-test-apr/a
+/li
+  
+li
 a 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-bio/;tomcat-tc7.0.x-test-bio/a
 /li
   

Modified: tomcat/site/trunk/xdocs/ci.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/ci.xml?rev=1658853r1=1658852r2=1658853view=diff
==
--- tomcat/site/trunk/xdocs/ci.xml (original)
+++ tomcat/site/trunk/xdocs/ci.xml Wed Feb 11 01:20:52 2015
@@ -179,6 +179,7 @@ prepared and published by ASF Buildbot,
 ul
   lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x/;tomcat-tc7.0.x/a/li
   lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-dbcp/;tomcat-tc7.0.x-dbcp/a/li
+  lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-apr/;tomcat-tc7.0.x-test-apr/a/li
   lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-bio/;tomcat-tc7.0.x-test-bio/a/li
   lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-nio/;tomcat-tc7.0.x-test-nio/a/li
   lia 
href=http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-validate/;tomcat-tc7.0.x-validate/a/li



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



svn commit: r1658873 - in /tomcat/taglibs/standard/trunk: build-tools/pom.xml compat/pom.xml impl/pom.xml jstlel/pom.xml pom.xml spec/pom.xml

2015-02-10 Thread jboynes
Author: jboynes
Date: Wed Feb 11 04:17:26 2015
New Revision: 1658873

URL: http://svn.apache.org/r1658873
Log:
[maven-release-plugin] prepare for next development iteration

Modified:
tomcat/taglibs/standard/trunk/build-tools/pom.xml
tomcat/taglibs/standard/trunk/compat/pom.xml
tomcat/taglibs/standard/trunk/impl/pom.xml
tomcat/taglibs/standard/trunk/jstlel/pom.xml
tomcat/taglibs/standard/trunk/pom.xml
tomcat/taglibs/standard/trunk/spec/pom.xml

Modified: tomcat/taglibs/standard/trunk/build-tools/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/build-tools/pom.xml?rev=1658873r1=1658872r2=1658873view=diff
==
--- tomcat/taglibs/standard/trunk/build-tools/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/build-tools/pom.xml Wed Feb 11 04:17:26 2015
@@ -24,20 +24,20 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 /parent
 
 artifactIdtaglibs-build-tools/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 nameApache Standard Taglib Build Tools/name
 
 descriptionBuild tool settings common to all Standard Taglib 
modules/description
 
 scm
-
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools/connection
-
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools
+
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/build-tools/connection
+
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/build-tools
 /developerConnection
-
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools/url
+
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/build-tools/url
 /scm
 
 /project

Modified: tomcat/taglibs/standard/trunk/compat/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/compat/pom.xml?rev=1658873r1=1658872r2=1658873view=diff
==
--- tomcat/taglibs/standard/trunk/compat/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/compat/pom.xml Wed Feb 11 04:17:26 2015
@@ -24,11 +24,11 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 /parent
 
 artifactIdtaglibs-standard-compat/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 packagingbundle/packaging
 
 nameApache Standard Taglib 1.0 Compatibility/name
@@ -39,10 +39,10 @@
 /description
 
 scm
-
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat/connection
-
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat
+
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/compat/connection
+
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/compat
 /developerConnection
-
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat/url
+
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/compat/url
 /scm
 
 developers
@@ -69,12 +69,12 @@
 dependency
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard-spec/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 /dependency
 dependency
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard-impl/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 /dependency
 
 dependency

Modified: tomcat/taglibs/standard/trunk/impl/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/pom.xml?rev=1658873r1=1658872r2=1658873view=diff
==
--- tomcat/taglibs/standard/trunk/impl/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/impl/pom.xml Wed Feb 11 04:17:26 2015
@@ -24,11 +24,11 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 /parent
 
 artifactIdtaglibs-standard-impl/artifactId
-version1.2.3/version
+version1.2.4-SNAPSHOT/version
 packagingbundle/packaging
 
 nameApache Standard Taglib Implementation/name
@@ -39,10 +39,10 @@
 /description
 
 scm
-

svn commit: r1658871 - in /tomcat/taglibs/standard/trunk: build-tools/pom.xml compat/pom.xml impl/pom.xml jstlel/pom.xml pom.xml spec/pom.xml

2015-02-10 Thread jboynes
Author: jboynes
Date: Wed Feb 11 04:17:23 2015
New Revision: 1658871

URL: http://svn.apache.org/r1658871
Log:
[maven-release-plugin] prepare release taglibs-standard-1.2.3

Modified:
tomcat/taglibs/standard/trunk/build-tools/pom.xml
tomcat/taglibs/standard/trunk/compat/pom.xml
tomcat/taglibs/standard/trunk/impl/pom.xml
tomcat/taglibs/standard/trunk/jstlel/pom.xml
tomcat/taglibs/standard/trunk/pom.xml
tomcat/taglibs/standard/trunk/spec/pom.xml

Modified: tomcat/taglibs/standard/trunk/build-tools/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/build-tools/pom.xml?rev=1658871r1=1658870r2=1658871view=diff
==
--- tomcat/taglibs/standard/trunk/build-tools/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/build-tools/pom.xml Wed Feb 11 04:17:23 2015
@@ -24,20 +24,20 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 /parent
 
 artifactIdtaglibs-build-tools/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 nameApache Standard Taglib Build Tools/name
 
 descriptionBuild tool settings common to all Standard Taglib 
modules/description
 
 scm
-
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/build-tools/connection
-
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/build-tools
+
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools/connection
+
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools
 /developerConnection
-
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/build-tools/url
+
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/build-tools/url
 /scm
 
 /project

Modified: tomcat/taglibs/standard/trunk/compat/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/compat/pom.xml?rev=1658871r1=1658870r2=1658871view=diff
==
--- tomcat/taglibs/standard/trunk/compat/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/compat/pom.xml Wed Feb 11 04:17:23 2015
@@ -24,11 +24,11 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 /parent
 
 artifactIdtaglibs-standard-compat/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 packagingbundle/packaging
 
 nameApache Standard Taglib 1.0 Compatibility/name
@@ -39,10 +39,10 @@
 /description
 
 scm
-
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/compat/connection
-
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk/compat
+
connectionscm:svn:http://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat/connection
+
developerConnectionscm:svn:https://svn.apache.org/repos/asf/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat
 /developerConnection
-
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/compat/url
+
urlhttp://svn.apache.org/viewvc/tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/compat/url
 /scm
 
 developers
@@ -69,12 +69,12 @@
 dependency
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard-spec/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 /dependency
 dependency
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard-impl/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 /dependency
 
 dependency

Modified: tomcat/taglibs/standard/trunk/impl/pom.xml
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/pom.xml?rev=1658871r1=1658870r2=1658871view=diff
==
--- tomcat/taglibs/standard/trunk/impl/pom.xml (original)
+++ tomcat/taglibs/standard/trunk/impl/pom.xml Wed Feb 11 04:17:23 2015
@@ -24,11 +24,11 @@
 parent
 groupIdorg.apache.taglibs/groupId
 artifactIdtaglibs-standard/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 /parent
 
 artifactIdtaglibs-standard-impl/artifactId
-version1.2.3-SNAPSHOT/version
+version1.2.3/version
 packagingbundle/packaging
 
 nameApache Standard Taglib Implementation/name
@@ -39,10 +39,10 @@
 /description
 
 scm
-

svn commit: r1658872 - /tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/

2015-02-10 Thread jboynes
Author: jboynes
Date: Wed Feb 11 04:17:25 2015
New Revision: 1658872

URL: http://svn.apache.org/r1658872
Log:
[maven-release-plugin]  copy for tag taglibs-standard-1.2.3

Added:
tomcat/taglibs/standard/tags/taglibs-standard-1.2.3/
  - copied from r1658871, tomcat/taglibs/standard/trunk/


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



svn commit: r1658754 - in /tomcat/taglibs/standard/trunk: ./ impl/src/main/java/org/apache/taglibs/standard/util/ impl/src/test/java/org/apache/taglibs/standard/util/

2015-02-10 Thread jboynes
Author: jboynes
Date: Tue Feb 10 16:32:13 2015
New Revision: 1658754

URL: http://svn.apache.org/r1658754
Log:
Check protocol when resolving on older JREs

Added:

tomcat/taglibs/standard/trunk/impl/src/test/java/org/apache/taglibs/standard/util/XmlUtilTest.java
   (with props)
Modified:
tomcat/taglibs/standard/trunk/CHANGES.txt

tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java

tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java

Modified: tomcat/taglibs/standard/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/CHANGES.txt?rev=1658754r1=1658753r2=1658754view=diff
==
--- tomcat/taglibs/standard/trunk/CHANGES.txt (original)
+++ tomcat/taglibs/standard/trunk/CHANGES.txt Tue Feb 10 16:32:13 2015
@@ -1,8 +1,9 @@
 Changes in 1.2.3 release
 
-57547 Fix regression with running on older JREs
-57548 Update library version number printed by Version class
+57560 Check protocol when resolving on older JREs
 57549 Fix reference to old SNAPSHOT version of taglibs-build-tools in pom 
files.
+57548 Update library version number printed by Version class
+57547 Fix regression with running on older JREs
 
 Changes in 1.2.2 release [WITHDRAWN due to regressions in older JREs]
 

Modified: 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java?rev=1658754r1=1658753r2=1658754view=diff
==
--- 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
 (original)
+++ 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
 Tue Feb 10 16:32:13 2015
@@ -77,4 +77,19 @@ public class UrlUtil {
 // if so, we've got an absolute url
 return true;
 }
+
+public static String getScheme(CharSequence url) {
+StringBuilder scheme = new StringBuilder();
+for (int i = 0; i  url.length(); i++) {
+char ch = url.charAt(i);
+if (ch == ':') {
+String result = scheme.toString();
+if (!jar.equals(result)) {
+return result;
+}
+}
+scheme.append(ch);
+}
+throw new IllegalArgumentException(No scheme found:  + url);
+}
 }

Modified: 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java?rev=1658754r1=1658753r2=1658754view=diff
==
--- 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java
 (original)
+++ 
tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java
 Tue Feb 10 16:32:13 2015
@@ -20,6 +20,7 @@ import java.io.FileNotFoundException;
 import java.io.InputStream;
 import java.io.Reader;
 import java.security.AccessController;
+import java.security.PrivilegedAction;
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
 import java.util.concurrent.Callable;
@@ -111,6 +112,27 @@ public class XmlUtil {
 }
 }
 
+private static final String SP_ALLOWED_PROTOCOLS = 
org.apache.taglibs.standard.xml.accessExternalEntity;
+private static final String ALLOWED_PROTOCOLS = 
AccessController.doPrivileged(new PrivilegedActionString() {
+public String run() {
+String defaultProtocols = System.getSecurityManager() == null ? 
all : ;
+return System.getProperty(SP_ALLOWED_PROTOCOLS, defaultProtocols);
+}
+});
+
+static void checkProtocol(String allowedProtocols, String uri) {
+if (all.equalsIgnoreCase(allowedProtocols)) {
+return;
+}
+String protocol = UrlUtil.getScheme(uri);
+for (String allowed : allowedProtocols.split(,)) {
+if (allowed.trim().equalsIgnoreCase(protocol)) {
+return;
+}
+}
+throw new SecurityException(Access to external URI not allowed:  + 
uri);
+}
+
 /**
  * Create a new empty document.
  *
@@ -219,7 +241,7 @@ public class XmlUtil {
 }
 
 /**
- * JSTL-specific implementation of EntityResolver.
+ * JSTL-specific implementation of EntityResolver, used by parsers.
  */
 public static class JstlEntityResolver implements EntityResolver {
 private final PageContext ctx;
@@ -242,6 +264,7 @@ public class XmlUtil {
 
 // we're only concerned with relative URLs
  

svn commit: r1658771 - in /tomcat/trunk/java/org/apache/coyote: AbstractProcessor.java Processor.java ajp/AjpProcessor.java http11/Http11Processor.java http11/upgrade/UpgradeProcessor.java spdy/SpdyPr

2015-02-10 Thread remm
Author: remm
Date: Tue Feb 10 18:04:17 2015
New Revision: 1658771

URL: http://svn.apache.org/r1658771
Log:
Drop unused throws clause.

Modified:
tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
tomcat/trunk/java/org/apache/coyote/Processor.java
tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java
tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java
tomcat/trunk/java/org/apache/coyote/spdy/SpdyProcessor.java

Modified: tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/AbstractProcessor.java Tue Feb 10 
18:04:17 2015
@@ -212,7 +212,7 @@ public abstract class AbstractProcessor
  * upgrade.
  */
 @Override
-public abstract SocketState upgradeDispatch(SocketStatus status) throws 
IOException;
+public abstract SocketState upgradeDispatch(SocketStatus status);
 
 @Override
 public abstract HttpUpgradeHandler getHttpUpgradeHandler();

Modified: tomcat/trunk/java/org/apache/coyote/Processor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/Processor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/Processor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/Processor.java Tue Feb 10 18:04:17 2015
@@ -40,7 +40,7 @@ public interface Processor {
 SocketState asyncPostProcess();
 
 HttpUpgradeHandler getHttpUpgradeHandler();
-SocketState upgradeDispatch(SocketStatus status) throws IOException;
+SocketState upgradeDispatch(SocketStatus status);
 
 void errorDispatch();
 

Modified: tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java Tue Feb 10 
18:04:17 2015
@@ -846,9 +846,9 @@ public class AjpProcessor extends Abstra
 
 
 @Override
-public SocketState upgradeDispatch(SocketStatus status) throws IOException 
{
+public SocketState upgradeDispatch(SocketStatus status) {
 // Should never reach this code but in case we do...
-throw new IOException(
+throw new IllegalStateException(
 sm.getString(ajpprocessor.httpupgrade.notsupported));
 }
 

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Tue Feb 10 
18:04:17 2015
@@ -1817,7 +1817,7 @@ public class Http11Processor extends Abs
 
 
 @Override
-public SocketState upgradeDispatch(SocketStatus status) throws IOException 
{
+public SocketState upgradeDispatch(SocketStatus status) {
 // Should never reach this code but in case we do...
 throw new IllegalStateException(
 sm.getString(http11Processor.upgrade));

Modified: 
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
Tue Feb 10 18:04:17 2015
@@ -96,7 +96,7 @@ public class UpgradeProcessor implements
 
 
 @Override
-public final SocketState upgradeDispatch(SocketStatus status) throws 
IOException {
+public final SocketState upgradeDispatch(SocketStatus status) {
 if (status == SocketStatus.OPEN_READ) {
 try {
 upgradeServletInputStream.onDataAvailable();

Modified: tomcat/trunk/java/org/apache/coyote/spdy/SpdyProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/spdy/SpdyProcessor.java?rev=1658771r1=1658770r2=1658771view=diff
==
--- tomcat/trunk/java/org/apache/coyote/spdy/SpdyProcessor.java 

Re: svn commit: r1658754 - in /tomcat/taglibs/standard/trunk: ./ impl/src/main/java/org/apache/taglibs/standard/util/ impl/src/test/java/org/apache/taglibs/standard/util/

2015-02-10 Thread Konstantin Kolinko
2015-02-10 19:32 GMT+03:00  jboy...@apache.org:
 Author: jboynes
 Date: Tue Feb 10 16:32:13 2015
 New Revision: 1658754

 URL: http://svn.apache.org/r1658754
 Log:
 Check protocol when resolving on older JREs

 Added:
 
 tomcat/taglibs/standard/trunk/impl/src/test/java/org/apache/taglibs/standard/util/XmlUtilTest.java
(with props)
 Modified:
 tomcat/taglibs/standard/trunk/CHANGES.txt
 
 tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
 
 tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/XmlUtil.java


 Modified: 
 tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
 URL: 
 http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java?rev=1658754r1=1658753r2=1658754view=diff
 ==
 --- 
 tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
  (original)
 +++ 
 tomcat/taglibs/standard/trunk/impl/src/main/java/org/apache/taglibs/standard/util/UrlUtil.java
  Tue Feb 10 16:32:13 2015
 @@ -77,4 +77,19 @@ public class UrlUtil {
  // if so, we've got an absolute url
  return true;
  }
 +
 +public static String getScheme(CharSequence url) {

This method is only used with a String argument.

For a String it would be easier to use indexOf(':'),
though it does special processing for jar protocol.

 +StringBuilder scheme = new StringBuilder();
 +for (int i = 0; i  url.length(); i++) {
 +char ch = url.charAt(i);
 +if (ch == ':') {
 +String result = scheme.toString();
 +if (!jar.equals(result)) {
 +return result;
 +}
 +}
 +scheme.append(ch);
 +}
 +throw new IllegalArgumentException(No scheme found:  + url);
 +}
  }


Best regards,
Konstantin Kolinko

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



svn commit: r1658757 - in /tomcat/taglibs/standard/trunk: CHANGES.txt README_bin.txt

2015-02-10 Thread jboynes
Author: jboynes
Date: Tue Feb 10 16:43:59 2015
New Revision: 1658757

URL: http://svn.apache.org/r1658757
Log:
Update changes and docs

Modified:
tomcat/taglibs/standard/trunk/CHANGES.txt
tomcat/taglibs/standard/trunk/README_bin.txt

Modified: tomcat/taglibs/standard/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/CHANGES.txt?rev=1658757r1=1658756r2=1658757view=diff
==
--- tomcat/taglibs/standard/trunk/CHANGES.txt (original)
+++ tomcat/taglibs/standard/trunk/CHANGES.txt Tue Feb 10 16:43:59 2015
@@ -5,6 +5,11 @@ Changes in 1.2.3 release
 57548 Update library version number printed by Version class
 57547 Fix regression with running on older JREs
 
+XML tags now enable FEATURE_SECURE_PROCESSING when parsing and transforming. 
The JSTL-specific
+EntityResolver also checks the protocol being used against a white-list 
specified in the system
+property org.apache.taglibs.standard.xml.accessExternalEntity in to order to 
limit access on
+older JREs.
+
 Changes in 1.2.2 release [WITHDRAWN due to regressions in older JREs]
 
 57433 Double-check locking in ExpressionEvaluatorManager

Modified: tomcat/taglibs/standard/trunk/README_bin.txt
URL: 
http://svn.apache.org/viewvc/tomcat/taglibs/standard/trunk/README_bin.txt?rev=1658757r1=1658756r2=1658757view=diff
==
--- tomcat/taglibs/standard/trunk/README_bin.txt (original)
+++ tomcat/taglibs/standard/trunk/README_bin.txt Tue Feb 10 16:43:59 2015
@@ -88,9 +88,16 @@ The JSTL tag library can be imported int
 ---
 COMPATIBILITY
 
-The 1.2 version of the Standard Taglib has been tested under Tomcat 7.0.47
+The 1.2 version of the Standard Taglib has been tested using Tomcat 7.0.57
 and should work in any compliant JSP 2.1 (or later) container.
 
+In version 1.2.3 and later, the XML libraries enable FEATURE_SECURE_PROCESSING
+when parsing and transforming. A new system property
+  org.apache.taglibs.standard.xml.accessExternalEntity
+can be used to further restrict the protocols over which external entities can
+be resolved. When a SecurityManager is enabled this will, by default, allow
+access to no protocols.
+
 ---
 COMMENTS AND QUESTIONS
 



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



[Bug 57560] New: Check protocol when resolving on older JREs

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57560

Bug ID: 57560
   Summary: Check protocol when resolving on older JREs
   Product: Taglibs
   Version: 1.2.1
  Hardware: All
OS: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: Standard Taglib
  Assignee: dev@tomcat.apache.org
  Reporter: jboy...@apache.org

Add a mechanism to check the protocol being used when resolving entities on
older JREs.

-- 
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: r1658641 - /tomcat/native/trunk/native/src/sslnetwork.c

2015-02-10 Thread rjung
Author: rjung
Date: Tue Feb 10 08:53:09 2015
New Revision: 1658641

URL: http://svn.apache.org/r1658641
Log:
Replace access to OpenSSL SSL internals
by accessor function. Internals are no longer
available for OpenSSL master.

Not compiled or tested, waiting for GUMP.

Modified:
tomcat/native/trunk/native/src/sslnetwork.c

Modified: tomcat/native/trunk/native/src/sslnetwork.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/trunk/native/src/sslnetwork.c?rev=1658641r1=1658640r2=1658641view=diff
==
--- tomcat/native/trunk/native/src/sslnetwork.c (original)
+++ tomcat/native/trunk/native/src/sslnetwork.c Tue Feb 10 08:53:09 2015
@@ -408,7 +408,7 @@ ssl_socket_recv(apr_socket_t *sock, char
 
 i = SSL_get_error(con-ssl, s);
 /* Special case if the close notify alert send by peer */
-if (s == 0  (con-ssl-shutdown  SSL_RECEIVED_SHUTDOWN)) {
+if (s == 0  (SSL_get_shutdown(con-ssl)  
SSL_RECEIVED_SHUTDOWN)) {
 *len = 0;
 return APR_EOF;
 }
@@ -633,7 +633,7 @@ TCN_IMPLEMENT_CALL(jint, SSLSocket, rene
 if (SSL_get_state(con-ssl) != SSL_ST_OK) {
 return APR_EGENERAL;
 }
-con-ssl-state = SSL_ST_ACCEPT;
+SSL_set_state(con-ssl, SSL_ST_ACCEPT);
 
 ecode = SSL_ERROR_WANT_READ;
 while (ecode == SSL_ERROR_WANT_READ) {



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



[Bug 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

--- Comment #3 from Mark Thomas ma...@apache.org ---
(In reply to Paul Gaylie from comment #2)
 Mark,
 
 Unfortunately, killing a browser is not a good simulation because the
 browser is smart enough to properly close the websocket when it gets killed.

With kill -9 the browser nver gets the chance. The close isn't clean - a bunch
of IOExceptions are triggered.

 Here is what to do: connect using a mobile browser on your phone (like
 Chrome on Android) and then shutoff the mobile data connection on your
 phone. Then you should see the doClose() operation happening on the server
 side.

I'll try but right now my money is on this already being fixed in the post
8.0.18 changes.

-- 
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



Re: Hi

2015-02-10 Thread Mark Thomas
On 10/02/2015 06:28, Pravallika Peddi wrote:
 Thanks Chris for the information.
 
 Mark  Chris:
 I have gone through one of the issues given by Mark:
 
 https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
 
 and found that this issue is fixed already.
 
 Steps i performed: 
 1) Download Tomcat 6.0 - 32 bit version and run it on 32-bit Windows
 platform. Tomcat Server started successfully using commands mentioned in
 issue. Stop the tomcat. Check the catalina.out folder and it has no
 error messages. Rename catalina.out file.
 
 2) Download Tomcat 6.0-64 bit version and extract tcnative-dll and copy
 it on to 32-bit Tomcat folder. Start the Tomcat Server and check the
 catalina.out file. It has proper error message Feb 10, 2015 11:42:54 AM
 org.apache.catalina.core.AprLifecycleListener init
 INFO: The APR based Apache Tomcat Native library which allows optimal
 performance in production environments was not found on the
 java.library.path:
 
 and i did not observe any error in command line as mentioned in the issue.
 
 I attached both the catalina.out files for your reference.
 Even i checked the recent Trunk AppLifecycleListener.init() java method
 and did not find the source code that was mentioned in the issue.
 
 Hence i feel the issue is fixed.
 Please confirm so that i will close the issue.

The issue is not fixed. Tomcat isn't finding the tc-native library so
the error isn't triggering.

Mark

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



[Bug 57554] Exception loading sessions from persistent storage

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57554

Rainer Jung rainer.j...@kippdata.de changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |INVALID

-- 
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: r1658646 - in /tomcat/native/branches/1.1.x: ./ native/src/sslnetwork.c

2015-02-10 Thread rjung
Author: rjung
Date: Tue Feb 10 09:00:11 2015
New Revision: 1658646

URL: http://svn.apache.org/r1658646
Log:
Replace access to OpenSSL SSL internals
by accessor function. Internals are no longer
available for OpenSSL master.

Not compiled or tested, waiting for GUMP.

Backport of r1658641 from trunk.

Modified:
tomcat/native/branches/1.1.x/   (props changed)
tomcat/native/branches/1.1.x/native/src/sslnetwork.c

Propchange: tomcat/native/branches/1.1.x/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 09:00:11 2015
@@ -1,3 +1,3 @@
-/tomcat/native/trunk:815411,1342003,1342008,1342013,1342020,1342024,1394258,1394342,1424947,1424971,1430753,1437081,1438342,1439337,1441884,1441886,1442579,1442581,1445972,1507113,1532577,1532590,1539594,1555184,1559180,1588195,1607262,1607267,1607278,1607291,1607477,1648821,1650119,1658557
+/tomcat/native/trunk:815411,1342003,1342008,1342013,1342020,1342024,1394258,1394342,1424947,1424971,1430753,1437081,1438342,1439337,1441884,1441886,1442579,1442581,1445972,1507113,1532577,1532590,1539594,1555184,1559180,1588195,1607262,1607267,1607278,1607291,1607477,1648821,1650119,1658557,1658641-1658642
 /tomcat/tc7.0.x/trunk:1199985,1200164,1349932,1434887,1435769
 
/tomcat/trunk:815418,832198,1001939,1033916,1043103,1044729,1078522,1145209,1145285,1149092,1241356,1241406-1241407,1242254,1292671,1299980,1300102,1434905,1437083

Modified: tomcat/native/branches/1.1.x/native/src/sslnetwork.c
URL: 
http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/src/sslnetwork.c?rev=1658646r1=1658645r2=1658646view=diff
==
--- tomcat/native/branches/1.1.x/native/src/sslnetwork.c (original)
+++ tomcat/native/branches/1.1.x/native/src/sslnetwork.c Tue Feb 10 09:00:11 
2015
@@ -410,7 +410,7 @@ ssl_socket_recv(apr_socket_t *sock, char
 rv  = apr_get_netos_error();
 i   = SSL_get_error(con-ssl, s);
 /* Special case if the close notify alert send by peer */
-if (s == 0  (con-ssl-shutdown  SSL_RECEIVED_SHUTDOWN)) {
+if (s == 0  (SSL_get_shutdown(con-ssl)  
SSL_RECEIVED_SHUTDOWN)) {
 con-shutdown_type = SSL_SHUTDOWN_TYPE_STANDARD;
 return APR_EOF;
 }
@@ -647,7 +647,7 @@ TCN_IMPLEMENT_CALL(jint, SSLSocket, rene
 if (SSL_get_state(con-ssl) != SSL_ST_OK) {
 return APR_EGENERAL;
 }
-con-ssl-state = SSL_ST_ACCEPT;
+SSL_set_state(con-ssl, SSL_ST_ACCEPT);
 
 apr_socket_timeout_get(con-sock, timeout);
 ecode = SSL_ERROR_WANT_READ;



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



Re: svn commit: r1658557 - /tomcat/native/trunk/native/src/sslinfo.c

2015-02-10 Thread Rainer Jung

Am 10.02.2015 um 00:18 schrieb Konstantin Kolinko:

2015-02-09 22:52 GMT+03:00  rj...@apache.org:

Author: rjung
Date: Mon Feb  9 19:52:36 2015
New Revision: 1658557

URL: http://svn.apache.org/r1658557
Log:
Replace access to OpenSSL session internals
by accessor function. Internals are no longer
available for OpenSSL master.

Not compiled or tested, waiting for GUMP.

Modified:
 tomcat/native/trunk/native/src/sslinfo.c


This change is in native trunk, but Gump builds the 1.1.x branch...

In project/tomcat-native.xml of Gump:
   svn repository=asf dir=tomcat/native/branches/1.1.x/


Thank you Konstantin, also for merging to 1.1. I thought if it is 
building against OpenSSL master branch it must be our trunk, but well ...


I hope I fixed the next file and there's not much SSL files left, so 
maybe not too many further iterations needed.


Regards,

Rainer

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



[Bug 57554] Exception loading sessions from persistent storage

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57554

--- Comment #1 from Rainer Jung rainer.j...@kippdata.de ---
Comment on attachment 32445
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=32445
catalina output

I don't think that it is related to the session ID generator change.
It doesn't look so to me.

Please post your problem to the users mailing list. It doesn't look like a TC
problem even though you experienced it after update. If discussion on the users
list shows evidence, that it is indeed a Tomcat problem, you can reopen this
ticket.

Regards,

Rainer

-- 
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 57556] New: getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

Bug ID: 57556
   Summary: getServletContext().getRealPath(/) returns path not
ending with /
   Product: Tomcat 8
   Version: trunk
  Hardware: PC
Status: NEW
  Severity: major
  Priority: P2
 Component: Specification APIs
  Assignee: dev@tomcat.apache.org
  Reporter: ps.kart...@gmail.com

I have been using this getServletContext().getRealPath(/) till Tomcat6 and
i got the return string path ending with slash.
For Example getServletContext().getRealPath(/) will return
E:/TomcatHome/webapps/CRM/

But when I tried with Tomcat 8.0.14 I got path without slash
For Example getServletContext().getRealPath(/) will return
E:/TomcatHome/webapps/CRM
Ending slash missing

Is there anything changed in Tomcat 8 and is there any configuration I can make
to get the result that I got in previous version of Tomcat, because if I have
to change in code, I have to change in lot of places in all the webapps I
created.

-- 
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 56085] IllegalArgumentException is thrown when calling getServletContext().getRealPath(\\)

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56085

karthic ps.kart...@gmail.com changed:

   What|Removed |Added

Version|8.0.0-RC10  |8.0.14

-- 
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 57556] getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

karthic ps.kart...@gmail.com changed:

   What|Removed |Added

 OS||All

-- 
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



async and thread constraint

2015-02-10 Thread Romain Manni-Bucau
Hi guys,

in org.apache.coyote.AsyncStateMachine#asyncDispatch tomcat checks
ContainerThreadMarker.isContainerThread()

basically it prevents to use JAXRS 2 @Suspended without creating
custom threads - which is not intended.

Was this limitation intended or just to try to avoid to consume
container threads?

PS: there is no log saying dispatching is skipped so it makes it very
hard to understand without opening an IDE and tomcat sources, could it
be enhanced please?


Romain Manni-Bucau
@rmannibucau
http://www.tomitribe.com
http://rmannibucau.wordpress.com
https://github.com/rmannibucau

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



Re: Hi

2015-02-10 Thread Pravallika Peddi
Got you Mark.
Now i am able to reproduce the problem by keeping I64 bit dll in x64 bit
windows platform.
I will fix it and let you know.

Thanks,
VIN

On Tue, Feb 10, 2015 at 2:19 PM, Mark Thomas ma...@apache.org wrote:

 On 10/02/2015 06:28, Pravallika Peddi wrote:
  Thanks Chris for the information.
 
  Mark  Chris:
  I have gone through one of the issues given by Mark:
 
  https://issues.apache.org/bugzilla/show_bug.cgi?id=57021
 
  and found that this issue is fixed already.
 
  Steps i performed:
  1) Download Tomcat 6.0 - 32 bit version and run it on 32-bit Windows
  platform. Tomcat Server started successfully using commands mentioned in
  issue. Stop the tomcat. Check the catalina.out folder and it has no
  error messages. Rename catalina.out file.
 
  2) Download Tomcat 6.0-64 bit version and extract tcnative-dll and copy
  it on to 32-bit Tomcat folder. Start the Tomcat Server and check the
  catalina.out file. It has proper error message Feb 10, 2015 11:42:54 AM
  org.apache.catalina.core.AprLifecycleListener init
  INFO: The APR based Apache Tomcat Native library which allows optimal
  performance in production environments was not found on the
  java.library.path:
 
  and i did not observe any error in command line as mentioned in the
 issue.
 
  I attached both the catalina.out files for your reference.
  Even i checked the recent Trunk AppLifecycleListener.init() java method
  and did not find the source code that was mentioned in the issue.
 
  Hence i feel the issue is fixed.
  Please confirm so that i will close the issue.

 The issue is not fixed. Tomcat isn't finding the tc-native library so
 the error isn't triggering.

 Mark

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




[Bug 57556] getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

--- Comment #1 from karthic ps.kart...@gmail.com ---
I mentioned I used till Tomcat6, it is typo, I have been using this code from
Tomcat5.5 to Tomcat7

-- 
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 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #18 from Remy Maucherat r...@apache.org ---
Adding your dummy classes means wrapping is supported and, probably, that there
is more API stability that what is actually the case. I continue to believe
that this is not a good idea. If people, like you, want to hack, feel free, but
it would be better to be able to disregard side effects and problems that could
be introduced. Especially if it is only hacking to be able to do useless
debugging.

Unless my veto is found invalid, I think this should be resolved as wontfix.

-- 
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 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #19 from mr.oliver.hernan...@gmail.com ---
Remy, please re-read my previous comments
(https://issues.apache.org/bugzilla/show_bug.cgi?id=45014#c5 
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014#c6).

This need is far from useless.  We're seeing these errors,

Caused by: java.io.IOException: Strange I/O stream, returned 0 bytes on read
at com.ctc.wstx.io.BaseReader.reportStrangeStream(BaseReader.java:216)
at com.ctc.wstx.io.UTF8Reader.loadMore(UTF8Reader.java:375)
at com.ctc.wstx.io.UTF8Reader.read(UTF8Reader.java:111)
at
com.ctc.wstx.io.ReaderBootstrapper.initialLoad(ReaderBootstrapper.java:250)
at
com.ctc.wstx.io.ReaderBootstrapper.bootstrapInput(ReaderBootstrapper.java:133)
at
com.ctc.wstx.stax.WstxInputFactory.doCreateSR(WstxInputFactory.java:546)
... 36 more

and the valve I put together has been a tremendous help in troubleshooting it.

Instead of this issue being categorized as a bug, maybe this should be
considered a feature request.

-- 
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 56085] IllegalArgumentException is thrown when calling getServletContext().getRealPath(\\)

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56085

Konstantin Kolinko knst.koli...@gmail.com changed:

   What|Removed |Added

Version|8.0.14  |8.0.0-RC10

--- Comment #3 from Konstantin Kolinko knst.koli...@gmail.com ---
Restore Version field back to 8.0.0-RC10

This issue was reported in 8.0-RC10 and fixed since 8.0.2.

-- 
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 57556] getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

Konstantin Kolinko knst.koli...@gmail.com changed:

   What|Removed |Added

Version|trunk   |8.0.14

-- 
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



Re: svn commit: r1658734 - in /tomcat/trunk/java/org/apache/coyote/http11/upgrade: LocalStrings.properties UpgradeProcessor.java

2015-02-10 Thread Mark Thomas
On 10/02/2015 15:59, Rémy Maucherat wrote:
 2015-02-10 16:43 GMT+01:00 ma...@apache.org:
 
 Author: markt
 Date: Tue Feb 10 15:43:46 2015
 New Revision: 1658734

 URL: http://svn.apache.org/r1658734
 Log:
 Ensure that a dropped connection does not leave references to the
 UpgradeProcessor

 Good find, but what happens if onDataAvailable or onWritePossible throw a
 runtime exception (like a NPE), since this is user code. It could also leak
 then ? Shouldn't the code catch everything, also call onError on the
 listener and close ?

Good point. I was thinking purely in terms of WebSocket but you are
right - this needs to be more general. I'll take another look.

Mark


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



svn commit: r1658777 - in /tomcat/site/trunk: docs/migration-8.html xdocs/migration-8.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 18:40:52 2015
New Revision: 1658777

URL: http://svn.apache.org/r1658777
Log:
Enhance wording and add a link to Apache Commons DBCP project

Modified:
tomcat/site/trunk/docs/migration-8.html
tomcat/site/trunk/xdocs/migration-8.xml

Modified: tomcat/site/trunk/docs/migration-8.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/migration-8.html?rev=1658777r1=1658776r2=1658777view=diff
==
--- tomcat/site/trunk/docs/migration-8.html (original)
+++ tomcat/site/trunk/docs/migration-8.html Tue Feb 10 18:40:52 2015
@@ -429,15 +429,21 @@ of Apache Tomcat./p
 div class=text
 
 
-pTomcat 8 embeds a packaged renamed version of Commons DBCP 2.x. There are
-a number of notable changes between Commons DBCP 1.x and Commons DBCP 2.x
+pTomcat 8 embeds a packaged renamed version of
+a href=http://commons.apache.org/proper/commons-dbcp/;Apache Commons 
DBCP/a
+2.x. There are a number of notable changes between Commons DBCP 1.x
+(used by Tomcat 7 and earlier) and Commons DBCP 2.x
 which are likely to require configuration changes./p
 
 ul
   
-liThe maxActive configuration option has been renamed to maxTotal/li
+liThe codemaxActive/code configuration option has been renamed
+  to codemaxTotal/code
+/li
   
-liThe maxWait configuration option has been renamed to maxWaitMillis/li
+liThe codemaxWait/code configuration option has been renamed to
+  codemaxWaitMillis/code
+/li
   
 liThe JDBC driver JAR may be placed in WEB-INF/lib as an alternative to
   $CATALINA_BASE/lib provided that the driver class is only used by 
that

Modified: tomcat/site/trunk/xdocs/migration-8.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/migration-8.xml?rev=1658777r1=1658776r2=1658777view=diff
==
--- tomcat/site/trunk/xdocs/migration-8.xml (original)
+++ tomcat/site/trunk/xdocs/migration-8.xml Tue Feb 10 18:40:52 2015
@@ -130,12 +130,16 @@ of Apache Tomcat./p
 
   subsection name=Database Connection Pooling
 
-pTomcat 8 embeds a packaged renamed version of Commons DBCP 2.x. There 
are
-a number of notable changes between Commons DBCP 1.x and Commons DBCP 2.x
+pTomcat 8 embeds a packaged renamed version of
+a href=http://commons.apache.org/proper/commons-dbcp/;Apache Commons 
DBCP/a
+2.x. There are a number of notable changes between Commons DBCP 1.x
+(used by Tomcat 7 and earlier) and Commons DBCP 2.x
 which are likely to require configuration changes./p
 ul
-  liThe maxActive configuration option has been renamed to maxTotal/li
-  liThe maxWait configuration option has been renamed to 
maxWaitMillis/li
+  liThe codemaxActive/code configuration option has been renamed
+  to codemaxTotal/code/li
+  liThe codemaxWait/code configuration option has been renamed to
+  codemaxWaitMillis/code/li
   liThe JDBC driver JAR may be placed in WEB-INF/lib as an alternative to
   $CATALINA_BASE/lib provided that the driver class is only used by 
that
   web application./li



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



[Bug 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #21 from Christopher Schultz ch...@christopherschultz.net ---
(In reply to Remy Maucherat from comment #20)
 Yes, I did note that I guess in hindsight I could have implemented a
 filter. This is a feature for hackers, my vote is to let people deal with
 it to avoid having any additional support burden.

I'm happy to simply leave the classes as attachments to this enhancement
request and not commit anything. I don't personally have a use for them at this
moment. A few years ago it occurred to me that they would be something useful
for Valve authors. Since Valve is an internal API, Valve authors should already
be comfortable with an unstable API.

We'll see if this gets any votes. It's remained untouched for 6 years until
this week. It's clear there is not much demand that the examples here will not
fill.

-- 
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



Re: Consider support for the Servlet profile of JSR 196 (JASPIC) in Tomcat 7.0.x

2015-02-10 Thread Fjodor Vershinin
Hello!
I am CS student and it looks like that this task is quite interesting. I
would take it for GSOC if ASF organization will be selected. Currently I
have some time to do research in Tomcat codebase. Could you provide me some
entry points?
Thanks,
Fjodor



--
View this message in context: 
http://tomcat.10.x6.nabble.com/Consider-support-for-the-Servlet-profile-of-JSR-196-JASPIC-in-Tomcat-7-0-x-tp4993387p5029627.html
Sent from the Tomcat - Dev mailing list archive at Nabble.com.

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



svn commit: r1658779 - in /tomcat/site/trunk: docs/migration-8.html xdocs/migration-8.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 18:55:04 2015
New Revision: 1658779

URL: http://svn.apache.org/r1658779
Log:
Enhance wording and add a link to jndi-datasource-examples-howto.html

Modified:
tomcat/site/trunk/docs/migration-8.html
tomcat/site/trunk/xdocs/migration-8.xml

Modified: tomcat/site/trunk/docs/migration-8.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/migration-8.html?rev=1658779r1=1658778r2=1658779view=diff
==
--- tomcat/site/trunk/docs/migration-8.html (original)
+++ tomcat/site/trunk/docs/migration-8.html Tue Feb 10 18:55:04 2015
@@ -431,7 +431,9 @@ of Apache Tomcat./p
 
 pTomcat 8 embeds a packaged renamed version of
 a href=http://commons.apache.org/proper/commons-dbcp/;Apache Commons 
DBCP/a
-2.x. There are a number of notable changes between Commons DBCP 1.x
+2.x as the default implementation of a
+a href=tomcat-8.0-doc/jndi-datasource-examples-howto.htmldatabase 
connection pool/a.
+There are a number of notable changes between Commons DBCP 1.x
 (used by Tomcat 7 and earlier) and Commons DBCP 2.x
 which are likely to require configuration changes./p
 

Modified: tomcat/site/trunk/xdocs/migration-8.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/migration-8.xml?rev=1658779r1=1658778r2=1658779view=diff
==
--- tomcat/site/trunk/xdocs/migration-8.xml (original)
+++ tomcat/site/trunk/xdocs/migration-8.xml Tue Feb 10 18:55:04 2015
@@ -132,7 +132,9 @@ of Apache Tomcat./p
 
 pTomcat 8 embeds a packaged renamed version of
 a href=http://commons.apache.org/proper/commons-dbcp/;Apache Commons 
DBCP/a
-2.x. There are a number of notable changes between Commons DBCP 1.x
+2.x as the default implementation of a
+a href=tomcat-8.0-doc/jndi-datasource-examples-howto.htmldatabase 
connection pool/a.
+There are a number of notable changes between Commons DBCP 1.x
 (used by Tomcat 7 and earlier) and Commons DBCP 2.x
 which are likely to require configuration changes./p
 ul



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



[Bug 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #22 from Christopher Schultz ch...@christopherschultz.net ---
(In reply to Remy Maucherat from comment #18)
 Adding your dummy classes means wrapping is supported and, probably, that
 there is more API stability that what is actually the case.

This is the first time you've mentioned API stability which, honestly, is a
real concern.

Here are the stats for changes to o.a.coyote.Request:

- Tomcat 9, root to tip: no changes
- Tomcat 8, root (r1636341) to tip: no changes
- Tomcat 7, root (r1155369) to tip: 5 commits: r1446110, r1446138, r1446138,
*r156*, r1561675, r1518197. The single *starred* commit changed the API.

This API is pretty stable, even if it is not officially so.

-- 
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: r1658781 - /tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 19:04:01 2015
New Revision: 1658781

URL: http://svn.apache.org/r1658781
Log:
Add note about DBCP changes between Tomcat 7 and 8,
replacing an old note that was saying about Tomcat 5.0 and 5.5

Modified:
tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml

Modified: tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml?rev=1658781r1=1658780r2=1658781view=diff
==
--- tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml (original)
+++ tomcat/trunk/webapps/docs/jndi-datasource-examples-howto.xml Tue Feb 10 
19:04:01 2015
@@ -52,9 +52,12 @@ to the wider audience, or if you feel we
 
 p
 bPlease note that JNDI resource configuration changed somewhat between
-Tomcat 5.0.x and Tomcat 5.5.x./b  You will most likely need to modify older
+Tomcat 7.x and Tomcat 8.x as they are using different versions of
+Apache Commons DBCP library./b  You will most likely need to modify older
 JNDI resource configurations to match the syntax in the example below in order
 to make them work in Tomcat version-major/.
+See a href=http://tomcat.apache.org/migration.html;Tomcat Migration 
Guide/a
+for details.
 /p
 
 p



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



[Bug 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #23 from Christopher Schultz ch...@christopherschultz.net ---
(In reply to Christopher Schultz from comment #22)
 - Tomcat 7, root (r1155369) to tip: 5 commits: r1446110, r1446138, r1446138,
 *r156*, r1561675, r1518197. The single *starred* commit changed the API.

This incorrectly identifies the commit which included the API change. The
correct commit is r1518197 for the API change.

-- 
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: r1658783 - in /tomcat/tc8.0.x/trunk: ./ webapps/docs/jndi-datasource-examples-howto.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 19:09:16 2015
New Revision: 1658783

URL: http://svn.apache.org/r1658783
Log:
Add note about DBCP changes between Tomcat 7 and 8,
replacing an old note that was saying about Tomcat 5.0 and 5.5

Merged r1658781 from tomcat/trunk.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/webapps/docs/jndi-datasource-examples-howto.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 19:09:16 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781

Modified: tomcat/tc8.0.x/trunk/webapps/docs/jndi-datasource-examples-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/jndi-datasource-examples-howto.xml?rev=1658783r1=1658782r2=1658783view=diff
==
--- 

Re: Consider support for the Servlet profile of JSR 196 (JASPIC) in Tomcat 7.0.x

2015-02-10 Thread Arjan Tijms
Hi,

On Tuesday, February 10, 2015, Fjodor Vershinin [via Tomcat] 
ml-node+s10n5029627...@n6.nabble.com wrote:

 Hello!
 I am CS student and it looks like that this task is quite interesting. I
 would take it for GSOC if ASF organization will be selected. Currently I
 have some time to do research in Tomcat codebase. Could you provide me some
 entry points?


Thanks for your interest in this. An entry point could be my original
JASPIC article that you can find here;
http://arjan-tijms.omnifaces.org/2012/11/implementing-container-authentication.html

At the end of the article you'll find a list of resources.

My approach would be to investigate how Tomcat integrates authentication
modules, eg look at the source of the JAAS support in Tomcat; that code has
to do similar integration. You can look at JBoss 7.x for an example too, it
used Tomcat and an integration Valve (WebJaspiAuthenticator seehttp://
grepcode.com/file/repository.jboss.org/nexus/content/repositories/releases/org.jboss.as/jboss-as-web/7.1.1.Final/org/jboss/as/web/security/jaspi/WebJASPIAuthenticator.java
)

Geronimo also implemented JASPIC and used Tomcat, so that implementation
would be high on the list to study too.

Many implementations have a (large) part of their code dedicated to
handling some xml file where jaspic auth modules are defined. Strictly
speaking this is not a required part of JASPIC, but it's somewhat expected
for configuring modules at the container side (as apposed to from within
the app archive).

I did actually more or less promise to do this implementation myself, but
so far havent found the time for it.

Let me know if this is enough to get started.

Kind regards,
Arjan


 Thanks,
 Fjodor

 --
  If you reply to this email, your message will be added to the discussion
 below:

 http://tomcat.10.x6.nabble.com/Consider-support-for-the-Servlet-profile-of-JSR-196-JASPIC-in-Tomcat-7-0-x-tp4993387p5029627.html
  To unsubscribe from Consider support for the Servlet profile of JSR 196
 (JASPIC) in Tomcat 7.0.x, click here
 http://tomcat.10.x6.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_codenode=4993387code=YXJqYW4udGlqbXNAZ21haWwuY29tfDQ5OTMzODd8LTM3MzU5NTg0OA==
 .
 NAML
 http://tomcat.10.x6.nabble.com/template/NamlServlet.jtp?macro=macro_viewerid=instant_html%21nabble%3Aemail.namlbase=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespacebreadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml





--
View this message in context: 
http://tomcat.10.x6.nabble.com/Consider-support-for-the-Servlet-profile-of-JSR-196-JASPIC-in-Tomcat-7-0-x-tp4993387p5029633.html
Sent from the Tomcat - Dev mailing list archive at Nabble.com.

[Bug 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

--- Comment #4 from Mark Thomas ma...@apache.org ---
Interesting. I tried testing with a separate machine so I could pull out the
network cable. That has triggered a problem that looks like the one described
here. I'm looking into this at the moment.

Musing on why I didn't see this when I kill -9'd the browser, I suspect that
the client OS was doing some clean-up since it was still connected.

-- 
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 57558] New: validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

Bug ID: 57558
   Summary: validator ant task throws NoClassDefFoundError -
Tomcat 8.0.18
   Product: Tomcat 8
   Version: 8.0.18
  Hardware: PC
Status: NEW
  Severity: normal
  Priority: P2
 Component: Jasper
  Assignee: dev@tomcat.apache.org
  Reporter: knst.koli...@gmail.com

This has been reported on the users list

http://mail-archives.apache.org/mod_mbox/tomcat-users/201502.mbox/%3CCAHvOsHzPt9hK8iGSTfXCzCcUSMq4N0DH5o1WRzTFvW-pKdVRRg%40mail.gmail.com%3E
http://tomcat.markmail.org/thread/du5j6z54cw2pfdvz

quote
I'm using Tomcat 8.0.18.
I've following ant build.xml:

import file=${tomcat.home}/bin/
catalina-tasks.xml/

   target name=jspc

jasper
 validateXml=false
 uriroot=${webapp.path}
 webXmlFragment=${webapp.path}/WEB-INF/generated_web.xml
 outputDir=${webapp.path}/WEB-INF/src /

  validator path=${webapp.path} /
...
  /target

jasper task works great, but validator task throws
java.lang.NoClassDefFoundError: org/apache/catalina/Globals

/quote


Searching the jars, org/apache/catalina/Globals.class is in catalina.jar,
which is not listed in bin/catalina-tasks.xml

I think that catalina-tasks.xml shall list all jars from
${catalina.home}/lib/*.jar, instead of naming just a few of those.  Besides
catalina.jar others that are missing are *-i18n-*.jar ones.

-- 
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 57556] getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

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

--- Comment #4 from Mark Thomas ma...@apache.org ---
Fixed in trunk for 9.0.x and in 8.0.x for 8.0.19 onwards.

-- 
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: r1658811 - /tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 21:12:23 2015
New Revision: 1658811

URL: http://svn.apache.org/r1658811
Log:
Correct a typo

Modified:
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml?rev=1658811r1=1658810r2=1658811view=diff
==
--- tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Tue Feb 10 21:12:23 2015
@@ -170,8 +170,8 @@
 (markt)
   /fix
   fix
-bug57558/bug: Add missing JAR in Ant task definition rquired by the
-validate task. (markt)
+bug57558/bug: Add missing JAR in Ant task definition required by
+the validate task. (markt)
   /fix
 /changelog
   /subsection



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



svn commit: r1658815 - in /tomcat/tc7.0.x/trunk: ./ bin/catalina-tasks.xml webapps/docs/changelog.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 21:22:44 2015
New Revision: 1658815

URL: http://svn.apache.org/r1658815
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
Add missing JAR in Ant task definition required by the validate task.

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/bin/catalina-tasks.xml
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 21:22:44 2015
@@ -1,2 +1,2 @@
-/tomcat/tc8.0.x/trunk:1636525,1637336,1637685,1637709,1638726,1640089,1640276,1640349,1640363,1640366,1640642,1640672,1640674,1640689,1640884,1641001,1641065,1641067,1641375,1641638,1641723,1641726,1641729-1641730,1641736,1641988,1642669-1642670,1642698,1642701,1643205,1643215,1643217,1643230,1643232,1643273,1643285,1643329-1643330,1643511,1643513,1643521,1643539,1643571,1643581-1643582,1643635,1643655,1643738,1643964,1644018,1644333,1644954,1644992,1645014,1645360,1645456,1645627,1645642,1645686,1645903-1645904,1645908-1645909,1645913,1645920,1646458,1646460-1646462,1647030,1647043,1648816,1651420-1651422,1651844,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1655109,1655112,1655114-1655115,1655117,1655120,1655445,1655467,1656300,1657049,1657356,1657360,1657370,1657460,1657908,1658208,1658737
-/tomcat/trunk:1156115-1157160,1157162-1157859,1157862-1157942,1157945-1160347,1160349-1163716,1163718-1166689,1166691-1174340,1174342-1175596,1175598-1175611,1175613-1175932,1175934-1177783,1177785-1177980,1178006-1180720,1180722-1183094,1183096-1187753,1187755,1187775,1187801,1187806,1187809,1187826-1188312,1188314-1188401,1188646-1188840,1188842-1190176,1190178-1195223,1195225-1195953,1195955,1195957-1201238,1201240-1203345,1203347-1206623,1206625-1208046,1208073,1208096,1208114,1208145,1208772,1209194-1212125,1212127-1220291,1220293,1220295-1221321,1221323-1222329,1222332-1222401,1222405-1222795,1222850-1222950,1222969-1225326,1225328-1225463,1225465,1225627,1225629-1226534,1226536-1228908,1228911-1228923,1228927-1229532,1229534-1230766,1230768-1231625,1231627-1233414,1233419-1235207,1235209-1237425,1237427,1237429-1237977,1237981,1237985,1237995,1238070,1238073,1239024-1239048,1239050-1239062,1239135,1239256,1239258-1239485,1239785-1240046,1240101,1240106,1240109,1240112,1240114
 
,1240116,1240118,1240121,1240329,1240474-1240850,1240857,1241087,1241160,1241408-1241822,1241908-1241909,1241912-1242110,1242371-1292130,1292134-1292458,1292464-1292670,1292672-1292776,1292780-1293392,1293397-1297017,1297019-1297963,1297965-1299820,1300108,1300111-1300460,1300520-1300948,1300997,1301006,1301280,1302332,1302348,1302608-1302610,1302649,1302837,1303138,1303163,1303338,1303521,1303587,1303698,1303803,1303852,1304011,1304035,1304037,1304135,1304249,1304253,1304260,1304271,1304275,1304468,1304895,1304930-1304932,1305194,1305943,1305965,1306556,1306579-1306580,1307084,1307310,1307511-1307512,1307579,1307591,1307597,1310636,1310639-1310640,1310642,1310701,1311212,1311995,1327617,1327670,1331766,1333161,1333173,1333827,1334787,1335026,1335257,1335547,1335692,1335711,1335731,1336515,1336813,1336864,1336868,1336884,1337419,1337426,1337546,1337572,1337591-1337595,1337643,1337707,1337719,1337734,1337741,1337745,1338151-1338154,1338178,1342027,1342029,1342315,1342320,1342476,1342
 
498,1342503,1342717,1342795,1342805,1343044-1343046,1343335,1343394,1343400,1343629,1343708,1343718,1343895,1344063,1344068,1344250,1344266,1344515,1344528,1344612,1344629,1344725,1344868,1344890,1344893,1344896,1344901,1345020,1345029,1345039,1345287-1345290,1345294,1345309,1345325,1345357,1345367,1345579-1345580,1345582,1345688,1345699,1345704,1345731-1345732,1345737,1345744,1345752,1345754,1345779,1345781,1345846,1346107,1346365,1346376,1346404,1346510,1346514,1346519,1346581,1346635,1346644,1346683,1346794,1346885,1346932,1347034,1347047,1347087,1347108-1347109,1347583,1347737,1348105,1348357,1348398,1348425,1348461-1348495,1348498,1348752,1348762,1348772,1348776,1348859,1348968,1348973,1348989,1349007,1349237,1349298,1349317,1349410,1349473,1349539,1349879,1349887,1349893,1349922,1349984,1350124,1350241,1350243,1350294-1350295,1350299,1350864,1350900,1351010,1351054,1351056,1351068,1351134-1351135,1351148,1351259,1351604,1351636-1351640,1351991,1351993,1352011,1352056,1352059,1
 

svn commit: r1658819 - /tomcat/tc6.0.x/trunk/STATUS.txt

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 21:28:53 2015
New Revision: 1658819

URL: http://svn.apache.org/r1658819
Log:
proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1658819r1=1658818r2=1658819view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Feb 10 21:28:53 2015
@@ -53,6 +53,15 @@ PATCHES PROPOSED TO BACKPORT:
   +1: markt, kkolinko
   -1:
 
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
+  Add missing JARs in Ant task definition. Expand the pattern in
+  catalina-tasks.xml to incude all jars in ${catalina.home}/lib.
+  This adds catalina.jar required by the validate task, as well as optional
+  i18n jars.
+  http://svn.apache.org/r1658815
+  +1: kkolinko
+  -1:
+
 
 PATCHES/ISSUES THAT ARE STALLED:
 



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



svn commit: r1658805 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/core/StandardContext.java test/org/apache/catalina/core/TestStandardContext.java webapps/docs/changelog.xml

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:55:27 2015
New Revision: 1658805

URL: http://svn.apache.org/r1658805
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57556
Align getRealPath() behaviour with that of earlier versions and include a 
trailing separator if the real path refers to a directory.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/java/org/apache/catalina/core/StandardContext.java
tomcat/tc8.0.x/trunk/test/org/apache/catalina/core/TestStandardContext.java
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 20:55:27 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790,1658799,1658802
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790,1658799,1658802,1658804

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/catalina/core/StandardContext.java
URL: 

[Bug 57558] validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

Konstantin Kolinko knst.koli...@gmail.com changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
  Component|Jasper  |Jasper
Version|8.0.18  |6.0.43
 Resolution|FIXED   |---
Product|Tomcat 8|Tomcat 6

--- Comment #2 from Konstantin Kolinko knst.koli...@gmail.com ---
This is observed on 7.0.59 and current 6.0.x as well. I will backport the fix.

-- 
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



Re: svn commit: r1658804 - in /tomcat/trunk: java/org/apache/catalina/core/StandardContext.java test/org/apache/catalina/core/TestStandardContext.java

2015-02-10 Thread Konstantin Kolinko
2015-02-10 23:54 GMT+03:00  ma...@apache.org:
 Author: markt
 Date: Tue Feb 10 20:54:07 2015
 New Revision: 1658804

 URL: http://svn.apache.org/r1658804
 Log:
 Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57556
 Align getRealPath() behaviour with that of earlier versions and include a 
 trailing separator if the real path refers to a directory.

 Modified:
 tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
 tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java

 Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
 URL: 
 http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardContext.java?rev=1658804r1=1658803r2=1658804view=diff
 ==
 --- tomcat/trunk/java/org/apache/catalina/core/StandardContext.java (original)
 +++ tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Tue Feb 
 10 20:54:07 2015
 @@ -4407,7 +4407,12 @@ public class StandardContext extends Con
  }
  if (resources != null) {
  try {
 -return resources.getResource(path).getCanonicalPath();
 +WebResource resource = resources.getResource(path);
 +if (resource.isDirectory()) {
 +return resource.getCanonicalPath() + File.separatorChar;

1) I think it is better to check whether it ends with separatorChar first.

E.g. if a Windows drive root is deployed as a web application. I think
that getCanonicalPath() will end with a backslash.

T:\ is the root directory of drive T,
T: is the current directory on drive T.

2) I think that this changes the value of getRealPath(),

I have not tested whether it was ending with a slash in the old versions.

 +} else {
 +return resource.getCanonicalPath();
 +}
  } catch (IllegalArgumentException iae) {
  // ServletContext.getRealPath() does not allow this to be 
 thrown
  }


Best regards,
Konstantin Kolinko

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



[Bug 57558] validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

--- Comment #5 from Konstantin Kolinko knst.koli...@gmail.com ---
For a record:

Stacktrace in Tomcat 6 is
[[[
build.xml:15: java.lang.NoClassDefFoundError: org
/apache/catalina/Globals
at org.apache.catalina.ant.ValidatorTask.execute(ValidatorTask.java:95)
at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:292)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at ...
]]]

The line that refers the Globals class in ValidatorTask.java:

   Digester digester = DigesterFactory.newDigester(
   true, true, null, Globals.IS_SECURITY_ENABLED);

-- 
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 57559] Decoded Request URI used for Asynchronous dispatch

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57559

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 OS||All

-- 
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: r1658804 - in /tomcat/trunk: java/org/apache/catalina/core/StandardContext.java test/org/apache/catalina/core/TestStandardContext.java

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:54:07 2015
New Revision: 1658804

URL: http://svn.apache.org/r1658804
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57556
Align getRealPath() behaviour with that of earlier versions and include a 
trailing separator if the real path refers to a directory.

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardContext.java?rev=1658804r1=1658803r2=1658804view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardContext.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Tue Feb 10 
20:54:07 2015
@@ -4407,7 +4407,12 @@ public class StandardContext extends Con
 }
 if (resources != null) {
 try {
-return resources.getResource(path).getCanonicalPath();
+WebResource resource = resources.getResource(path);
+if (resource.isDirectory()) {
+return resource.getCanonicalPath() + File.separatorChar;
+} else {
+return resource.getCanonicalPath();
+}
 } catch (IllegalArgumentException iae) {
 // ServletContext.getRealPath() does not allow this to be 
thrown
 }

Modified: tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java?rev=1658804r1=1658803r2=1658804view=diff
==
--- tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestStandardContext.java Tue Feb 
10 20:54:07 2015
@@ -937,6 +937,32 @@ public class TestStandardContext extends
 Assert.assertNull(realPath);
 }
 
+/*
+ * Check real path for directories ends with File.separator for consistency
+ * with previous major versions.
+ */
+@Test
+public void testBug57556() throws Exception {
+Tomcat tomcat = getTomcatInstanceTestWebapp(false, true);
+Context testContext = ((Context) tomcat.getHost().findChildren()[0]);
+doTestBug57556(testContext, /, true);
+doTestBug57556(testContext, /jsp, true);
+doTestBug57556(testContext, /jsp/, true);
+doTestBug57556(testContext, /index.html, false);
+// Doesn't exist so Tomcat will assume it is a file, not a directory.
+doTestBug57556(testContext, /foo, false);
+}
+
+private void doTestBug57556(Context testContext, String path, boolean 
endsInSeparator) throws Exception {
+String realPath = testContext.getRealPath(path);
+Assert.assertNotNull(realPath);
+if (endsInSeparator) {
+Assert.assertTrue(realPath, realPath.endsWith(File.separator));
+} else {
+Assert.assertFalse(realPath, realPath.endsWith(File.separator));
+}
+}
+
 @Test
 public void testBug56903() {
 Context context = new StandardContext();



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



[Bug 57558] validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

--- Comment #3 from Konstantin Kolinko knst.koli...@gmail.com ---
Complete sample build.xml (to be put into $CATALINA_HOME):

[[[
project default=jspc
  property name=tomcat.home location=${basedir} /
  property name=webapp.path location=${basedir}/webapps/examples /

  import file=${tomcat.home}/bin/catalina-tasks.xml/

   target name=jspc

jasper
 validateXml=false
 uriroot=${webapp.path}
 webXmlFragment=${webapp.path}/WEB-INF/generated_web.xml
 outputDir=${webapp.path}/WEB-INF/src /

validator path=${webapp.path} /

  /target
/project
]]]

-- 
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 57558] validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

--- Comment #4 from Konstantin Kolinko knst.koli...@gmail.com ---
Fixed in Tomcat 7 by r1658815 and will be in 7.0.60.
Proposed for Tomcat 6.

-- 
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: r1658796 - in /tomcat/site/trunk: docs/download-70.html xdocs/download-70.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 20:30:21 2015
New Revision: 1658796

URL: http://svn.apache.org/r1658796
Log:
Add links to SHA-1 checksum files for Tomcat 7.

Modified:
tomcat/site/trunk/docs/download-70.html
tomcat/site/trunk/xdocs/download-70.xml

Modified: tomcat/site/trunk/docs/download-70.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-70.html?rev=1658796r1=1658795r2=1658796view=diff
==
--- tomcat/site/trunk/docs/download-70.html (original)
+++ tomcat/site/trunk/docs/download-70.html Tue Feb 10 20:30:21 2015
@@ -219,9 +219,9 @@
 release file.  This signature should be matched against the
 a href=https://www.apache.org/dist/tomcat/tomcat-7/KEYS;KEYS/a file
 which contains the OpenPGP keys of Tomcat's Release Managers. We also
-provide an codeMD5/code checksum for every release file. After you
-download the file, you should calculate a checksum for your download, and
-make sure it is the same as ours.
+provide codeMD5/code and codeSHA-1/code checksums for every
+release file. After you download the file, you should calculate a
+checksum for your download, and make sure it is the same as ours.
 /p
   
 /div
@@ -276,42 +276,48 @@
 
 a href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].zip 
rel=nofollowzip/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].zip.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].zip.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].zip.sha1;sha1/a)
   /li
   
 li
 
 a href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].tar.gz 
rel=nofollowtar.gz/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].tar.gz.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].tar.gz.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].tar.gz.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].tar.gz.sha1;sha1/a)
   /li
   
 li
 
 a 
href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x86.zip 
rel=nofollow32-bit Windows zip/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.sha1;sha1/a)
   /li
   
 li
 
 a 
href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x64.zip 
rel=nofollow64-bit Windows zip/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.sha1;sha1/a)
   /li
   
 li
 
 a 
href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-i64.zip 
rel=nofollow64-bit Itanium Windows zip/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-i64.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-i64.zip.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-i64.zip.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v]-windows-i64.zip.sha1;sha1/a)
   /li
   
 li
 
 a href=[preferred]tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].exe 
rel=nofollow32-bit/64-bit Windows Service Installer/a 
 (a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].exe.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].exe.md5;md5/a)
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].exe.md5;md5/a,
+a 
href=https://www.apache.org/dist/tomcat/tomcat-7/v[v]/bin/apache-tomcat-[v].exe.sha1;sha1/a)
   /li
   
 /ul
@@ -325,7 +331,8 @@
 

Re: svn commit: r1658799 - /tomcat/trunk/bin/catalina-tasks.xml

2015-02-10 Thread Mark Thomas
On 10/02/2015 20:36, Konstantin Kolinko wrote:
 2015-02-10 23:32 GMT+03:00  ma...@apache.org:
 Author: markt
 Date: Tue Feb 10 20:32:45 2015
 New Revision: 1658799

 URL: http://svn.apache.org/r1658799
 Log:
 Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
 Add missing JARs
 JARs in alpha order

 Modified:
 tomcat/trunk/bin/catalina-tasks.xml

 Modified: tomcat/trunk/bin/catalina-tasks.xml
 URL: 
 http://svn.apache.org/viewvc/tomcat/trunk/bin/catalina-tasks.xml?rev=1658799r1=1658798r2=1658799view=diff
 ==
 --- tomcat/trunk/bin/catalina-tasks.xml (original)
 +++ tomcat/trunk/bin/catalina-tasks.xml Tue Feb 10 20:32:45 2015
 @@ -28,16 +28,20 @@
typedef resource=org/apache/catalina/ant/catalina.tasks
  classpath
fileset file=${catalina.home}/bin/tomcat-juli.jar/
 -  fileset file=${catalina.home}/lib/tomcat-api.jar/
 -  fileset file=${catalina.home}/lib/tomcat-util.jar/
 -  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
 +  fileset file=${catalina.home}/lib/catalina-ant.jar/
 +  fileset file=${catalina.home}/lib/catalina.jar/
 +  fileset file=${catalina.home}/lib/el-api.jar/
fileset file=${catalina.home}/lib/jasper.jar/
fileset file=${catalina.home}/lib/jasper-el.jar/
 -  fileset file=${catalina.home}/lib/el-api.jar/
fileset file=${catalina.home}/lib/jsp-api.jar/
fileset file=${catalina.home}/lib/servlet-api.jar/
 -  fileset file=${catalina.home}/lib/catalina-ant.jar/
 +  fileset file=${catalina.home}/lib/tomcat-api.jar/
fileset file=${catalina.home}/lib/tomcat-coyote.jar/
 +  fileset file=${catalina.home}/lib/tomcat-util.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-es.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-fr.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-ja.jar/
 +  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
  /classpath
/typedef
 
 
 I suspect that the above will fail if *i18n* jars are not present. I
 think one should be allowed to remove them from an installation.

See r1658802

The downside of r1658802 is if a JAR is missing it isn't immediately
obvious which one it is.

Mark


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



Re: Consider support for the Servlet profile of JSR 196 (JASPIC) in Tomcat 7.0.x

2015-02-10 Thread Mark Thomas
On 10/02/2015 18:42, Arjan Tijms wrote:
 Hi,
 
 On Tuesday, February 10, 2015, Fjodor Vershinin [via Tomcat] 
 ml-node+s10n5029627...@n6.nabble.com wrote:
 
 Hello!
 I am CS student and it looks like that this task is quite interesting. I
 would take it for GSOC if ASF organization will be selected. Currently I
 have some time to do research in Tomcat codebase. Could you provide me some
 entry points?
 
 
 Thanks for your interest in this. An entry point could be my original
 JASPIC article that you can find here;
 http://arjan-tijms.omnifaces.org/2012/11/implementing-container-authentication.html
 
 At the end of the article you'll find a list of resources.
 
 My approach would be to investigate how Tomcat integrates authentication
 modules, eg look at the source of the JAAS support in Tomcat; that code has
 to do similar integration. You can look at JBoss 7.x for an example too, it
 used Tomcat and an integration Valve (WebJaspiAuthenticator seehttp://
 grepcode.com/file/repository.jboss.org/nexus/content/repositories/releases/org.jboss.as/jboss-as-web/7.1.1.Final/org/jboss/as/web/security/jaspi/WebJASPIAuthenticator.java

If you do look at JBoss keep in mind it is GPL licensed and we need to
be very careful that we don't end up with GPL'd code in Tomcat.

 )
 
 Geronimo also implemented JASPIC and used Tomcat, so that implementation
 would be high on the list to study too.

Personally, I'd look more much more closely at Geronimo.

Keep in mind that part of the goal is to replace the existing
authenticators with JASPIC modules. (As suggested on the Servlet EG list.)

 Many implementations have a (large) part of their code dedicated to
 handling some xml file where jaspic auth modules are defined. Strictly
 speaking this is not a required part of JASPIC, but it's somewhat expected
 for configuring modules at the container side (as apposed to from within
 the app archive).

Tomcat already has a lot of the infrastructure for handling this sort of
thing. It could be as simple as adding a few digester rules.

 I did actually more or less promise to do this implementation myself, but
 so far havent found the time for it.

I think we all know that feeling - hence why I suggested it for GSoC.

Mark

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



Re: svn commit: r1658737 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/AbstractProcessor.java java/org/apache/coyote/http11/upgrade/LocalStrings.properties webapps/docs/changelo

2015-02-10 Thread Christopher Schultz
Mark,

On 2/10/15 10:53 AM, ma...@apache.org wrote:
 Author: markt
 Date: Tue Feb 10 15:53:06 2015
 New Revision: 1658737
 
 URL: http://svn.apache.org/r1658737
 Log:
 Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57546
 Ensure that a dropped connection does not leave references to the 
 UpgradeProcessor
 
 Modified:
 tomcat/tc8.0.x/trunk/   (props changed)
 
 tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java
 
 tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
 tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
 
 Propchange: tomcat/tc8.0.x/trunk/
 --
 --- svn:mergeinfo (original)
 +++ svn:mergeinfo Tue Feb 10 15:53:06 2015
 @@ -1 +1 @@
 -/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,164489
2
  
 ,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,165
7
  907,1658207
 +/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,164489
2
  
 ,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,165
7
  907,1658207,1658734
 
 Modified: 
 tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java
 URL: 
 

[Bug 57509] maxHttpHeaderSize maybe case ArrayIndexOutOfBoundsException

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57509

Shelson Ferrari supo...@vitplay.com changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

--- Comment #7 from Shelson Ferrari supo...@vitplay.com ---
 - 4 bytes added is the correct fixe ?


The method sendHeader --

write(name);
headerBuffer[pos++] = Constants.COLON;
headerBuffer[pos++] = Constants.SP;
write(value);
headerBuffer[pos++] = Constants.CR;
headerBuffer[pos++] = Constants.LF;

call the method write(MessageBytes mb) 2 times and at the end of this method,
call the method write(mb.getByteChunk()

this method call checkLengthBeforeWrite(length)

In the first call write(name) -- 
this block that increments 2 positions is not called
headerBuffer[pos++] = Constants.COLON;
headerBuffer[pos++] = Constants.SP;


The class AbstractHttp11Processor method private void prepareResponse()
call the class AOB --
int size = headers.size();
for (int i = 0; i  size; i++) {
getOutputBuffer().sendHeader(headers.getName(i), headers.getValue(i));
}
getOutputBuffer().endHeaders();



in other words, I believe:


the method checkLengthBeforeWrite in the first call 
if (pos + length  headerBuffer.length)

the method checkLengthBeforeWrite in the second call 
if (pos + length + 2  headerBuffer.length)

and if have a third scan after getOutputBuffer().endHeaders() - there is no
such verification - and also would not make sense
if (pos + length + 4  headerBuffer.length)

and would be after getOutputBuffer().endHeaders()


Please check.

-- 
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



Re: svn commit: r1658799 - /tomcat/trunk/bin/catalina-tasks.xml

2015-02-10 Thread Konstantin Kolinko
2015-02-10 23:32 GMT+03:00  ma...@apache.org:
 Author: markt
 Date: Tue Feb 10 20:32:45 2015
 New Revision: 1658799

 URL: http://svn.apache.org/r1658799
 Log:
 Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
 Add missing JARs
 JARs in alpha order

 Modified:
 tomcat/trunk/bin/catalina-tasks.xml

 Modified: tomcat/trunk/bin/catalina-tasks.xml
 URL: 
 http://svn.apache.org/viewvc/tomcat/trunk/bin/catalina-tasks.xml?rev=1658799r1=1658798r2=1658799view=diff
 ==
 --- tomcat/trunk/bin/catalina-tasks.xml (original)
 +++ tomcat/trunk/bin/catalina-tasks.xml Tue Feb 10 20:32:45 2015
 @@ -28,16 +28,20 @@
typedef resource=org/apache/catalina/ant/catalina.tasks
  classpath
fileset file=${catalina.home}/bin/tomcat-juli.jar/
 -  fileset file=${catalina.home}/lib/tomcat-api.jar/
 -  fileset file=${catalina.home}/lib/tomcat-util.jar/
 -  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
 +  fileset file=${catalina.home}/lib/catalina-ant.jar/
 +  fileset file=${catalina.home}/lib/catalina.jar/
 +  fileset file=${catalina.home}/lib/el-api.jar/
fileset file=${catalina.home}/lib/jasper.jar/
fileset file=${catalina.home}/lib/jasper-el.jar/
 -  fileset file=${catalina.home}/lib/el-api.jar/
fileset file=${catalina.home}/lib/jsp-api.jar/
fileset file=${catalina.home}/lib/servlet-api.jar/
 -  fileset file=${catalina.home}/lib/catalina-ant.jar/
 +  fileset file=${catalina.home}/lib/tomcat-api.jar/
fileset file=${catalina.home}/lib/tomcat-coyote.jar/
 +  fileset file=${catalina.home}/lib/tomcat-util.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-es.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-fr.jar/
 +  fileset file=${catalina.home}/lib/tomcat-i18n-ja.jar/
 +  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
  /classpath
/typedef


I suspect that the above will fail if *i18n* jars are not present. I
think one should be allowed to remove them from an installation.

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



[Bug 57558] validator ant task throws NoClassDefFoundError - Tomcat 8.0.18

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57558

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
 OS||All

--- Comment #1 from Mark Thomas ma...@apache.org ---
Fixed in trunk and 8.0.x for 8.0.19 onwards.

-- 
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: r1658803 - in /tomcat/tc8.0.x/trunk: ./ bin/catalina-tasks.xml

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:40:13 2015
New Revision: 1658803

URL: http://svn.apache.org/r1658803
Log:
Simplify. Use all available JARs.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 20:40:13 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790,1658799
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790,1658799,1658802

Modified: tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml?rev=1658803r1=1658802r2=1658803view=diff
==
--- tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml (original)
+++ tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml Tue Feb 10 20:40:13 2015
@@ -28,20 +28,7 @@
   typedef 

svn commit: r1658790 - in /tomcat/trunk/java/org/apache/coyote/http11/upgrade: LocalStrings.properties UpgradeProcessor.java UpgradeServletInputStream.java UpgradeServletOutputStream.java

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:05:12 2015
New Revision: 1658790

URL: http://svn.apache.org/r1658790
Log:
Make code calling user provided listeners more robust.

Modified:
tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java

tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeServletInputStream.java

tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeServletOutputStream.java

Modified: 
tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties?rev=1658790r1=1658789r2=1658790view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties 
Tue Feb 10 20:05:12 2015
@@ -14,16 +14,18 @@
 # limitations under the License.
 
 upgradeProcessor.isCloseFail=Failed to close input stream associated with 
upgraded connection
-upgradeProcessor.onDataAvailableFail=Failed to process data available event
-upgradeProcessor.onWritePossibleFail=Failed to process write possible event
 upgradeProcessor.osCloseFail=Failed to close output stream associated with 
upgraded connection
 
+upgrade.sis.errorCloseFail=Failed to close InputStream cleanly after a 
previous error
 upgrade.sis.isFinished.ise=It is illegal to call isFinished() when the 
ServletInputStream is not in non-blocking mode (i.e. setReadListener() must be 
called first)
 upgrade.sis.isReady.ise=It is illegal to call isReady() when the 
ServletInputStream is not in non-blocking mode (i.e. setReadListener() must be 
called first)
+upgrade.sis.onErrorFail=onError processing for the registered ReadListener 
triggered this further error which was swallowed
 upgrade.sis.readListener.null=It is illegal to pass null to setReadListener()
 upgrade.sis.readListener.set=It is illegal to call setReadListener() more than 
once for the same upgraded connection
 upgrade.sis.read.ise=It is illegal to call any of the read() methods in 
non-blocking mode without first checking that there is data available by 
calling isReady()
+upgrade.sos.errorCloseFail=Failed to close OutputStream cleanly after a 
previous error
 upgrade.sos.canWrite.ise=It is illegal to call canWrite() when the 
ServletOutputStream is not in non-blocking mode (i.e. setWriteListener() must 
be called first)
+upgrade.sos.onErrorFail=onError processing for the registered WriteListener 
triggered this further error which was swallowed
 upgrade.sos.writeListener.null=It is illegal to pass null to setWriteListener()
 upgrade.sos.writeListener.set=It is illegal to call setWriteListener() more 
than once for the same upgraded connection
 upgrade.sis.write.ise=It is illegal to call any of the write() methods in 
non-blocking mode without first checking that there is space available by 
calling isReady()

Modified: 
tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java?rev=1658790r1=1658789r2=1658790view=diff
==
--- tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
(original)
+++ tomcat/trunk/java/org/apache/coyote/http11/upgrade/UpgradeProcessor.java 
Tue Feb 10 20:05:12 2015
@@ -98,24 +98,10 @@ public class UpgradeProcessor implements
 @Override
 public final SocketState upgradeDispatch(SocketStatus status) {
 if (status == SocketStatus.OPEN_READ) {
-try {
-upgradeServletInputStream.onDataAvailable();
-upgradeServletOutputStream.checkWriteDispatch();
-} catch (IOException ioe) {
-// The error handling within the ServletInputStream should have
-// marked the stream for closure which will get picked up 
below,
-// triggering the clean-up of this processor.
-
log.debug(sm.getString(upgradeProcessor.onDataAvailableFail), ioe);
-}
+upgradeServletInputStream.onDataAvailable();
+upgradeServletOutputStream.checkWriteDispatch();
 } else if (status == SocketStatus.OPEN_WRITE) {
-try {
-upgradeServletOutputStream.onWritePossible();
-} catch (IOException ioe) {
-// The error handling within the ServletOutputStream should 
have
-// marked the stream for closure which will get picked up 
below,
-// triggering the clean-up of this processor.
-
log.debug(sm.getString(upgradeProcessor.onWritePossibleFail), ioe);
-}
+upgradeServletOutputStream.onWritePossible();
 } 

svn commit: r1658799 - /tomcat/trunk/bin/catalina-tasks.xml

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:32:45 2015
New Revision: 1658799

URL: http://svn.apache.org/r1658799
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
Add missing JARs
JARs in alpha order

Modified:
tomcat/trunk/bin/catalina-tasks.xml

Modified: tomcat/trunk/bin/catalina-tasks.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/bin/catalina-tasks.xml?rev=1658799r1=1658798r2=1658799view=diff
==
--- tomcat/trunk/bin/catalina-tasks.xml (original)
+++ tomcat/trunk/bin/catalina-tasks.xml Tue Feb 10 20:32:45 2015
@@ -28,16 +28,20 @@
   typedef resource=org/apache/catalina/ant/catalina.tasks
 classpath
   fileset file=${catalina.home}/bin/tomcat-juli.jar/
-  fileset file=${catalina.home}/lib/tomcat-api.jar/
-  fileset file=${catalina.home}/lib/tomcat-util.jar/
-  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
+  fileset file=${catalina.home}/lib/catalina-ant.jar/
+  fileset file=${catalina.home}/lib/catalina.jar/
+  fileset file=${catalina.home}/lib/el-api.jar/
   fileset file=${catalina.home}/lib/jasper.jar/
   fileset file=${catalina.home}/lib/jasper-el.jar/
-  fileset file=${catalina.home}/lib/el-api.jar/
   fileset file=${catalina.home}/lib/jsp-api.jar/
   fileset file=${catalina.home}/lib/servlet-api.jar/
-  fileset file=${catalina.home}/lib/catalina-ant.jar/
+  fileset file=${catalina.home}/lib/tomcat-api.jar/
   fileset file=${catalina.home}/lib/tomcat-coyote.jar/
+  fileset file=${catalina.home}/lib/tomcat-util.jar/
+  fileset file=${catalina.home}/lib/tomcat-i18n-es.jar/
+  fileset file=${catalina.home}/lib/tomcat-i18n-fr.jar/
+  fileset file=${catalina.home}/lib/tomcat-i18n-ja.jar/
+  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
 /classpath
   /typedef
   typedef resource=org/apache/catalina/ant/jmx/jmxaccessor.tasks



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



svn commit: r1658802 - /tomcat/trunk/bin/catalina-tasks.xml

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:39:48 2015
New Revision: 1658802

URL: http://svn.apache.org/r1658802
Log:
Simplify. Use all available JARs.

Modified:
tomcat/trunk/bin/catalina-tasks.xml

Modified: tomcat/trunk/bin/catalina-tasks.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/bin/catalina-tasks.xml?rev=1658802r1=1658801r2=1658802view=diff
==
--- tomcat/trunk/bin/catalina-tasks.xml (original)
+++ tomcat/trunk/bin/catalina-tasks.xml Tue Feb 10 20:39:48 2015
@@ -28,20 +28,7 @@
   typedef resource=org/apache/catalina/ant/catalina.tasks
 classpath
   fileset file=${catalina.home}/bin/tomcat-juli.jar/
-  fileset file=${catalina.home}/lib/catalina-ant.jar/
-  fileset file=${catalina.home}/lib/catalina.jar/
-  fileset file=${catalina.home}/lib/el-api.jar/
-  fileset file=${catalina.home}/lib/jasper.jar/
-  fileset file=${catalina.home}/lib/jasper-el.jar/
-  fileset file=${catalina.home}/lib/jsp-api.jar/
-  fileset file=${catalina.home}/lib/servlet-api.jar/
-  fileset file=${catalina.home}/lib/tomcat-api.jar/
-  fileset file=${catalina.home}/lib/tomcat-coyote.jar/
-  fileset file=${catalina.home}/lib/tomcat-util.jar/
-  fileset file=${catalina.home}/lib/tomcat-i18n-es.jar/
-  fileset file=${catalina.home}/lib/tomcat-i18n-fr.jar/
-  fileset file=${catalina.home}/lib/tomcat-i18n-ja.jar/
-  fileset file=${catalina.home}/lib/tomcat-util-scan.jar/
+  fileset dir=${catalina.home}/lib includes=*.jar/
 /classpath
   /typedef
   typedef resource=org/apache/catalina/ant/jmx/jmxaccessor.tasks



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



Re: Consider support for the Servlet profile of JSR 196 (JASPIC) in Tomcat 7.0.x

2015-02-10 Thread Arjan Tijms
Hi,

On Tue, Feb 10, 2015 at 8:34 PM, Mark Thomas-2 [via Tomcat]
ml-node+s10n502964...@n6.nabble.com wrote:
 If you do look at JBoss keep in mind it is GPL licensed and we need to
 be very careful that we don't end up with GPL'd code in Tomcat.

That's absolutely true. The code there shouldn't not be copied in any
way. It's only useful as an example of how a Tomcat Valve can
integrate with something like JASPIC. As for the JASPIC code there, it
wouldn't make sense to copy it anyway, since A) it's JBoss specific
(builds up JBoss principal, calls JBoss security service, etc) and B)
there are various issues with it (it looks like JBoss pretty much
rewrote everything from scratch for Undertow, which is completely
different).

 Personally, I'd look more much more closely at Geronimo.

You're right, and since that one is Apache licensed one can even copy
from it if needed.

 Keep in mind that part of the goal is to replace the existing
 authenticators with JASPIC modules. (As suggested on the Servlet EG list.)

It's good to have that as part of the goal indeed. Such auth modules
could even be implemented by a separate (group) of students if needed,
as they would not necessarily depend on the JASPIC implementation for
Tomcat. As long as that one is not finished they could test it on any
existing JASPIC implementation (e.g. the RI, GlassFish).

 I think we all know that feeling - hence why I suggested it for GSoC.

Yeah, I get that, thanks! It's still something that I'd really love to
do, but with the work for the startup zeef.com, open source projects
OmniFaces and OmniSecurity, the work for the JSF EG and perhaps soon
for the security EG, there is not always much time left. I had this on
my sketchy todo list for ~end of this month, but I'll see what happens
with the GSoC project now ;)

Kind regards,
Arjan





 Mark

 -
 To unsubscribe, e-mail: [hidden email]
 For additional commands, e-mail: [hidden email]



 
 If you reply to this email, your message will be added to the discussion
 below:
 http://tomcat.10.x6.nabble.com/Consider-support-for-the-Servlet-profile-of-JSR-196-JASPIC-in-Tomcat-7-0-x-tp4993387p5029643.html
 To unsubscribe from Consider support for the Servlet profile of JSR 196
 (JASPIC) in Tomcat 7.0.x, click here.
 NAML




--
View this message in context: 
http://tomcat.10.x6.nabble.com/Consider-support-for-the-Servlet-profile-of-JSR-196-JASPIC-in-Tomcat-7-0-x-tp4993387p5029664.html
Sent from the Tomcat - Dev mailing list archive at Nabble.com.

Re: svn commit: r1658737 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/AbstractProcessor.java java/org/apache/coyote/http11/upgrade/LocalStrings.properties webapps/docs/changelo

2015-02-10 Thread Mark Thomas
On 10/02/2015 19:54, Christopher Schultz wrote:
 Mark,
 
 On 2/10/15 10:53 AM, ma...@apache.org wrote:
 Author: markt
 Date: Tue Feb 10 15:53:06 2015
 New Revision: 1658737

 URL: http://svn.apache.org/r1658737
 Log:
 Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57546
 Ensure that a dropped connection does not leave references to the 
 UpgradeProcessor

 Modified:
 tomcat/tc8.0.x/trunk/   (props changed)
 
 tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java
 
 tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties
 tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

 Propchange: tomcat/tc8.0.x/trunk/
 --
 --- svn:mergeinfo (original)
 +++ svn:mergeinfo Tue Feb 10 15:53:06 2015
 @@ -1 +1 @@
 -/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,164
489
 2
  
 ,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,
165
 7
  907,1658207
 +/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,164
489
 2
  
 ,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,
165
 7
  907,1658207,1658734

 Modified: 
 

[Bug 57509] maxHttpHeaderSize maybe case ArrayIndexOutOfBoundsException

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57509

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Mark Thomas ma...@apache.org ---
(In reply to Shelson Ferrari from comment #7)
  - 4 bytes added is the correct fixe ?

Yes.

-- 
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: r1658795 - in /tomcat/site/trunk: docs/download-60.html docs/download-70.html xdocs/download-60.xml xdocs/download-70.xml

2015-02-10 Thread kkolinko
Author: kkolinko
Date: Tue Feb 10 20:23:19 2015
New Revision: 1658795

URL: http://svn.apache.org/r1658795
Log:
Simplify download page by using ezt.py variable to hold a version number.

Documentation for ezt.py is at https://code.google.com/p/ezt/wiki/Syntax

Modified:
tomcat/site/trunk/docs/download-60.html
tomcat/site/trunk/docs/download-70.html
tomcat/site/trunk/xdocs/download-60.xml
tomcat/site/trunk/xdocs/download-70.xml

Modified: tomcat/site/trunk/docs/download-60.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-60.html?rev=1658795r1=1658794r2=1658795view=diff
==
--- tomcat/site/trunk/docs/download-60.html (original)
+++ tomcat/site/trunk/docs/download-60.html Tue Feb 10 20:23:19 2015
@@ -202,10 +202,11 @@
 /div
 h3 id=Quick_NavigationQuick Navigation/h3
 div class=text
-
-a href=https://www.apache.org/dist/tomcat/tomcat-6/KEYS;KEYS/a |
-a href=#6.0.436.0.43/a |
-a href=[preferred]tomcat/tomcat-6/v6.0.43 rel=nofollowBrowse/a 
|
+
+[define v]6.0.43[end]
+a href=https://www.apache.org/dist/tomcat/tomcat-6/KEYS;KEYS/a |
+a href=#[v][v]/a |
+a href=[preferred]tomcat/tomcat-6/v[v] rel=nofollowBrowse/a |
 a href=http://archive.apache.org/dist/tomcat/tomcat-6;Archives/a
   
 /div
@@ -252,12 +253,12 @@

   
 /div
-h3 id=6.0.436.0.43/h3
+h3 id=[v][v]/h3
 div class=text
   
 p
   Please see the 
-  a href=[preferred]tomcat/tomcat-6/v6.0.43/README.html 
rel=nofollowREADME/a
+  a href=[preferred]tomcat/tomcat-6/v[v]/README.html 
rel=nofollowREADME/a
   file for packaging information.  It explains what every distribution 
contains.
   /p
 
@@ -273,44 +274,44 @@
   
 li
 
-a href=[preferred]tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.zip 
rel=nofollowzip/a 
-(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.zip.md5;md5/a)
+a href=[preferred]tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].zip 
rel=nofollowzip/a 
+(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].zip.asc;pgp/a,
 
+a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].zip.md5;md5/a)
   /li
   
 li
 
-a href=[preferred]tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.tar.gz 
rel=nofollowtar.gz/a 
-(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.tar.gz.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43.tar.gz.md5;md5/a)
+a href=[preferred]tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].tar.gz 
rel=nofollowtar.gz/a 
+(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].tar.gz.asc;pgp/a,
 
+a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v].tar.gz.md5;md5/a)
   /li
   
 li
 
-a 
href=[preferred]tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x86.zip
 rel=nofollow32-bit Windows zip/a 
-(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x86.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x86.zip.md5;md5/a)
+a 
href=[preferred]tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x86.zip 
rel=nofollow32-bit Windows zip/a 
+(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.asc;pgp/a,
 
+a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x86.zip.md5;md5/a)
   /li
   
 li
 
-a 
href=[preferred]tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x64.zip
 rel=nofollow64-bit Windows zip/a 
-(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x64.zip.asc;pgp/a,
 
-a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-x64.zip.md5;md5/a)
+a 
href=[preferred]tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x64.zip 
rel=nofollow64-bit Windows zip/a 
+(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.asc;pgp/a,
 
+a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v[v]/bin/apache-tomcat-[v]-windows-x64.zip.md5;md5/a)
   /li
   
 li
 
-a 
href=[preferred]tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-i64.zip
 rel=nofollow64-bit Itanium Windows zip/a 
-(a 
href=https://www.apache.org/dist/tomcat/tomcat-6/v6.0.43/bin/apache-tomcat-6.0.43-windows-i64.zip.asc;pgp/a,
 
-a 

svn commit: r1658794 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:21:38 2015
New Revision: 1658794

URL: http://svn.apache.org/r1658794
Log:
Make code calling user provided listeners more robust.

Modified:
tomcat/tc8.0.x/trunk/   (props changed)

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractServletInputStream.java

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractServletOutputStream.java

tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/LocalStrings.properties

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 20:21:38 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/coyote/http11/upgrade/AbstractProcessor.java
URL: 

svn commit: r1658801 - in /tomcat/tc8.0.x/trunk: ./ bin/catalina-tasks.xml webapps/docs/changelog.xml

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 20:35:24 2015
New Revision: 1658801

URL: http://svn.apache.org/r1658801
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57558
Add missing JARs
JARs in alpha order

Modified:
tomcat/tc8.0.x/trunk/   (props changed)
tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Feb 10 20:35:24 2015
@@ -1 +1 @@
-/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790
+/tomcat/trunk:1636524,1637156,1637176,1637188,1637331,1637684,1637695,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,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,1655438,1655441,1655454,168,1656087,1656299,1656319,1656331,1656345,1656350,1656590,1656648-1656650,1656657,1657041,1657054,1657374,1657492,1657510,1657565,1657580,1657584,1657586,1657589,1657592,1657607,1657609,1657682,1657
 907,1658207,1658734,1658781,1658790,1658799

Modified: tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml?rev=1658801r1=1658800r2=1658801view=diff
==
--- tomcat/tc8.0.x/trunk/bin/catalina-tasks.xml (original)
+++ 

Re: async and thread constraint

2015-02-10 Thread Konstantin Kolinko
2015-02-10 14:16 GMT+03:00 Romain Manni-Bucau rmannibu...@gmail.com:
 Hi guys,

 in org.apache.coyote.AsyncStateMachine#asyncDispatch tomcat checks
 ContainerThreadMarker.isContainerThread()

 basically it prevents to use JAXRS 2 @Suspended without creating
 custom threads - which is not intended.

I do not know what the above is about.

 Was this limitation intended or just to try to avoid to consume
 container threads?

Looking at blame results, those lines were added in
http://svn.apache.org/r1594198
Address root cause of ReadPendingException rather than swallowing it.

Discussion that preceded that commit is in Re:r1593303 thread,
http://tomcat.markmail.org/thread/4kwnhzwhhxi43sob

quote
I think I have got to the bottom of this. An async dispatch always
adds the socket to the poller but if the dispatch occurs on a
container thread then it will also be added to the poller when that
thread exits the AbstractConnectionHandler.process() method.

This looks like a bug with all the connectors but one that is more
obvious with Nio2 because it triggers the ReadPendingException. Fixing
this bug highlighted a minor issue on shutdown. I have a fix for that
too. Commits to follow shortly.
/quote

Best regards,
Konstantin Kolinko

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



[Bug 49785] Enabling TLS for JNDIRealm

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49785

Felix Schumacher felix.schumac...@internetallee.de changed:

   What|Removed |Added

  Attachment #25916|0   |1
is obsolete||

--- Comment #3 from Felix Schumacher felix.schumac...@internetallee.de ---
Created attachment 32448
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=32448action=edit
Enable starttls for JNDIRealm

I integrated the functionality into JNDIRealm. There is no documentation yet
and I am not sure, whether I should include the HostnameVerifier as an enum. I
will extend the method, so that I can use the given string for construction of
one.

I will commit the code together with documentation to trunk if noone objects.

-- 
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: r1658827 - /tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java

2015-02-10 Thread markt
Author: markt
Date: Tue Feb 10 22:42:15 2015
New Revision: 1658827

URL: http://svn.apache.org/r1658827
Log:
Add test case to confirm current behaviour re BZ 57559.

Modified:
tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java

Modified: tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java?rev=1658827r1=1658826r2=1658827view=diff
==
--- tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java Tue 
Feb 10 22:42:15 2015
@@ -2144,4 +2144,46 @@ public class TestAsyncContextImpl extend
 ac.dispatch(target);
  }
 }
+
+// https://issues.apache.org/bugzilla/show_bug.cgi?id=57559
+@Test
+public void testAsyncRequestURI() throws Exception {
+// Setup Tomcat instance
+Tomcat tomcat = getTomcatInstance();
+
+// No file system docBase required
+Context ctx = tomcat.addContext(, null);
+
+Servlet servlet = new AsyncRequestUriServlet();
+Wrapper wrapper1 = Tomcat.addServlet(ctx, bug57559, servlet);
+wrapper1.setAsyncSupported(true);
+ctx.addServletMapping(/, bug57559);
+
+tomcat.start();
+
+String uri = /foo/%24/bar;
+String uriDecoded = /foo/$/bar;
+
+ByteChunk body = getUrl(http://localhost:; + getPort()+ uri);
+
+Assert.assertEquals(uriDecoded, body.toString());
+}
+
+private static class AsyncRequestUriServlet extends HttpServlet {
+
+private static final long serialVersionUID = 1L;
+
+@Override
+protected void doGet(HttpServletRequest req, HttpServletResponse resp)
+throws ServletException, IOException {
+
+if (DispatcherType.ASYNC.equals(req.getDispatcherType())) {
+resp.setContentType(text/plain);
+resp.setCharacterEncoding(UTF-8);
+resp.getWriter().write(req.getRequestURI());
+} else {
+req.startAsync().dispatch();
+}
+}
+}
 }



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



[Bug 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

Konstantin Kolinko knst.koli...@gmail.com changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
  Component|WebSocket   |WebSocket
Version|8.0.18  |7.0.59
 Resolution|FIXED   |---
Product|Tomcat 8|Tomcat 7
   Target Milestone||---

--- Comment #6 from Konstantin Kolinko knst.koli...@gmail.com ---
Only the first part of the fix was applied to Tomcat 7 (r1658738).

The second part was applied to Tomcat 8 (r1658794), but has not been ported to
Tomcat 7 yet. Thus I am REOPENing this issue, so that it is not forgotten.


The issues in the first part:
- as noted by Remy (in Re: r1658734):

 Good find, but what happens if onDataAvailable or onWritePossible throw a
runtime exception (like a NPE), since this is user code. It could also leak
then ? Shouldn't the code catch everything, also call onError on the
listener and close ?

- as noted by Chris (in Re: r1658737):

 Is it worth predicating the log call with if(getLog().isDebugEnabled)?

-- 
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 57489] java.util.concurrent.ExecutionException: java.io.IOException: Key must be cancelled

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57489

--- Comment #10 from wodenc...@gmail.com ---
Hi Mark,

I built Tomcat from Trunk and tested again.

Unfortunately the issue happened again, here is the Stack Trace:

2015-02-10 16:01:13,056 CST ERROR [null] - java.io.IOException: 
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.startMessageBlock(WsRemoteEndpointImplBase.java:282)
at
org.apache.tomcat.websocket.WsSession.sendCloseMessage(WsSession.java:570)
at org.apache.tomcat.websocket.WsSession.doClose(WsSession.java:478)
at org.apache.tomcat.websocket.WsSession.close(WsSession.java:445)
at org.apache.tomcat.websocket.WsSession.close(WsSession.java:439)
at
org.atmosphere.container.version.JSR356WebSocket.close(JSR356WebSocket.java:104)
at org.atmosphere.websocket.WebSocket.close(WebSocket.java:264)
at org.atmosphere.cpr.AtmosphereResponse.close(AtmosphereResponse.java:836)
at
org.atmosphere.cpr.AtmosphereResourceImpl.cancel(AtmosphereResourceImpl.java:720)
at
org.atmosphere.cpr.AtmosphereResourceImpl.close(AtmosphereResourceImpl.java:785)
at
com.vaadin.server.communication.AtmospherePushConnection.disconnect(AtmospherePushConnection.java:294)
at
com.vaadin.server.communication.AtmospherePushConnection.connect(AtmospherePushConnection.java:232)
at com.vaadin.server.communication.PushHandler$2.run(PushHandler.java:135)
at
com.vaadin.server.communication.PushHandler.callWithUi(PushHandler.java:253)
at
com.vaadin.server.communication.PushHandler.access$200(PushHandler.java:56)
at
com.vaadin.server.communication.PushHandler$1.onRequest(PushHandler.java:74)
at
org.atmosphere.cpr.AsynchronousProcessor.action(AsynchronousProcessor.java:174)
at
org.atmosphere.cpr.AsynchronousProcessor.suspended(AsynchronousProcessor.java:95)
at
org.atmosphere.container.Servlet30CometSupport.service(Servlet30CometSupport.java:66)
at
org.atmosphere.cpr.AtmosphereFramework.doCometSupport(AtmosphereFramework.java:1802)
at
org.atmosphere.websocket.DefaultWebSocketProcessor.dispatch(DefaultWebSocketProcessor.java:432)
at
org.atmosphere.websocket.DefaultWebSocketProcessor.open(DefaultWebSocketProcessor.java:186)
at org.atmosphere.container.JSR356Endpoint.onOpen(JSR356Endpoint.java:160)
at
org.apache.tomcat.websocket.server.WsHttpUpgradeHandler.init(WsHttpUpgradeHandler.java:138)
at
org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:687)
at
org.apache.coyote.http11.Http11NioProtocol$Http11ConnectionHandler.process(Http11NioProtocol.java:223)
at
org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1558)
at
org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.run(NioEndpoint.java:1515)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at
org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
at java.lang.Thread.run(Unknown Source)
Caused by: java.util.concurrent.ExecutionException: java.io.IOException: Key
must be cancelled
at
org.apache.tomcat.websocket.FutureToSendHandler.get(FutureToSendHandler.java:102)
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.startMessageBlock(WsRemoteEndpointImplBase.java:275)
... 31 more
Caused by: java.io.IOException: Key must be cancelled
at
org.apache.coyote.http11.upgrade.NioServletOutputStream.doWriteInternal(NioServletOutputStream.java:84)
at
org.apache.coyote.http11.upgrade.NioServletOutputStream.doWrite(NioServletOutputStream.java:61)
at
org.apache.coyote.http11.upgrade.AbstractServletOutputStream.writeInternal(AbstractServletOutputStream.java:165)
at
org.apache.coyote.http11.upgrade.AbstractServletOutputStream.write(AbstractServletOutputStream.java:132)
at
org.apache.tomcat.websocket.server.WsRemoteEndpointImplServer.onWritePossible(WsRemoteEndpointImplServer.java:99)
at
org.apache.tomcat.websocket.server.WsRemoteEndpointImplServer.doWrite(WsRemoteEndpointImplServer.java:81)
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.writeMessagePart(WsRemoteEndpointImplBase.java:450)
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.startMessage(WsRemoteEndpointImplBase.java:338)
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.startMessageBlock(WsRemoteEndpointImplBase.java:270)
... 31 more
Caused by: java.util.concurrent.ExecutionException:
at
org.apache.tomcat.websocket.FutureToSendHandler.get(FutureToSendHandler.java:102)
at
org.apache.tomcat.websocket.WsRemoteEndpointImplBase.startMessageBlock(WsRemoteEndpointImplBase.java:275)
at
org.apache.tomcat.websocket.WsSession.sendCloseMessage(WsSession.java:570)
at org.apache.tomcat.websocket.WsSession.doClose(WsSession.java:478)
at org.apache.tomcat.websocket.WsSession.close(WsSession.java:445)
at 

[Bug 57559] Decoded Request URI used for Asynchronous dispatch

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57559

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |WONTFIX

--- Comment #1 from Mark Thomas ma...@apache.org ---
The specification text you quoted skips over a few things. The main one is how
tricky it is to split an undecoded URI into contextPath, servletPath and
pathInfo. See the getContextPath() implementation in [1] for an idea of just
how messy this could get. I have no desire to see that sort of code in
something that is meant to be a convenience method.

Elsewhere in the Servlet spec (including for async) dispatches are handled in
terms of decoded paths relative to a context root. The convenience dispatch()
methods needs to be handled the same way to avoid a whole pile of unnecessary
complexity.

If you need the original request URI it is available via the usual request
attribute.

I'll raise this with the Servlet EG to see if the wording of this can be
changed/improved but - as far as Tomcat is concerned - this is a WONTFIX. Note
if the EG opt for the behavior you are asking for I'll re-open this issue.

I've added a test case that confirms the observed behavior.


[1]
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Request.java?view=annotate

-- 
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 57546] Memory Leak in SecureNioChannel

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57546

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Mark Thomas ma...@apache.org ---
This wasn't forgotten it deliberately wasn't back-ported because this isn't
user code in Tomcat 7. It is a Tomcat internal API only used by WebSockets and
we know exactly which exceptions are thrown. Therefore the additional
protection added in trunk and 8.0.x is not necessary in 7.0.x.

Regarding the debug, the look-up in the StringManager is likely to be much
faster than the throwing of the Exception that triggers this look-up.
Performance is pretty much a lost cause once an Exception has been 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



Re: async and thread constraint

2015-02-10 Thread Mark Thomas
On 10/02/2015 23:06, Konstantin Kolinko wrote:
 2015-02-10 14:16 GMT+03:00 Romain Manni-Bucau rmannibu...@gmail.com:
 Hi guys,

 in org.apache.coyote.AsyncStateMachine#asyncDispatch tomcat checks
 ContainerThreadMarker.isContainerThread()

 basically it prevents to use JAXRS 2 @Suspended without creating
 custom threads - which is not intended.
 
 I do not know what the above is about.

Me neither.

 Was this limitation intended or just to try to avoid to consume
 container threads?

In addition to the reason Konstantin explains below, it is also a
performance benefit. If we are already on a container thread and need to
process a dispatch() it is much more efficient to stay on that container
thread to process the dispatch() than it is to return the current
container thread to the pool, monkey about with the poller (or
equivalent for the connector in question) to trigger an event, wait for
the event to fire and poller to allocate the socket to a container
thread and then process the dispatch.

Mark


 
 Looking at blame results, those lines were added in
 http://svn.apache.org/r1594198
 Address root cause of ReadPendingException rather than swallowing it.
 
 Discussion that preceded that commit is in Re:r1593303 thread,
 http://tomcat.markmail.org/thread/4kwnhzwhhxi43sob
 
 quote
 I think I have got to the bottom of this. An async dispatch always
 adds the socket to the poller but if the dispatch occurs on a
 container thread then it will also be added to the poller when that
 thread exits the AbstractConnectionHandler.process() method.
 
 This looks like a bug with all the connectors but one that is more
 obvious with Nio2 because it triggers the ReadPendingException. Fixing
 this bug highlighted a minor issue on shutdown. I have a fix for that
 too. Commits to follow shortly.
 /quote
 
 Best regards,
 Konstantin Kolinko
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
 For additional commands, e-mail: dev-h...@tomcat.apache.org
 


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



[Bug 57489] java.util.concurrent.ExecutionException: java.io.IOException: Key must be cancelled

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57489

--- Comment #9 from Mark Thomas ma...@apache.org ---
OK. I've made various changes to 8.0.x that should at least make this less
likely and hopefully prevent. If you could test with the current 8.0.x and let
us know the results that would be great.

-- 
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 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #17 from mr.oliver.hernan...@gmail.com ---
I tend to agree more with Christopher, mainly because of this section of the
javadoc for the Valve interface that is misleading:

Consume bytes from the input stream associated with the Request, unless it is
completely generating the response, **OR WRAPPING THE REQUEST** before passing
it on.

When I read that, I had high hopes of implementing a custom valve easily.  Then
it was surprising that I could only find one example, Christopher's, of how to
wrap the request when a valve needs to consume data from the input stream, and
that Tomcat did not provide any wrapper class.  So I refactored an example
servlet filter from
http://architects.dzone.com/articles/debugging-soap-logging, and got it to work
as a valve.  The tricky part was figuring out that the HttpServletRequest is
doubly wrapped inside the org.apache.catalina.connector.Request class.

If the convenience of a built-in wrapper class will not be added, hopefully the
attached project will suffice for any others needing to do the same thing we
did.

-- 
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 57556] getServletContext().getRealPath(/) returns path not ending with /

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57556

--- Comment #2 from Mark Thomas ma...@apache.org ---
This will be the new resources implementation introduced in 8.0.x.

I don't see any reason not to restore the trailing separator for a directory
for backwards compatibility but I say that not having looked at the code.

-- 
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 45014] Request and Response classes should have wrappers for use in Valves

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45014

--- Comment #16 from mr.oliver.hernan...@gmail.com ---
Created attachment 32446
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=32446action=edit
Maven project for a valve that dumps the full request with the body

NOTE: For our use case, we needed a valve in JBoss, so this was coded against
the forked version of Tomcat within JBoss.  It should be simple to refactor for
use with the main trunk of Tomcat.

-- 
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 57534] CorsFilter.SIMPLE_HTTP_REQUEST_CONTENT_TYPE_VALUES check shall ignore content-type parameters

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57534

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 OS||All

--- Comment #1 from Mark Thomas ma...@apache.org ---
This has the potential to get expensive performance-wise. Parsing mime-types
from a content-type header is non-trivial but we do have
o.a.tomcat.util.http.parser.MediaType and MediaTypeCache

-- 
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 57534] CorsFilter.SIMPLE_HTTP_REQUEST_CONTENT_TYPE_VALUES check shall ignore content-type parameters

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57534

--- Comment #2 from Konstantin Kolinko knst.koli...@gmail.com ---
Parsing is needed when you need the value of a parameter (e.g. charset).

In this case just trimming at the first ';' is OK.

-- 
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 57559] Decoded Request URI used for Asynchronous dispatch

2015-02-10 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=57559

--- Comment #2 from Konstantin Kolinko knst.koli...@gmail.com ---
Previous discussion of this issue, ~8 months ago (June 2014):

Decoded URL set on asynchronous request
http://tomcat.markmail.org/thread/e33tqu7ayp2fguh4

-- 
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



  1   2   >