svn commit: r1665244 - in /tomcat/trunk: test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java webapps/docs/config/http.xml

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:10:57 2015
New Revision: 1665244

URL: http://svn.apache.org/r1665244
Log:
Correct statement on ordering preferences.
Add info on which OpenSSL branch the filtering tracks.

Modified:
tomcat/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
tomcat/trunk/webapps/docs/config/http.xml

Modified: 
tomcat/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java?rev=1665244r1=1665243r2=1665244view=diff
==
--- 
tomcat/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java 
(original)
+++ 
tomcat/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java 
Mon Mar  9 14:10:57 2015
@@ -58,8 +58,9 @@ public class TesterOpenSSL {
 Cipher.SSL2_RC4_128_EXPORT40_WITH_MD5)));
 
 static {
-// Note: The tests are configured for OpenSSL 1.1.0. Running with a
-//   different version is likely to trigger failures
+// Note: The tests are configured for the OpenSSL 1.1.0 development
+//   branch. Running with a different version is likely to trigger
+//   failures.
 String expected_version = 
System.getProperty(tomcat.test.openssl.version, );
 String versionString = null;
 try {

Modified: tomcat/trunk/webapps/docs/config/http.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/http.xml?rev=1665244r1=1665243r2=1665244view=diff
==
--- tomcat/trunk/webapps/docs/config/http.xml (original)
+++ tomcat/trunk/webapps/docs/config/http.xml Mon Mar  9 14:10:57 2015
@@ -1050,11 +1050,13 @@
   testing purposes only./p
   pThe list can also use ':' as a separator, in that case
   it will use the OpenSSL syntax (see OpenSSL documentation for the list
-  of ciphers supported and the syntax)./p
+  of ciphers supported and the syntax). The behaviour of this filtering is
+  kept aligned with the behaviour of the OpenSSL 1.1.0 development
+  branch./p
   pIf not specified, a default (using the OpenSSL notation) of
   codeHIGH:!aNULL:!eNULL:!EXPORT:!DES:!RC4:!MD5/code will be used./p
-  pNote that Java does treat the order in which ciphers are defined as an
-  order of preference./p
+  pNote that Java does not treat the order in which ciphers are defined 
as
+  an order of preference. See codeuseServerCipherSuitesOrder/code./p
 /attribute
 
 attribute name=clientAuth required=false



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



svn commit: r1665242 - /tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:04:51 2015
New Revision: 1665242

URL: http://svn.apache.org/r1665242
Log:
OpenSSL 1.0.2-dev includes EXPORT ciphers in COMPLEMENTOFDEFAULT

Modified:

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java?rev=1665242r1=1665241r2=1665242view=diff
==
--- 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 Mon Mar  9 14:04:51 2015
@@ -485,6 +485,7 @@ public class OpenSSLCipherConfigurationP
 complementOfDefault = filterByAuthentication(complementOfDefault, 
Collections.singleton(Authentication.aNULL));
 complementOfDefault.removeAll(aliases.get(eNULL));
 complementOfDefault.addAll(aliases.get(SSLv2));
+complementOfDefault.addAll(aliases.get(EXPORT));
 addListAlias(COMPLEMENTOFDEFAULT, complementOfDefault);
 }
 



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



svn commit: r1665241 - /tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:02:47 2015
New Revision: 1665241

URL: http://svn.apache.org/r1665241
Log:
Read test results the wrong way around.
OpenSSL 1.0.2-dev includes SSLv2 ciphers in COMPLEMENTOFDEFAULT

Modified:

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java?rev=1665241r1=1665240r2=1665241view=diff
==
--- 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 Mon Mar  9 14:02:47 2015
@@ -484,7 +484,7 @@ public class OpenSSLCipherConfigurationP
 SetCipher complementOfDefault = filterByKeyExchange(all, new 
HashSet(Arrays.asList(KeyExchange.EDH,KeyExchange.EECDH)));
 complementOfDefault = filterByAuthentication(complementOfDefault, 
Collections.singleton(Authentication.aNULL));
 complementOfDefault.removeAll(aliases.get(eNULL));
-complementOfDefault.removeAll(aliases.get(SSLv2));
+complementOfDefault.addAll(aliases.get(SSLv2));
 addListAlias(COMPLEMENTOFDEFAULT, complementOfDefault);
 }
 



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



svn commit: r1665245 - in /tomcat/tc8.0.x/trunk: test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java webapps/docs/changelog.xml webapps/docs/config/http.xml

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:14:33 2015
New Revision: 1665245

URL: http://svn.apache.org/r1665245
Log:
Correct statement on ordering preferences.
Add info on which OpenSSL branch the filtering tracks.

Modified:

tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc8.0.x/trunk/webapps/docs/config/http.xml

Modified: 
tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java?rev=1665245r1=1665244r2=1665245view=diff
==
--- 
tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/net/jsse/openssl/TesterOpenSSL.java
 Mon Mar  9 14:14:33 2015
@@ -72,8 +72,8 @@ public class TesterOpenSSL {
 
 
 static {
-// Note: The tests are configured for OpenSSL 1.0.2. Running with a
-//   different version is likely to trigger failures
+// Note: The tests are configured for OpenSSL 1.0.2 stable branch.
+//   Running with a different version is likely to trigger 
failures.
 String expected_version = 
System.getProperty(tomcat.test.openssl.version, );
 String versionString = null;
 try {

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=1665245r1=1665244r2=1665245view=diff
==
--- tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Mon Mar  9 14:14:33 2015
@@ -148,6 +148,10 @@
 body chunk larger than the socket write buffer is being written. This
 typically requires a larger than default AJP packetSize. (markt)
   /fix
+  update
+Align the OpenSSL syntax cipher configuration with the OpenSSL 1.0.2
+branch. (markt)
+  /update
 /changelog
   /subsection
   subsection name=Jasper

Modified: tomcat/tc8.0.x/trunk/webapps/docs/config/http.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/config/http.xml?rev=1665245r1=1665244r2=1665245view=diff
==
--- tomcat/tc8.0.x/trunk/webapps/docs/config/http.xml (original)
+++ tomcat/tc8.0.x/trunk/webapps/docs/config/http.xml Mon Mar  9 14:14:33 2015
@@ -1106,11 +1106,12 @@
   testing purposes only./p
   pThe list can also use ':' as a separator, in that case
   it will use the OpenSSL syntax (see OpenSSL documentation for the list
-  of ciphers supported and the syntax)./p
+  of ciphers supported and the syntax). The behaviour of this filtering is
+  kept aligned with the behaviour of the OpenSSL 1.0.2 stable branch./p
   pIf not specified, a default (using the OpenSSL notation) of
   codeHIGH:!aNULL:!eNULL:!EXPORT:!DES:!RC4:!MD5/code will be used./p
-  pNote that Java does treat the order in which ciphers are defined as an
-  order of preference./p
+  pNote that Java does not treat the order in which ciphers are defined 
as
+  an order of preference. See codeuseServerCipherSuitesOrder/code./p
 /attribute
 
 attribute name=clientAuth required=false



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



svn commit: r1665192 - /tomcat/tc7.0.x/trunk/build.xml

2015-03-09 Thread rjung
Author: rjung
Date: Mon Mar  9 11:28:57 2015
New Revision: 1665192

URL: http://svn.apache.org/r1665192
Log:
Try to fix gump build:

Guess java 7 home from JAVA_HOME if java.7.home
is not set. JAVA_HOME is only used, if the
current JVM is at least java 7.

This will not work correctly, if the current
JVM is not the one given by JAVA_HOME.

Unfortunately I didn't find a way to use (read)
an environment variable in a gump project file,
nor does the JVM or ant provide a way to get
the path to the currently running KDK (only JRE).

Modified:
tomcat/tc7.0.x/trunk/build.xml

Modified: tomcat/tc7.0.x/trunk/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/build.xml?rev=1665192r1=1665191r2=1665192view=diff
==
--- tomcat/tc7.0.x/trunk/build.xml (original)
+++ tomcat/tc7.0.x/trunk/build.xml Mon Mar  9 11:28:57 2015
@@ -31,6 +31,7 @@
   property file=${user.home}/build.properties/
   property file=build.properties/
   property file=build.properties.default/
+  property environment=env/
 
   !-- Project Name --
   property name=project   value=apache-tomcat /
@@ -451,17 +452,31 @@
   /patternset
 
   !--
-Verify availability of Java 7 JDK if one was configured.
-Fail if java.7.home was set, but points to a non-existing path.
+Guess path to Java 7 JDK from our JAVA_HOME if not already set
+and the current JVM is at least version 7.
+This will be wrong, if JAVA_HOME is set, but the current JVM
+is not the one given in JAVA_HOME.
   --
-  fail message=Invalid java.7.home setting: [${java.7.home}]. The path 
$${java.7.home}/bin/ does not exist.
-condition
-  and
-isset property=java.7.home /
-notavailable file=${java.7.home}/bin/ type=dir //not
-  /and
+  target name=guess-java7 unless=${java.7.home}
+condition property=java.7.home value=${env.JAVA_HOME}
+  matches string=${ant.java.version} pattern=^1\.([789]|[0-9][0-9]) /
 /condition
-  /fail
+  /target
+
+  target name=check-java7 depends=guess-java7
+!--
+  Verify availability of Java 7 JDK if one was configured.
+  Fail if java.7.home was set, but points to a non-existing path.
+--
+fail message=Invalid java.7.home setting: [${java.7.home}]. The path 
$${java.7.home}/bin/ does not exist.
+  condition
+and
+  isset property=java.7.home /
+  notavailable file=${java.7.home}/bin/ type=dir //not
+/and
+  /condition
+/fail
+  /target
 
   !-- === Build targets === 
--
 
@@ -609,7 +624,7 @@
 /copy
   /target
 
-  target name=compile depends=compile-java6,compile-java7 /
+  target name=compile depends=compile-java6,check-java7,compile-java7 /
 
   target name=compile-java6
   depends=build-prepare,download-compile,compile-prepare,validate
@@ -1378,7 +1393,7 @@
 available file=${test.apr.loc} property=apr.exists /
   /target
 
-  target name=test-init
+  target name=test-init depends=check-java7
 condition property=java.bin.path value=${java.7.home}/bin/
   isset property=java.7.home /
 /condition



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



svn commit: r1665222 - /tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 12:59:05 2015
New Revision: 1665222

URL: http://svn.apache.org/r1665222
Log:
Fix the 9.0.x version of BZ 57638. The assumption that a blocking read would 
fill the provided buffer was not valid.

Modified:
tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java

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=1665222r1=1665221r2=1665222view=diff
==
--- tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java Mon Mar  9 
12:59:05 2015
@@ -17,6 +17,7 @@
 package org.apache.coyote.ajp;
 
 import java.io.ByteArrayInputStream;
+import java.io.EOFException;
 import java.io.IOException;
 import java.io.InterruptedIOException;
 import java.net.InetAddress;
@@ -1517,8 +1518,17 @@ public class AjpProcessor extends Abstra
  */
 private boolean read(byte[] buf, int pos, int n, boolean block) throws 
IOException {
 int read = socketWrapper.read(block, buf, pos, n);
-if (!block  read  0  read  n) {
-socketWrapper.read(true, buf, pos + n, n - read);
+if (read  0  read  n) {
+int left = n - read;
+int start = pos + read;
+while (left  0) {
+read = socketWrapper.read(true, buf, start, left);
+if (read == -1) {
+throw new EOFException();
+}
+left = left - read;
+start = start + read;
+}
 }
 
 return read  0;



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



[Bug 57532] Session expire message sent to cluster nodes even with DeltaSession configuration not to

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57532

--- Comment #6 from andrew jardine andrew.jard...@jardineworks.com ---
Hey Mark,

Sorry for the delay -- other priorities came up. I'm trying this again right
now, though I think I tried that as well already. Could be though that I did
not have the session replication configured correctly when I tried it. If
memory serves, this is what we found (though I am double checking this morning)

1. We set the jvmRoute attribute on the Engine / envelope. Node 01 had a
value of jvmRoute=01, Node 02, a value of jvmRoute=02 etc.

2. We configured Apache to use a sticky session based on the jSessionID.

3. This worked and my first request when to Node 01 so I ended up with a
session id similar to :: ABCD1234567890-01

4. All my subsequent traffic was routed to the 01 server.

5. Shutdown 01.

6. Apache started directing my requests to 02 -- but my jSessionID was now
post-fixing the 02 jvmRoute value so I had -- ABCD1234567890-02 

7. Node 02 did not find a session with that ID, so it was creating a new
session. 

-- again, I could have not had the replication working properly perhaps so I'll
do some more testing today and update this ticket with anything I find.

-- 
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: r1665240 - /tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:00:02 2015
New Revision: 1665240

URL: http://svn.apache.org/r1665240
Log:
OpenSSL 1.0.2-dev now excludes SSLv2 ciphers from DEFAULT and 
COMPLEMENTOFDEFAULT

Modified:

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java?rev=1665240r1=1665239r2=1665240view=diff
==
--- 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 Mon Mar  9 14:00:02 2015
@@ -484,6 +484,7 @@ public class OpenSSLCipherConfigurationP
 SetCipher complementOfDefault = filterByKeyExchange(all, new 
HashSet(Arrays.asList(KeyExchange.EDH,KeyExchange.EECDH)));
 complementOfDefault = filterByAuthentication(complementOfDefault, 
Collections.singleton(Authentication.aNULL));
 complementOfDefault.removeAll(aliases.get(eNULL));
+complementOfDefault.removeAll(aliases.get(SSLv2));
 addListAlias(COMPLEMENTOFDEFAULT, complementOfDefault);
 }
 



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



svn commit: r1665238 - /tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 13:57:07 2015
New Revision: 1665238

URL: http://svn.apache.org/r1665238
Log:
OpenSSL 1.0.2-dev now excludes export ciphers from DEFAULT

Modified:

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java?rev=1665238r1=1665237r2=1665238view=diff
==
--- 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 (original)
+++ 
tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/net/jsse/openssl/OpenSSLCipherConfigurationParser.java
 Mon Mar  9 13:57:07 2015
@@ -479,7 +479,7 @@ public class OpenSSLCipherConfigurationP
 addListAlias(SRP, filterByKeyExchange(allCiphers, 
Collections.singleton(KeyExchange.SRP)));
 initialized = true;
 // Despite what the OpenSSL docs say, DEFAULT also excludes SSLv2
-addListAlias(DEFAULT, parse(ALL:!eNULL:!aNULL:!SSLv2));
+addListAlias(DEFAULT, parse(ALL:!EXPORT:!eNULL:!aNULL:!SSLv2));
 // COMPLEMENTOFDEFAULT is also not exactly as defined by the docs
 SetCipher complementOfDefault = filterByKeyExchange(all, new 
HashSet(Arrays.asList(KeyExchange.EDH,KeyExchange.EECDH)));
 complementOfDefault = filterByAuthentication(complementOfDefault, 
Collections.singleton(Authentication.aNULL));



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



svn commit: r1665251 - /tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:44:43 2015
New Revision: 1665251

URL: http://svn.apache.org/r1665251
Log:
Follow-up to r1665061
Correct fix for BZ57674

Modified:
tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java

Modified: tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java?rev=1665251r1=1665250r2=1665251view=diff
==
--- tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java 
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/coyote/ajp/AjpNio2Processor.java Mon 
Mar  9 14:44:43 2015
@@ -142,7 +142,7 @@ public class AjpNio2Processor extends Ab
 
 ByteBuffer writeBuffer =
 socketWrapper.getSocket().getBufHandler().getWriteBuffer();
-int toWrite = Math.min(length, writeBuffer.remaining());
+int toWrite = Math.min(length, writeBuffer.capacity());
 
 int result = 0;
 if (block) {



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



[Bug 57532] Session expire message sent to cluster nodes even with DeltaSession configuration not to

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57532

--- Comment #7 from andrew jardine andrew.jard...@jardineworks.com ---
Hey Mark,

UPDATE:

I am seeing the same behaviour, even with the JvmRouteBinderValve. At first I
enabled the jvmRoute, but that was problematic because, as previusly mentioned,
the route was post fixed to the sessionid. I removed the jvmRoute, but LEFT the
JvmRouteBinderValve and restarted everything.

Session replication works. I have my 3 nodes behind an apache proxy that does
RR load balancing to each of the nodes. When I shut down a node from the
cluster, sometimes the session remains, other times it is lost. It appears,
again, as though the primary node shut down is the problem.

For my requirements, the only time I would want a session expiration to be
broadcast across the cluster is when a user logs out. The application handles
this for me already so for Tomcat, the application server need only kill its
own list of sessions on shut down. Perhaps my scenario is something that is
outside the norm of what others/tomcat expects. Either way, the only solution I
have found to date is to modify that method so that it does not default to
broadcasting across the cluster.

-- 
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: r1665062 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/coyote/ajp/AjpNioProcessor.java test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java webapps/docs/changelog.xml

2015-03-09 Thread Christopher Schultz
Mark,

On 3/8/15 1:45 PM, ma...@apache.org wrote:
 Author: markt
 Date: Sun Mar  8 17:45:44 2015
 New Revision: 1665062
 
 URL: http://svn.apache.org/r1665062
 Log:
 Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57674
 Avoid a BufferOverflowException when an AJP response body chunk larger than 
 the socket write buffer is being written. This typically requires a larger 
 than default AJP packetSize.
 
 Modified:
 tomcat/tc7.0.x/trunk/   (props changed)
 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 
 tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java
 tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
 
 Propchange: tomcat/tc7.0.x/trunk/
 --
 --- svn:mergeinfo (original)
 +++ svn:mergeinfo Sun Mar  8 17:45: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,1647043,1648816,1651420-1651422,1651844,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872
 +/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,1647043,1648816,1651420-1651422,1651844,1652939-1652940,1652973,1653798,1653817,1653841,1654042,1654161,1654736,1654767,1654787,1656592,1662986,1663265,1663278,1663325,1663535,1663567,1663679,1663997,1664175,1664321,1664872,1665061
  
 /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,124011
4
  
 ,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,134
2
  
 

[Bug 57674] BufferOverflowException in AjpNioProcessor when writing content larger than the underlying buffer

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57674

Christopher Schultz ch...@christopherschultz.net changed:

   What|Removed |Added

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

--- Comment #2 from Christopher Schultz ch...@christopherschultz.net ---
I think there is something missing with this fix. I get no errors on the Tomcat
side, now, but mod_jk certainly isn't happy:

[Mon Mar 09 11:38:55.820 2015] [7087:140653078521600] [error]
ajp_connection_tcp_get_message::jk_ajp_common.c (1372): (worker21) wrong
message format 0x6f6e from 127.0.0.1:8215
[Mon Mar 09 11:38:55.824 2015] [7087:140653078521600] [error]
ajp_get_reply::jk_ajp_common.c (2289): (worker21) Tomcat is down or network
problems. Part of the response has already been sent to the client
[Mon Mar 09 11:38:55.824 2015] [7087:140653078521600] [info]
ajp_service::jk_ajp_common.c (2773): (worker21) sending request to tomcat
failed (recoverable), because of protocol error (attempt=1)
[Mon Mar 09 11:38:56.287 2015] [7087:140653078521600] [error]
ajp_connection_tcp_get_message::jk_ajp_common.c (1372): (worker21) wrong
message format 0x6f6e from 127.0.0.1:8215
[Mon Mar 09 11:38:56.299 2015] [7087:140653078521600] [error]
ajp_get_reply::jk_ajp_common.c (2289): (worker21) Tomcat is down or network
problems. Part of the response has already been sent to the client
[Mon Mar 09 11:38:56.299 2015] [7087:140653078521600] [info]
ajp_service::jk_ajp_common.c (2773): (worker21) sending request to tomcat
failed (recoverable), because of protocol error (attempt=2)
[Mon Mar 09 11:38:56.299 2015] [7087:140653078521600] [error]
ajp_service::jk_ajp_common.c (2794): (worker21) connecting to tomcat failed
(rc=-11, errors=6, client_errors=0).
[Mon Mar 09 11:38:56.299 2015] [7087:140653078521600] [info]
jk_handler::mod_jk.c (2991): Service error=-11 for worker=worker21


From my earlier instrumentation, I'm getting log messages telling me what is
being send to the output() method. I'm including them in case they are helpful:

WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=65536, starting at offset=0, len=8200
Mar 09, 2015 11:38:56 AM org.apache.coyote.ajp.AjpNioProcessor output
WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=65536, starting at offset=0, len=8200
Mar 09, 2015 11:38:56 AM org.apache.coyote.ajp.AjpNioProcessor output
WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=65536, starting at offset=0, len=8200
Mar 09, 2015 11:38:56 AM org.apache.coyote.ajp.AjpNioProcessor output
WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=65536, starting at offset=0, len=7240
Mar 09, 2015 11:38:56 AM org.apache.coyote.ajp.AjpNioProcessor output
WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=8, starting at offset=0, len=8
Mar 09, 2015 11:38:56 AM org.apache.coyote.ajp.AjpNioProcessor output
WARNING: Writing to output buffer of capacity=8192, position=0, limit=8192,
remaining=8192 from source buffer of size=6, starting at offset=0, len=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: r1665296 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/tomcat/websocket/server/LocalStrings.properties java/org/apache/tomcat/websocket/server/WsServerContainer.java webapps/docs/changelo

2015-03-09 Thread schultz
Author: schultz
Date: Mon Mar  9 16:29:15 2015
New Revision: 1665296

URL: http://svn.apache.org/r1665296
Log:
Back-port r1665292 to fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57676
- Report conflicting WebSocket endpoint classes when there is a path conflict.

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

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties

tomcat/tc8.0.x/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc8.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Mar  9 16:29:15 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,1658804,1658833,1658840,1658966,1659043,1659053,1659059,1659188-1659189,1659216,1659263,1659293,1659304,1659306-1659307,1659382,1659384,1659428,1659471,1659486,1659505,1659516,1659521,1659524,1659559,1659562,1659803,1659806,1659814,1659833,1659862,1659905,1659919,1659948,1659967,1659983-1659984,1660060,1660074,1660077,1660133,1660168,1660331-1660332,1660353,1660358,1660924,1661386,1661867,1661972,1661990,1662200,1662308-1662309,1662548,1662614,1662736,1662985,1662988-1662989,1663264,1663277,1663298,1663324,1663534,1663562,1663676,1663715,1663754,1663768,1663772,1663781,1663893,1663995,1664143,1664163,1664174,1664301,1664317,1664347,1664657,1664659,1664710,1664863-1664864,1664866,1665085
+/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
 

svn commit: r1665252 - /tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 14:48:06 2015
New Revision: 1665252

URL: http://svn.apache.org/r1665252
Log:
Fix back-port error in unit test

Modified:

tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java?rev=1665252r1=1665251r2=1665252view=diff
==
--- 
tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/test/org/apache/coyote/ajp/TestAbstractAjpProcessor.java 
Mon Mar  9 14:48:06 2015
@@ -661,7 +661,7 @@ public class TestAbstractAjpProcessor ex
 tomcat.getConnector().setProperty(packetSize, 
Integer.toString(ajpPacketSize));
 
 // No file system docBase required
-Context ctx = tomcat.addContext(, null);
+Context ctx = tomcat.addContext(, 
System.getProperty(java.io.tmpdir));
 
 FixedResponseSizeServlet servlet = new FixedResponseSizeServlet(15000, 
16000);
 Tomcat.addServlet(ctx, FixedResponseSizeServlet, servlet);



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



Impossible branch in o.a.t.websocket.server.WsServerContainer.addEndpoint

2015-03-09 Thread Christopher Schultz
All,

I was looking at https://bz.apache.org/bugzilla/show_bug.cgi?id=57676 (a
simple proposed patch to improve an error message) and I was trying to
figure out what to do with a bit of code in addEndpoint. Reading the
code, I think I've spotted a problem:

[187]   UriTemplate uriTemplate = new UriTemplate(path);
if (uriTemplate.hasParameters()) {
Integer key = Integer.valueOf(uriTemplate.getSegmentCount());
SortedSetTemplatePathMatch templateMatches =
configTemplateMatchMap.get(key);
if (templateMatches == null) {
// Ensure that if concurrent threads execute this block they
// both end up using the same TreeSet instance
templateMatches = new TreeSet(
TemplatePathMatchComparator.getInstance());
configTemplateMatchMap.putIfAbsent(key, templateMatches);
templateMatches = configTemplateMatchMap.get(key);
}
[200]   if (!templateMatches.add(new TemplatePathMatch(sec,
uriTemplate))) {
// Duplicate uriTemplate;
throw new DeploymentException(
sm.getString(serverContainer.duplicatePaths, path,
 sec.getEndpointClass(),
 sec.getEndpointClass()));
}
}

The problem is on line 200, where we check to see if
templateMatches.add() returns false (meaning that the object we added to
the set replaced one that was already there).

I don't believe this branch can /ever/ be reached because the
TemplatePathMatch class doesn't override Object.equals(), thus no two
objects of that type will ever equal each other. Therefore, a new
TemplatePathMatch object will always be added to the set.

I'm not sure what the implications are of multiple TemplatePathMatch
objects being in that set, but it looks like it's bad (we throw an
exception in that case), so someone with a better understanding of such
things might want to take a look.

(Also, if anyone would care to comment on how to get the pre-existing
match for the error message -- which is different than current-trunk as
it's what I'm working on -- I'd love some insight.)

Thanks,
-chris



signature.asc
Description: OpenPGP digital signature


svn commit: r1665292 - in /tomcat/trunk/java/org/apache/tomcat/websocket/server: LocalStrings.properties WsServerContainer.java

2015-03-09 Thread schultz
Author: schultz
Date: Mon Mar  9 16:21:15 2015
New Revision: 1665292

URL: http://svn.apache.org/r1665292
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57676
Improve error message when multiple Websocket endpoints are bound to the same 
path.

Modified:
tomcat/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties
tomcat/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java

Modified: 
tomcat/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties?rev=1665292r1=1665291r2=1665292view=diff
==
--- 
tomcat/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties 
(original)
+++ 
tomcat/trunk/java/org/apache/tomcat/websocket/server/LocalStrings.properties 
Mon Mar  9 16:21:15 2015
@@ -14,7 +14,7 @@
 # limitations under the License.
 serverContainer.addNotAllowed=No further Endpoints may be registered once an 
attempt has been made to use one of the previously registered endpoints
 serverContainer.configuratorFail=Failed to create configurator of type [{0}] 
for POJO of type [{1}]
-serverContainer.duplicatePaths=Multiple Endpoints may not be deployed to the 
same path [{0}]
+serverContainer.duplicatePaths=Multiple Endpoints may not be deployed to the 
same path [{0}] : existing endpoint was {1} and new endpoint is {2}
 serverContainer.encoderFail=Unable to create encoder of type [{0}]
 serverContainer.endpointDeploy=Endpoint class [{0}] deploying to path [{1}] in 
ServletContext [{2}]
 serverContainer.missingAnnotation=Cannot deploy POJO class [{0}] as it is not 
annotated with @ServerEndpoint

Modified: 
tomcat/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java?rev=1665292r1=1665291r2=1665292view=diff
==
--- tomcat/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/server/WsServerContainer.java 
Mon Mar  9 16:21:15 2015
@@ -200,7 +200,9 @@ public class WsServerContainer extends W
 if (!templateMatches.add(new TemplatePathMatch(sec, uriTemplate))) 
{
 // Duplicate uriTemplate;
 throw new DeploymentException(
-sm.getString(serverContainer.duplicatePaths, path));
+sm.getString(serverContainer.duplicatePaths, path,
+ sec.getEndpointClass(),
+ sec.getEndpointClass()));
 }
 } else {
 // Exact match
@@ -208,7 +210,9 @@ public class WsServerContainer extends W
 if (old != null) {
 // Duplicate path mappings
 throw new DeploymentException(
-sm.getString(serverContainer.duplicatePaths, path));
+sm.getString(serverContainer.duplicatePaths, path,
+ old.getEndpointClass(),
+ sec.getEndpointClass()));
 }
 }
 



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



[Bug 57674] BufferOverflowException in AjpNioProcessor when writing content larger than the underlying buffer

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57674

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

   What|Removed |Added

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

-- 
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: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 16:32:43 2015
New Revision: 1665297

URL: http://svn.apache.org/r1665297
Log:
Follow up to r1665062
Callers assume that output() will fully write the data that is passed to it. 
Ensure that this is the case when the AJP message size is larger than the 
output buffer.

Modified:
tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

Modified: tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java?rev=1665297r1=1665296r2=1665297view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java Mon 
Mar  9 16:32:43 2015
@@ -302,24 +302,29 @@ public class AjpNioProcessor extends Abs
 ByteBuffer writeBuffer =
 socketWrapper.getSocket().getBufHandler().getWriteBuffer();
 
-int toWrite = Math.min(length, writeBuffer.remaining());
-writeBuffer.put(src, offset, toWrite);
-
-writeBuffer.flip();
-
-long writeTimeout = att.getWriteTimeout();
-Selector selector = null;
-try {
-selector = pool.get();
-} catch ( IOException x ) {
-//ignore
-}
-try {
-pool.write(writeBuffer, socketWrapper.getSocket(), selector,
-writeTimeout, true);
-}finally { 
-writeBuffer.clear();
-if ( selector != null ) pool.put(selector);
+int left = length;
+int written = 0;
+while (left  0) {
+int toWrite = Math.min(left, writeBuffer.remaining());
+writeBuffer.put(src, offset, toWrite);
+
+writeBuffer.flip();
+
+long writeTimeout = att.getWriteTimeout();
+Selector selector = null;
+try {
+selector = pool.get();
+} catch ( IOException x ) {
+//ignore
+}
+try {
+written = pool.write(writeBuffer, socketWrapper.getSocket(),
+selector, writeTimeout, true);
+} finally { 
+writeBuffer.clear();
+if ( selector != null ) pool.put(selector);
+}
+left -= written;
 }
 }
 



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



[Bug 57676] WebSocket check same path endpoint implementation enhancement

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57676

Christopher Schultz ch...@christopherschultz.net changed:

   What|Removed |Added

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

--- Comment #1 from Christopher Schultz ch...@christopherschultz.net ---
Fixed in trunk in r1665292 and Tomcat 8.0.x in r1665296.
Will be in Tomcat 8.0.21.

-- 
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 57676] WebSocket check same path endpoint implementation enhancement

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57676

--- Comment #2 from Christopher Schultz ch...@christopherschultz.net ---
Note that I do not intend to back-port this to Tomcat 7 unless someone requests
it.

-- 
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: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread Mark Thomas
On 9 March 2015 16:35:57 GMT+00:00, Christopher Schultz 
ch...@christopherschultz.net wrote:
Mark,

On 3/9/15 12:32 PM, ma...@apache.org wrote:
 Author: markt
 Date: Mon Mar  9 16:32:43 2015
 New Revision: 1665297
 
 URL: http://svn.apache.org/r1665297
 Log:
 Follow up to r1665062
 Callers assume that output() will fully write the data that is passed
to it. Ensure that this is the case when the AJP message size is larger
than the output buffer.
 
 Modified:

tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 
 Modified:
tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 URL:
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java?rev=1665297r1=1665296r2=1665297view=diff

==
 ---
tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
(original)
 +++
tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
Mon Mar  9 16:32:43 2015
 @@ -302,24 +302,29 @@ public class AjpNioProcessor extends Abs
  ByteBuffer writeBuffer =
 
socketWrapper.getSocket().getBufHandler().getWriteBuffer();
  
 -int toWrite = Math.min(length, writeBuffer.remaining());
 -writeBuffer.put(src, offset, toWrite);
 -
 -writeBuffer.flip();
 -
 -long writeTimeout = att.getWriteTimeout();
 -Selector selector = null;
 -try {
 -selector = pool.get();
 -} catch ( IOException x ) {
 -//ignore
 -}
 -try {
 -pool.write(writeBuffer, socketWrapper.getSocket(),
selector,
 -writeTimeout, true);
 -}finally { 
 -writeBuffer.clear();
 -if ( selector != null ) pool.put(selector);
 +int left = length;
 +int written = 0;
 +while (left  0) {
 +int toWrite = Math.min(left, writeBuffer.remaining());
 +writeBuffer.put(src, offset, toWrite);
 +
 +writeBuffer.flip();
 +
 +long writeTimeout = att.getWriteTimeout();
 +Selector selector = null;
 +try {
 +selector = pool.get();
 +} catch ( IOException x ) {
 +//ignore
 +}
 +try {
 +written = pool.write(writeBuffer,
socketWrapper.getSocket(),
 +selector, writeTimeout, true);
 +} finally { 
 +writeBuffer.clear();
 +if ( selector != null ) pool.put(selector);
 +}
 +left -= written;
  }
  }

Okay, yes, this was more like what I was expecting to see. I'll give it
another try.

Glad to see it is working now. The next jobs are a) documenting the expected 
behavior re full vs partial write and b) take a look at trying to reduce the 
number of array copies (or equivalent) that the I/O codes does.

Mark

Thanks!
-chris



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



Re: svn commit: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread Christopher Schultz
Mark,

On 3/9/15 12:32 PM, ma...@apache.org wrote:
 Author: markt
 Date: Mon Mar  9 16:32:43 2015
 New Revision: 1665297
 
 URL: http://svn.apache.org/r1665297
 Log:
 Follow up to r1665062
 Callers assume that output() will fully write the data that is passed to it. 
 Ensure that this is the case when the AJP message size is larger than the 
 output buffer.
 
 Modified:
 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 
 Modified: tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 URL: 
 http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java?rev=1665297r1=1665296r2=1665297view=diff
 ==
 --- tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java 
 (original)
 +++ tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java Mon 
 Mar  9 16:32:43 2015
 @@ -302,24 +302,29 @@ public class AjpNioProcessor extends Abs
  ByteBuffer writeBuffer =
  socketWrapper.getSocket().getBufHandler().getWriteBuffer();
  
 -int toWrite = Math.min(length, writeBuffer.remaining());
 -writeBuffer.put(src, offset, toWrite);
 -
 -writeBuffer.flip();
 -
 -long writeTimeout = att.getWriteTimeout();
 -Selector selector = null;
 -try {
 -selector = pool.get();
 -} catch ( IOException x ) {
 -//ignore
 -}
 -try {
 -pool.write(writeBuffer, socketWrapper.getSocket(), selector,
 -writeTimeout, true);
 -}finally { 
 -writeBuffer.clear();
 -if ( selector != null ) pool.put(selector);
 +int left = length;
 +int written = 0;
 +while (left  0) {
 +int toWrite = Math.min(left, writeBuffer.remaining());
 +writeBuffer.put(src, offset, toWrite);
 +
 +writeBuffer.flip();
 +
 +long writeTimeout = att.getWriteTimeout();
 +Selector selector = null;
 +try {
 +selector = pool.get();
 +} catch ( IOException x ) {
 +//ignore
 +}
 +try {
 +written = pool.write(writeBuffer, socketWrapper.getSocket(),
 +selector, writeTimeout, true);
 +} finally { 
 +writeBuffer.clear();
 +if ( selector != null ) pool.put(selector);
 +}
 +left -= written;
  }
  }

Okay, yes, this was more like what I was expecting to see. I'll give it
another try.

Thanks!
-chris



signature.asc
Description: OpenPGP digital signature


[Bug 57674] BufferOverflowException in AjpNioProcessor when writing content larger than the underlying buffer

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57674

--- Comment #4 from Christopher Schultz ch...@christopherschultz.net ---
This is looking much better.

I was fortunate that I could reproduce this very easily in my application and
we are humming-along quite smoothly, now.

-- 
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 57674] BufferOverflowException in AjpNioProcessor when writing content larger than the underlying buffer

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57674

--- Comment #3 from Christopher Schultz ch...@christopherschultz.net ---
Thanks for the fixes, Mark.

I probably could have done these changes myself, but I wasn't sure of the
implications of making changes in these areas.

-- 
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 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

--- Comment #13 from Mark Thomas ma...@apache.org ---
No. It requires Java 7 and Tomcat 7 has to run on Java 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



Re: svn commit: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread Christopher Schultz
Mark,

On 3/9/15 4:26 PM, Mark Thomas wrote:
 On 09/03/2015 18:34, Christopher Schultz wrote:
 Mark,
 
 Glad to see it is working now. The next jobs are a) documenting the
 expected behavior re full vs partial write and b) take a look at trying
 to reduce the number of array copies (or equivalent) that the I/O codes
 does.

 I'm not sure it's really worth it to fix, at least not for Tomcat 7/8.
 I'd focus on Tomcat 9 where you guys have been doing good work
 
 Agreed that is the focus. I haven't really formed a firm opinion on
 what, if anything, should be back-ported.
 
 
 (except
 for removing BIO! Boo! Hiss! ... yeah, I know why and I agree but I'm
 still not terribly happy about it).
 
 As I have said before, if you can come up with a BIO implementation that
 works with the new API, I wouldn't object to adding BIO support back.

I think you and I both know that if you can't conceive of such an
implementation with more than 3 seconds of thought, I don't have a
prayer of designing and implementing it for quite some time.

:)

-chris



signature.asc
Description: OpenPGP digital signature


[Bug 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

--- Comment #14 from Christopher Schultz ch...@christopherschultz.net ---
(In reply to Mark Thomas from comment #13)
 No. It requires Java 7 and Tomcat 7 has to run on Java 6.

Could we wrap some of this capability in Jre7Compat.isSupported()?

Lots of folks are running Tomcat 7 on Java 7+. Java 6 is EOL so ... pretty much
everybody should have moved up by now.

-- 
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: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread Mark Thomas
On 09/03/2015 18:34, Christopher Schultz wrote:
 Mark,

 Glad to see it is working now. The next jobs are a) documenting the
 expected behavior re full vs partial write and b) take a look at trying
 to reduce the number of array copies (or equivalent) that the I/O codes
 does.
 
 I'm not sure it's really worth it to fix, at least not for Tomcat 7/8.
 I'd focus on Tomcat 9 where you guys have been doing good work

Agreed that is the focus. I haven't really formed a firm opinion on
what, if anything, should be back-ported.


 (except
 for removing BIO! Boo! Hiss! ... yeah, I know why and I agree but I'm
 still not terribly happy about it).

As I have said before, if you can come up with a BIO implementation that
works with the new API, I wouldn't object to adding BIO support back.

Mark

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



[Bug 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

--- Comment #12 from kotur...@amazon.com kotur...@amazon.com ---
Hi, is it possible to port this change to Tomcat 7 in
/tomcat/tc7.0.x/trunk/java/org/apache/catalina/loader/WebappClassLoader.java ?

-- 
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: r1665297 - /tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

2015-03-09 Thread Christopher Schultz
Mark,

On 3/9/15 1:23 PM, Mark Thomas wrote:
 On 9 March 2015 16:35:57 GMT+00:00, Christopher Schultz 
 ch...@christopherschultz.net wrote:
 Mark,

 On 3/9/15 12:32 PM, ma...@apache.org wrote:
 Author: markt
 Date: Mon Mar  9 16:32:43 2015
 New Revision: 1665297

 URL: http://svn.apache.org/r1665297
 Log:
 Follow up to r1665062
 Callers assume that output() will fully write the data that is passed
 to it. Ensure that this is the case when the AJP message size is larger
 than the output buffer.

 Modified:

 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java

 Modified:
 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 URL:
 http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java?rev=1665297r1=1665296r2=1665297view=diff

 ==
 ---
 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 (original)
 +++
 tomcat/tc7.0.x/trunk/java/org/apache/coyote/ajp/AjpNioProcessor.java
 Mon Mar  9 16:32:43 2015
 @@ -302,24 +302,29 @@ public class AjpNioProcessor extends Abs
  ByteBuffer writeBuffer =
 
 socketWrapper.getSocket().getBufHandler().getWriteBuffer();
  
 -int toWrite = Math.min(length, writeBuffer.remaining());
 -writeBuffer.put(src, offset, toWrite);
 -
 -writeBuffer.flip();
 -
 -long writeTimeout = att.getWriteTimeout();
 -Selector selector = null;
 -try {
 -selector = pool.get();
 -} catch ( IOException x ) {
 -//ignore
 -}
 -try {
 -pool.write(writeBuffer, socketWrapper.getSocket(),
 selector,
 -writeTimeout, true);
 -}finally { 
 -writeBuffer.clear();
 -if ( selector != null ) pool.put(selector);
 +int left = length;
 +int written = 0;
 +while (left  0) {
 +int toWrite = Math.min(left, writeBuffer.remaining());
 +writeBuffer.put(src, offset, toWrite);
 +
 +writeBuffer.flip();
 +
 +long writeTimeout = att.getWriteTimeout();
 +Selector selector = null;
 +try {
 +selector = pool.get();
 +} catch ( IOException x ) {
 +//ignore
 +}
 +try {
 +written = pool.write(writeBuffer,
 socketWrapper.getSocket(),
 +selector, writeTimeout, true);
 +} finally { 
 +writeBuffer.clear();
 +if ( selector != null ) pool.put(selector);
 +}
 +left -= written;
  }
  }

 Okay, yes, this was more like what I was expecting to see. I'll give it
 another try.
 
 Glad to see it is working now. The next jobs are a) documenting the
 expected behavior re full vs partial write and b) take a look at trying
 to reduce the number of array copies (or equivalent) that the I/O codes
 does.

I'm not sure it's really worth it to fix, at least not for Tomcat 7/8.
I'd focus on Tomcat 9 where you guys have been doing good work (except
for removing BIO! Boo! Hiss! ... yeah, I know why and I agree but I'm
still not terribly happy about it).

Definitely reducing otherwise useless copying is good, but don't kill
yourself over it in stable versions unless there is a specific
complaint. There are too many opportunities for breakage. I'm actually
shocked that nobody found this problem before. I guess not that many
people use mod_jk, or maybe most of them use it with the default packet
size.

I only found this because of two events that occurred very far apart in
time:

1a. I was doing testing on client certificate handling ... years ago.
Client certificates tend to grow large and since the whole request
headers must fit into a single AJP packet, I had to increase
max_packet_size in mod_jk. I did this for my template worker, to
affect all workers that derive from it.
1b. I changed my client certificate test application to use a matching
max packet size to 1a.

2. Recent use of GWT had started sending rather large payloads back and
forth to our servers. (Yes, I'm also curious why GWT, supposedly to
improve performance, is now sending more data to and from the server
than just refreshing the page... but where was I?) This notified us that
the packetSize in Tomcat was incorrect.

Adjusting the packetSize in Tomcat immediately caused all hell to break
loose.

Well, at any rate, I'm glad it's fixed, now.

Thanks,
-chris



signature.asc
Description: OpenPGP digital signature


[Bug 57681] New: Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

Bug ID: 57681
   Summary: Allow parallel class loading in web application class
loader by synchronizing on class specific object
   Product: Tomcat 7
   Version: trunk
  Hardware: PC
Status: NEW
  Severity: trivial
  Priority: P2
 Component: Catalina
  Assignee: dev@tomcat.apache.org
  Reporter: kotur...@amazon.com

Created attachment 32553
  -- https://bz.apache.org/bugzilla/attachment.cgi?id=32553action=edit
Parallel classloading port from Tc8.0 into Tc7.0

Related to fix in Tomcat8.0:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=56530
Add a web application class loader implementation that supports the parallel
loading of web application classes.

Source file to patch:
https://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/loader/WebappClassLoader.java?revision=1661811view=markup

-- 
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 57681] Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

--- Comment #2 from Mark Thomas ma...@apache.org ---
And for the record the patch is nowhere close to correct.

-- 
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 57681] Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

--- Comment #3 from Alex Koturanov kotur...@amazon.com ---
The fix in bug 56530 is for Tomcat 8, not Tomcat 7. Why my request is marked as
resolved?

-- 
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 57681] Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

--- Comment #4 from Mark Thomas ma...@apache.org ---
Because we don't have separate Bugzilla entries for exactly the same issue in
different versions 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 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

--- Comment #17 from Alex Koturanov kotur...@amazon.com ---
Please consider not only the performance impact but also stability of web
application. I am seeing 2-10 sec pauses for all requests in my web application
caused by some bad code in just one 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 57681] Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

Alex Koturanov kotur...@amazon.com changed:

   What|Removed |Added

 CC||kotur...@amazon.com
 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



[Bug 57681] Allow parallel class loading in web application class loader by synchronizing on class specific object

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681

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

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #1 from Mark Thomas ma...@apache.org ---


*** This bug has been marked as a duplicate of bug 56530 ***

-- 
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 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

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

   What|Removed |Added

 CC||kotur...@amazon.com

--- Comment #16 from Mark Thomas ma...@apache.org ---
*** Bug 57681 has been marked as a duplicate of this bug. ***

-- 
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 56530] WebappClassLoader.loadClass synchronization issue due to coarse lock at WebappClassLoader instance level instead of been by class name

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56530

--- Comment #15 from Mark Thomas ma...@apache.org ---
Oracle's public support has ended for Java 6 but other vendors may have
different policies and if you pay Oracle $ you can still get support, updates
etc. (and lots of companies do).

Using JreCompat looks doable. The two key questions would be how invasive is
the patch and what is the impact on performance.

-- 
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-trunk-test-apr (in module tomcat-trunk) failed

2015-03-09 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-trunk-test-apr has an issue affecting its community integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-test-apr :  Tomcat 9.x, a web server implementing the Java 
Servlet 4.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-test-apr/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -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.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/logs-APR
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/test-tmp-APR/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-test-apr/gump_work/build_tomcat-trunk_tomcat-trunk-test-apr.html
Work Name: build_tomcat-trunk_tomcat-trunk-test-apr (Type: Build)
Work ended in a state of : Failed
Elapsed: 52 mins 26 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 
-Dobjenesis.jar=/srv/gump/public/workspace/objenesis/main/target/objenesis-2.2-SNAPSHOT.jar
 -Dtest.reports=output/logs-APR 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dexamples.sources.skip=true 
-Djdt.jar=/srv/gump/packages/eclipse/plugins/R-4.4-201406061215/ecj-4.4.jar 
-Dtest.apr.loc=/srv/gump/public/workspace/tomcat-native/dest-20150309/lib 
-Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20150309.jar
 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dtest.temp=output/test-tmp-APR -Dtest.accesslog=true -Dexecute.test.nio=false 
-Dtest
 .openssl.path=/srv/gump/public/workspace/openssl/dest-20150309/bin/openssl 
-Dexecute.test.apr=true -Dexecute.test.nio2=false 
-Deasymock.jar=/srv/gump/public/workspace/easymock/easymock/target/easymock-3.4-SNAPSHOT.jar
 -Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar 
-Dcglib.jar=/srv/gump/packages/cglib/cglib-nodep-2.2.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-trunk/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-trunk/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/servlet-api.ja
 
r:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/websocket-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-storeconfig.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ha.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-jni.jar:/srv/gump/public/workspace/tomcat-trunk/output/bu
 
ild/lib/tomcat-spdy.jar:/srv

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

2015-03-09 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 4 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
 -WARNING- No directory 
[/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
 -WARNING- No directory 
[/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: 48 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-20150309-native-src.tar.gz
 -Dtest.reports=output/logs-NIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-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-20150309.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-20150309.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
 
r:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-7.0.x/output

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

2015-03-09 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 4 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
 -WARNING- No directory 
[/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
 -WARNING- No directory 
[/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: 57 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-20150309-native-src.tar.gz
 -Dtest.reports=output/logs-BIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-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-20150309.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-20150309.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
 
r:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-7.0.x/output

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

2015-03-09 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-test-nio has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 2 runs.
The current state of this project is 'Failed', with reason 'Build Timed Out'.
For reference only, the following projects are affected by this:
- tomcat-tc8.0.x-test-nio :  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-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 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.
 -INFO- Failed with reason build timed out
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-8.0.x/output/logs-NIO
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-8.0.x/output/test-tmp-NIO/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-test-nio/gump_work/build_tomcat-8.0.x_tomcat-tc8.0.x-test-nio.html
Work Name: build_tomcat-8.0.x_tomcat-tc8.0.x-test-nio (Type: Build)
Work ended in a state of : Failed
Elapsed: 1 hour 2 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 
-Dobjenesis.jar=/srv/gump/public/workspace/objenesis/main/target/objenesis-2.2-SNAPSHOT.jar
 -Dtest.reports=output/logs-NIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dexamples.sources.skip=true 
-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-20150309.jar
 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dtest.temp=output/test-tmp-NIO -Dtest.accesslog=true -Dexecute.test.nio=true 
-Dtest.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20150309/bin/op
 enssl -Dexecute.test.apr=false -Dexecute.test.bio=false 
-Dexecute.test.nio2=false 
-Deasymock.jar=/srv/gump/public/workspace/easymock/easymock/target/easymock-3.4-SNAPSHOT.jar
 -Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar 
-Dcglib.jar=/srv/gump/packages/cglib/cglib-nodep-2.2.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-8.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-8.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-8.0.x/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/servlet-api.ja
 
r:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/websocket-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-storeconfig.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-ha.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-jni.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/bu
 
ild/lib/tomcat

[Bug 57676] WebSocket check same path endpoint implementation enhancement

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57676

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

   What|Removed |Added

 OS||All
   Severity|normal  |enhancement
Summary|A small suggestion: |WebSocket check same path
   |WebSocket check same path   |endpoint implementation
   |endpoint implement may need |enhancement
   |to enhance  |

-- 
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-tc8.0.x-test-apr (in module tomcat-8.0.x) failed

2015-03-09 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-test-apr has an issue affecting its community 
integration.
This issue affects 1 projects.
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-test-apr :  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-test-apr/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -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.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-8.0.x/output/logs-APR
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-8.0.x/output/test-tmp-APR/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-8.0.x/tomcat-tc8.0.x-test-apr/gump_work/build_tomcat-8.0.x_tomcat-tc8.0.x-test-apr.html
Work Name: build_tomcat-8.0.x_tomcat-tc8.0.x-test-apr (Type: Build)
Work ended in a state of : Failed
Elapsed: 56 mins 23 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 
-Dobjenesis.jar=/srv/gump/public/workspace/objenesis/main/target/objenesis-2.2-SNAPSHOT.jar
 -Dtest.reports=output/logs-APR 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dexamples.sources.skip=true 
-Djdt.jar=/srv/gump/packages/eclipse/plugins/R-4.4-201406061215/ecj-4.4.jar 
-Dtest.apr.loc=/srv/gump/public/workspace/tomcat-native/dest-20150309/lib 
-Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20150309.jar
 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dtest.temp=output/test-tmp-APR -Dtest.accesslog=true -Dexecute.test.nio=false 
-Dtest
 
.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20150309/bin/openssl
 -Dexecute.test.apr=true -Dexecute.test.bio=false -Dexecute.test.nio2=false 
-Deasymock.jar=/srv/gump/public/workspace/easymock/easymock/target/easymock-3.4-SNAPSHOT.jar
 -Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar 
-Dcglib.jar=/srv/gump/packages/cglib/cglib-nodep-2.2.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-8.0.x]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-8.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-8.0.x/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/servlet-api.ja
 
r:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/websocket-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-storeconfig.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/catalina-ha.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-api.jar:/srv/gump/public/workspace/tomcat-8.0.x/output/build/lib/tomcat-jni.jar:/srv/gump/public/workspace/tomcat

[GUMP@vmgump]: Project tomcat-trunk-test-nio (in module tomcat-trunk) failed

2015-03-09 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-trunk-test-nio has an issue affecting its community integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-test-nio :  Tomcat 9.x, a web server implementing the Java 
Servlet 4.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-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 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.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/logs-NIO
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/test-tmp-NIO/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-test-nio/gump_work/build_tomcat-trunk_tomcat-trunk-test-nio.html
Work Name: build_tomcat-trunk_tomcat-trunk-test-nio (Type: Build)
Work ended in a state of : Failed
Elapsed: 55 mins 3 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 
-Dobjenesis.jar=/srv/gump/public/workspace/objenesis/main/target/objenesis-2.2-SNAPSHOT.jar
 -Dtest.reports=output/logs-NIO 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dexamples.sources.skip=true 
-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-20150309.jar
 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dtest.temp=output/test-tmp-NIO -Dtest.accesslog=true -Dexecute.test.nio=true 
-Dtest.openssl.path=/srv/gump/public/workspace/openssl/dest-20150309/bin/openssl
 
 -Dexecute.test.apr=false -Dexecute.test.nio2=false 
-Deasymock.jar=/srv/gump/public/workspace/easymock/easymock/target/easymock-3.4-SNAPSHOT.jar
 -Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar 
-Dcglib.jar=/srv/gump/packages/cglib/cglib-nodep-2.2.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-trunk/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-trunk/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/servlet-api.ja
 
r:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/websocket-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-storeconfig.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ha.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-jni.jar:/srv/gump/public/workspace/tomcat-trunk/output/bu
 
ild/lib/tomcat-spdy.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-util.jar:/srv

[GUMP@vmgump]: Project tomcat-trunk-test-nio2 (in module tomcat-trunk) failed

2015-03-09 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-trunk-test-nio2 has an issue affecting its community integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-test-nio2 :  Tomcat 9.x, a web server implementing the Java 
Servlet 4.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-test-nio2/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -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.
 -INFO- Failed with reason build failed
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/logs-NIO2
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-trunk/output/test-tmp-NIO2/logs



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-test-nio2/gump_work/build_tomcat-trunk_tomcat-trunk-test-nio2.html
Work Name: build_tomcat-trunk_tomcat-trunk-test-nio2 (Type: Build)
Work ended in a state of : Failed
Elapsed: 54 mins 15 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 
-Dobjenesis.jar=/srv/gump/public/workspace/objenesis/main/target/objenesis-2.2-SNAPSHOT.jar
 -Dtest.reports=output/logs-NIO2 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dexamples.sources.skip=true 
-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-20150309.jar
 
-Dcommons-daemon.native.src.tgz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-native-src.tar.gz
 -Dtest.temp=output/test-tmp-NIO2 -Dtest.accesslog=true 
-Dexecute.test.nio=false 
-Dtest.openssl.path=/srv/gump/public/workspace/openssl/dest-20150309/bin/opens
 sl -Dexecute.test.apr=false -Dexecute.test.nio2=true 
-Deasymock.jar=/srv/gump/public/workspace/easymock/easymock/target/easymock-3.4-SNAPSHOT.jar
 -Dhamcrest.jar=/srv/gump/packages/hamcrest/hamcrest-core-1.3.jar 
-Dcglib.jar=/srv/gump/packages/cglib/cglib-nodep-2.2.jar test 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-8-oracle/lib/tools.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/webapps/examples/WEB-INF/classes:/srv/gump/public/workspace/tomcat-trunk/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-trunk/output/build/bin/bootstrap.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/bin/tomcat-juli.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/annotations-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/servlet-api.ja
 
r:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/websocket-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-storeconfig.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-tribes.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/catalina-ha.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-api.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat-jni.jar:/srv/gump/public/workspace/tomcat-trunk/output/bu
 
ild/lib/tomcat-spdy.jar:/srv/gump/public/workspace/tomcat-trunk/output/build/lib/tomcat

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

2015-03-09 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-apr has an issue affecting its community 
integration.
This issue affects 1 projects.
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-apr :  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-apr/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-APR
 -WARNING- No directory 
[/srv/gump/public/workspace/tomcat-7.0.x/output/logs-APR]
 -INFO- Project Reports in: 
/srv/gump/public/workspace/tomcat-7.0.x/output/test-tmp-APR/logs
 -WARNING- No directory 
[/srv/gump/public/workspace/tomcat-7.0.x/output/test-tmp-APR/logs]



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-test-apr/gump_work/build_tomcat-7.0.x_tomcat-tc7.0.x-test-apr.html
Work Name: build_tomcat-7.0.x_tomcat-tc7.0.x-test-apr (Type: Build)
Work ended in a state of : Failed
Elapsed: 47 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-20150309-native-src.tar.gz
 -Dtest.reports=output/logs-APR 
-Dtomcat-native.tar.gz=/srv/gump/public/workspace/apache-commons/daemon/dist/bin/commons-daemon-20150309-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 
-Dtest.apr.loc=/srv/gump/public/workspace/tomcat-native/dest-20150309/lib 
-Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20150309.jar
 
-Dtomcat-dbcp-src.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-src.jar
 -Dexecute.te
 st.nio=false -Dtest.accesslog=true 
-Dcommons-pool.home=/srv/gump/public/workspace/commons-pool-1.x 
-Dcommons-dbcp.home=/srv/gump/public/workspace/commons-dbcp-1.x 
-Dexecute.test.apr=true -Dexecute.test.bio=false 
-Dtomcat-dbcp.jar=/srv/gump/public/workspace/tomcat-7.0.x/tomcat-deps/tomcat-dbcp-20150309.jar
 -Dtest.temp=output/test-tmp-APR 
-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
 
r:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jsp-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/el-api.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-ant.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/tomcat-coyote.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/jasper-el.jar:/srv/gump/public/workspace/tomcat-7.0.x/output/build/lib/catalina-tribes.jar:/srv/gump

svn commit: r1665169 - in /tomcat/tc7.0.x/trunk: java/org/apache/jasper/servlet/JspCServletContext.java webapps/docs/changelog.xml

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 10:00:59 2015
New Revision: 1665169

URL: http://svn.apache.org/r1665169
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57643
Use new File(URI) to correctly convert between a file:// URL and an absolute 
path. Fixes JspC when getResource() returns an encoded URL.

Modified:
tomcat/tc7.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java?rev=1665169r1=1665168r2=1665169view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
Mon Mar  9 10:00:59 2015
@@ -242,17 +242,16 @@ public class JspCServletContext implemen
 public String getRealPath(String path) {
 
 if (!myResourceBaseURL.getProtocol().equals(file))
-return (null);
+return null;
 if (!path.startsWith(/))
-return (null);
+return null;
 try {
-return
-(getResource(path).getFile().replace('/', File.separatorChar));
+File f = new File(getResource(path).toURI());
+return f.getAbsolutePath();
 } catch (Throwable t) {
 ExceptionUtils.handleThrowable(t);
-return (null);
+return null;
 }
-
 }
 
 

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1665169r1=1665168r2=1665169view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Mon Mar  9 10:00:59 2015
@@ -239,6 +239,11 @@
 bug57662/bug: Update all references to the ECJ compiler to version
 4.4.2. (violetagg)
   /update
+  fix
+bug57643/bug: Fix JspC when the path to the web application to
+compile contains a character that must be encoded when used in a URL
+such as space. (mark)
+  /fix
 /changelog
   /subsection
   subsection name=WebSocket



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



[Bug 57643] jasper ant task fails when there is a space in uriroot

2015-03-09 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57643

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

   What|Removed |Added

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

--- Comment #1 from Mark Thomas ma...@apache.org ---
Thanks for the report. This has been fixed in 7.0.x and will be incuded in
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



svn commit: r1665171 - /tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 10:04:57 2015
New Revision: 1665171

URL: http://svn.apache.org/r1665171
Log:
Use new File(URI) to correctly convert between a file:// URL and an absolute 
path.

Modified:
tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java

Modified: tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java?rev=1665171r1=1665170r2=1665171view=diff
==
--- tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
(original)
+++ tomcat/trunk/java/org/apache/jasper/servlet/JspCServletContext.java Mon Mar 
 9 10:04:57 2015
@@ -301,19 +301,17 @@ public class JspCServletContext implemen
  */
 @Override
 public String getRealPath(String path) {
-
 if (!myResourceBaseURL.getProtocol().equals(file))
-return (null);
+return null;
 if (!path.startsWith(/))
-return (null);
+return null;
 try {
-return
-(getResource(path).getFile().replace('/', File.separatorChar));
+File f = new File(getResource(path).toURI());
+return f.getAbsolutePath();
 } catch (Throwable t) {
 ExceptionUtils.handleThrowable(t);
-return (null);
+return null;
 }
-
 }
 
 



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



svn commit: r1665172 - /tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java

2015-03-09 Thread markt
Author: markt
Date: Mon Mar  9 10:05:10 2015
New Revision: 1665172

URL: http://svn.apache.org/r1665172
Log:
Use new File(URI) to correctly convert between a file:// URL and an absolute 
path.

Modified:
tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java

Modified: 
tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java?rev=1665172r1=1665171r2=1665172view=diff
==
--- tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
(original)
+++ tomcat/tc8.0.x/trunk/java/org/apache/jasper/servlet/JspCServletContext.java 
Mon Mar  9 10:05:10 2015
@@ -301,19 +301,17 @@ public class JspCServletContext implemen
  */
 @Override
 public String getRealPath(String path) {
-
 if (!myResourceBaseURL.getProtocol().equals(file))
-return (null);
+return null;
 if (!path.startsWith(/))
-return (null);
+return null;
 try {
-return
-(getResource(path).getFile().replace('/', File.separatorChar));
+File f = new File(getResource(path).toURI());
+return f.getAbsolutePath();
 } catch (Throwable t) {
 ExceptionUtils.handleThrowable(t);
-return (null);
+return null;
 }
-
 }
 
 



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