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

2015-06-01 Thread Bill Barker
/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-20150601-native-src.tar.gz

buildbot failure in ASF Buildbot on tomcat-7-trunk

2015-06-01 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-7-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-7-trunk/builds/667 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The

buildbot success in ASF Buildbot on tomcat-7-trunk

2015-06-01 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-7-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-7-trunk/builds/669 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The

svn commit: r1682879 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/server/LocalStrings.properties java/org/apache/tomcat/websocket/server/WsServerContainer.java webapps/docs/changelo

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 12:06:25 2015 New Revision: 1682879 URL: http://svn.apache.org/r1682879 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57676 Merged revision 1665292 from tomcat/trunk: Improve error message when multiple Websocket endpoints are bound to the same path.

svn commit: r1682885 - in /tomcat/trunk: java/org/apache/coyote/http2/Http2Parser.java test/org/apache/coyote/http2/Http2TestBase.java test/org/apache/coyote/http2/TestHttp2Section_3_2.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 12:31:56 2015 New Revision: 1682885 URL: http://svn.apache.org/r1682885 Log: Refactor - put the test only code in the test classes - reorganise to make it easier to test various connection scenarios Implement a couple of TODO tests Modified:

svn commit: r1682871 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/Util.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 11:05:36 2015 New Revision: 1682871 URL: http://svn.apache.org/r1682871 Log: Merged revision 1630199 from tomcat/tc8.0.x/trunk: NPE protection. Note NPE only possible with test code. Modified: tomcat/tc7.0.x/trunk/ (props changed)

svn commit: r1682875 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/server/LocalStrings.properties java/org/apache/tomcat/websocket/server/UpgradeUtil.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 11:35:29 2015 New Revision: 1682875 URL: http://svn.apache.org/r1682875 Log: Merged revision 1630194 from tomcat/trunk: Fix an i18n TODO Modified: tomcat/tc7.0.x/trunk/ (props changed)

svn commit: r1682873 - in /tomcat/trunk/test/org/apache/coyote/http2: Http2TestBase.java TestHttp2Base.java TestHttp2Section_3_2.java TestHttp2Section_3_2_1.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 11:11:18 2015 New Revision: 1682873 URL: http://svn.apache.org/r1682873 Log: Rename so base test isn't picked up as a test class Added: tomcat/trunk/test/org/apache/coyote/http2/Http2TestBase.java - copied, changed from r1682848,

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

2015-06-01 Thread Bill Barker
-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-20150601-native-src.tar.gz -Dexamples.sources.skip=true

svn commit: r1682843 - /tomcat/trunk/java/org/apache/coyote/http2/Http2Parser.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 09:18:47 2015 New Revision: 1682843 URL: http://svn.apache.org/r1682843 Log: Writing unit tests means we need an HTTP/2 client. Start to extract the HTTP/2 parsing code into a separate class so it can be shared by the test class and the server. This is a work in

buildbot success in ASF Buildbot on tomcat-8-trunk

2015-06-01 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-8-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-8-trunk/builds/283 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The

svn commit: r1682878 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/server/UpgradeUtil.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 11:40:29 2015 New Revision: 1682878 URL: http://svn.apache.org/r1682878 Log: Merged revision 1629906 from tomcat/trunk: Use constants Modified: tomcat/tc7.0.x/trunk/ (props changed)

[Bug 57676] WebSocket check same path endpoint implementation enhancement

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57676 --- Comment #3 from Violeta Georgieva violet...@apache.org --- Fixed in Tomcat 7 trunk and will be in 7.0.63 onwards. -- You are receiving this mail because: You are the assignee for the bug.

svn commit: r1682881 - /tomcat/tc7.0.x/trunk/java/org/apache/tomcat/websocket/server/UpgradeUtil.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 12:17:00 2015 New Revision: 1682881 URL: http://svn.apache.org/r1682881 Log: Use the correct constant. Modified: tomcat/tc7.0.x/trunk/java/org/apache/tomcat/websocket/server/UpgradeUtil.java Modified:

[Bug 57977] New: The original class loader isn't re-bound to the thread in PersistentValve.invoke()

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57977 Bug ID: 57977 Summary: The original class loader isn't re-bound to the thread in PersistentValve.invoke() Product: Tomcat 8 Version: trunk Hardware: All

svn commit: r1682847 - in /tomcat/trunk/java/org/apache/coyote/http2: ConnectionPrefaceParser.java Http2Protocol.java Http2UpgradeHandler.java LocalStrings.properties

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 09:24:58 2015 New Revision: 1682847 URL: http://svn.apache.org/r1682847 Log: Remove the (overly complex) ConnectionPrefaceParser and use the new Http2Parser instead. Initial unit tests identified a need to be able to set timeouts. Provide the ability to set these

buildbot exception in ASF Buildbot on tomcat-trunk

2015-06-01 Thread buildbot
The Buildbot has detected a build exception on builder tomcat-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-trunk/builds/1298 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The

svn commit: r1682868 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/WsWebSocketContainer.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 10:37:49 2015 New Revision: 1682868 URL: http://svn.apache.org/r1682868 Log: Merged revision 1669370 from tomcat/trunk: Use constants for constant byte arrays. Modified: tomcat/tc7.0.x/trunk/ (props changed)

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

2015-06-01 Thread Bill Barker
/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-20150601-native-src.tar.gz

svn commit: r1682846 - in /tomcat/tc8.0.x/trunk: ./ build.properties.default webapps/docs/changelog.xml

2015-06-01 Thread kkolinko
Author: kkolinko Date: Mon Jun 1 09:21:41 2015 New Revision: 1682846 URL: http://svn.apache.org/r1682846 Log: Update to Checkstyle 6.7 Merged r1682842 from tomcat/trunk. Modified: tomcat/tc8.0.x/trunk/ (props changed) tomcat/tc8.0.x/trunk/build.properties.default

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

2015-06-01 Thread Bill Barker
/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-20150601-native-src.tar.gz

svn commit: r1682840 - /tomcat/trunk/java/org/apache/coyote/ProtocolException.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 09:12:09 2015 New Revision: 1682840 URL: http://svn.apache.org/r1682840 Log: New exception intended to be used when a protocol error occurs (e.g. client sent bad data) and we need to close the connection but we can't throw an unchecked exception (e.g.

svn commit: r1682842 - /tomcat/trunk/build.properties.default

2015-06-01 Thread kkolinko
Author: kkolinko Date: Mon Jun 1 09:17:06 2015 New Revision: 1682842 URL: http://svn.apache.org/r1682842 Log: Update to Checkstyle 6.7 Modified: tomcat/trunk/build.properties.default Modified: tomcat/trunk/build.properties.default URL:

buildbot failure in ASF Buildbot on tomcat-trunk

2015-06-01 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-trunk/builds/1299 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The

svn commit: r1682841 - in /tomcat/trunk/java/org/apache/coyote: AbstractProtocol.java LocalStrings.properties

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 09:13:00 2015 New Revision: 1682841 URL: http://svn.apache.org/r1682841 Log: Add handling for the new (and as yet unused) exception. Modified: tomcat/trunk/java/org/apache/coyote/AbstractProtocol.java

svn commit: r1682848 - in /tomcat/trunk/test/org/apache/coyote/http2: TestHttp2Base.java TestHttp2Section_3_2.java TestHttp2Section_3_2_1.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 09:26:13 2015 New Revision: 1682848 URL: http://svn.apache.org/r1682848 Log: Start to provide unit tests for the HTTP/2 implementation. Added: tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Base.java (with props)

svn commit: r1682925 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/PerMessageDeflate.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 15:04:49 2015 New Revision: 1682925 URL: http://svn.apache.org/r1682925 Log: Merged revisions 1630210, 1630375 from tomcat/trunk: - Correct logic (error detected by Autobahn test suite) - Window size validation needs to vary for client vs server as well

svn commit: r1682928 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/LocalStrings.properties java/org/apache/tomcat/websocket/WsFrameClient.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 15:11:44 2015 New Revision: 1682928 URL: http://svn.apache.org/r1682928 Log: Merged revision 1630203 from tomcat/trunk: Add debug logging for IOEs while reading data from server Modified: tomcat/tc7.0.x/trunk/ (props changed)

[Bug 57982] build.xml assumes Javadoc 7+ is used but does not check this

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57982 --- Comment #1 from Sebb s...@apache.org --- (In reply to Sebb from comment #0) The build file in Tomcat 6 forces the use of Java 7 for running javadoc. s/6/7/ The build.xml file for Tomcat 9 (and 8) assumes that javadoc 7+ will be used

[Bug 57982] build.xml assumes Javadoc 7+ is used but does not check this

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57982 Mark Thomas ma...@apache.org changed: What|Removed |Added Resolution|--- |WONTFIX

[Bug 57982] build.xml assumes Javadoc 7+ is used but does not check this

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57982 --- Comment #2 from Sebb s...@apache.org --- Possible solution: add the following to any tasks that run javadoc: available property=jdk1.7+ classname=java.lang.AutoCloseable/ fail unless=jdk1.7+ message=Must use Java 7 or

[Bug 57980] New: After submitting a multipart form-data, if the Http request object is not parsed and return from server without parsing, causes browser crashed ('program cannot display the webpage' m

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57980 Bug ID: 57980 Summary: After submitting a multipart form-data, if the Http request object is not parsed and return from server without parsing, causes browser crashed ('program

svn commit: r1682922 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/ java/org/apache/tomcat/websocket/server/ test/org/apache/tomcat/websocket/ webapps/docs/

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 14:58:45 2015 New Revision: 1682922 URL: http://svn.apache.org/r1682922 Log: Merged revisions 1630065, 1630092 from tomcat/trunk: - Extend support for permessage-deflate to the WebSocket client implementation. - Correct type as spotted by Martin Grigorov

[Bug 57977] The original class loader isn't re-bound to the thread in PersistentValve.invoke()

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57977 Christopher Schultz ch...@christopherschultz.net changed: What|Removed |Added Keywords|

[Bug 57982] New: build.xml assumes Javadoc 7+ is used but does not check this

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57982 Bug ID: 57982 Summary: build.xml assumes Javadoc 7+ is used but does not check this Product: Tomcat 9 Version: unspecified Hardware: All OS: All

[Bug 57977] The original class loader isn't re-bound to the thread in PersistentValve.invoke()

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57977 --- Comment #1 from Christopher Schultz ch...@christopherschultz.net --- Would it be possible to revert the changes (likely whitespace) that don't affect the code at all? It would make for a much cleaner and readable diff. -- You are

[Bug 57980] After submitting a multipart form-data, if the Http request object is not parsed and return from server without parsing, causes browser crashed ('program cannot display the webpage' messag

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57980 Christopher Schultz ch...@christopherschultz.net changed: What|Removed |Added Resolution|---

svn commit: r1682963 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/websocket/Constants.java java/org/apache/tomcat/websocket/WsWebSocketContainer.java

2015-06-01 Thread violetagg
Author: violetagg Date: Mon Jun 1 17:54:50 2015 New Revision: 1682963 URL: http://svn.apache.org/r1682963 Log: Merged revision 1642327 from tomcat/trunk: Make use of the new case-insensitive map implementation. Modified: tomcat/tc7.0.x/trunk/ (props changed)

svn commit: r1682984 - in /tomcat/trunk: java/org/apache/coyote/http2/ test/org/apache/coyote/http2/

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:03:13 2015 New Revision: 1682984 URL: http://svn.apache.org/r1682984 Log: Fix an overflow issue in 4 byte to unsigned integer conversion (needed to be a long). Fix a related TODO in ConnectionSettings Added:

svn commit: r1682989 - /tomcat/trunk/java/org/apache/coyote/http2/Stream.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:10:30 2015 New Revision: 1682989 URL: http://svn.apache.org/r1682989 Log: Format Modified: tomcat/trunk/java/org/apache/coyote/http2/Stream.java Modified: tomcat/trunk/java/org/apache/coyote/http2/Stream.java URL:

svn commit: r1682991 - in /tomcat/trunk: java/org/apache/coyote/http2/Http2Parser.java java/org/apache/coyote/http2/Http2UpgradeHandler.java test/org/apache/coyote/http2/Http2TestBase.java test/org/ap

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:15:46 2015 New Revision: 1682991 URL: http://svn.apache.org/r1682991 Log: Make fill() consistent with other read methods Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2Parser.java

svn commit: r1682977 - in /tomcat/trunk/java/org/apache/coyote/http2: AbstractStream.java Http2UpgradeHandler.java Stream.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 19:32:34 2015 New Revision: 1682977 URL: http://svn.apache.org/r1682977 Log: Store connectionId as a string so it is only converted once (it is only ever used in String form) Modified: tomcat/trunk/java/org/apache/coyote/http2/AbstractStream.java

svn commit: r1682985 - /tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:07:03 2015 New Revision: 1682985 URL: http://svn.apache.org/r1682985 Log: Use a constant for the default max frame size Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java Modified:

svn commit: r1682990 - /tomcat/trunk/java/org/apache/coyote/http2/Stream.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:12:37 2015 New Revision: 1682990 URL: http://svn.apache.org/r1682990 Log: If a write fills the buffer exactly, flush shouldn't empty it. It might be the last write of the response and if we flush we'll need to send an empty data frame later with end of stream

svn commit: r1683000 - /tomcat/trunk/java/org/apache/coyote/http2/Http2Parser.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:51:15 2015 New Revision: 1683000 URL: http://svn.apache.org/r1683000 Log: Remove unwanted space from client preface start Make package private to enable re-use in testing Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2Parser.java Modified:

svn commit: r1683006 - in /tomcat/trunk: java/org/apache/coyote/http2/ test/org/apache/coyote/http2/

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 21:11:48 2015 New Revision: 1683006 URL: http://svn.apache.org/r1683006 Log: Add a unit request for a complete HTTP/1.1 request with upgrade to HTTP/2 response. Expand (most copy/paste Http2UpgradeHandler) the Http2Parser to support the additional frame types

svn commit: r1682994 - in /tomcat/trunk: java/org/apache/coyote/http2/Http2Parser.java test/org/apache/coyote/http2/Http2TestBase.java test/org/apache/coyote/http2/TestHttp2Section_3_2.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:22:14 2015 New Revision: 1682994 URL: http://svn.apache.org/r1682994 Log: Various minor clean-ups Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2Parser.java tomcat/trunk/test/org/apache/coyote/http2/Http2TestBase.java

svn commit: r1682995 - in /tomcat/trunk: java/org/apache/coyote/http2/Http2UpgradeHandler.java test/org/apache/coyote/http2/TestHttp2Section_3_2.java

2015-06-01 Thread markt
Author: markt Date: Mon Jun 1 20:26:40 2015 New Revision: 1682995 URL: http://svn.apache.org/r1682995 Log: More clean-up Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_3_2.java Modified:

[Bug 57977] The original class loader isn't re-bound to the thread in PersistentValve.invoke()

2015-06-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57977 Kyohei Nakamura nakamura.kyohei@gmail.com changed: What|Removed |Added Attachment #32769|0 |1