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

2017-03-01 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-nio2 has an issue affecting its community integration

[Bug 60461] SIGSEGV in SSLSocket.getInfos

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60461 --- Comment #9 from matt...@cacorp.com --- Created attachment 34790 --> https://bz.apache.org/bugzilla/attachment.cgi?id=34790&action=edit 8.5.11 Windows Crash Log This crash still exists in 8.5.11 on Windows. I've uploaded my log. -- You a

[Bug 60798] Nested Jar entry could not be read twice consecutively

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Charles-Edouard Poisnel changed: What|Removed |Added CC||charles-edouard.poisnel@kos

[Bug 60798] Nested Jar entry could not be read twice consecutively

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 --- Comment #1 from Charles-Edouard Poisnel --- Note: tomcat-embed-core-8.5.11 is used. -- You are receiving this mail because: You are the assignee for the bug. - To unsubs

[Bug 60798] New: Nested Jar entry could not be read twice consecutively

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Bug ID: 60798 Summary: Nested Jar entry could not be read twice consecutively Product: Tomcat 8 Version: 8.5.11 Hardware: PC OS: Linux Status: NEW Sever

buildbot success in on tomcat-trunk

2017-03-01 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while building . Full details are available at: https://ci.apache.org/builders/tomcat-trunk/builds/2139 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchScheduler sch

[Bug 60597] Add ability to set cipher suites for websocket client connections

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60597 Mark Thomas changed: What|Removed |Added Severity|minor |enhancement -- You are receiving this m

[Bug 60750] A response that uses Transfer-Encoding: chunked is missing the last zero-length chunk

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60750 Mark Thomas changed: What|Removed |Added Status|REOPENED|NEEDINFO --- Comment #11 from Mark Thoma

[Bug 60769] Problem with Jsp character encoding configuration

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60769 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

svn commit: r1785038 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/jasper/compiler/ test/org/apache/jasper/compiler/ test/org/apache/jasper/servlet/ test/webapp/WEB-INF/ test/webapp/jsp/encoding/ web

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 21:00:02 2017 New Revision: 1785038 URL: http://svn.apache.org/viewvc?rev=1785038&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60769 Fix regression in JSP encoding detection refactoring Added: tomcat/tc8.5.x/trunk/test/webapp/jsp/encoding/bug

svn commit: r1785037 - /tomcat/trunk/test/org/apache/jasper/servlet/TestJspCServletContext.java

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 20:59:20 2017 New Revision: 1785037 URL: http://svn.apache.org/viewvc?rev=1785037&view=rev Log: Update expected value after changes to support other tests Modified: tomcat/trunk/test/org/apache/jasper/servlet/TestJspCServletContext.java Modified: tomcat/trunk/

buildbot failure in on tomcat-trunk

2017-03-01 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-trunk while building . Full details are available at: https://ci.apache.org/builders/tomcat-trunk/builds/2138 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchScheduler schedu

svn commit: r1785032 - in /tomcat/trunk: java/org/apache/jasper/compiler/ test/org/apache/jasper/compiler/ test/webapp/WEB-INF/ test/webapp/jsp/encoding/ webapps/docs/

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 20:44:22 2017 New Revision: 1785032 URL: http://svn.apache.org/viewvc?rev=1785032&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60769 Fix regression in JSP encoding detection refactoring Added: tomcat/trunk/test/webapp/jsp/encoding/bug60769a.j

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #13 from Markus Malkusch --- Then let me add more details to the described case: The intended symetric round trip behaviour was not given. The Cookie was initially created with the Servlet API (containing only alphanumeric character

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 Mark Thomas changed: What|Removed |Added Resolution|--- |WONTFIX Status|REOPENED

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

2017-03-01 Thread Bill Barker
ump/public/workspace/objenesis/main/target/objenesis-2.6-SNAPSHOT.jar -Dexamples.sources.skip=true -Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20170301.jar -Dtest.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20170301/bin/openssl -Dex

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #11 from Markus Malkusch --- Yes, I saw that wiki page already and it feels like some hair got lost on implementing that. Let me know when you remember the reason and also if you plan to keep it like that. For me it's not a big dea

[Bug 60764] SlowQueryReport causing connection leak

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60764 --- Comment #2 from mark.hou...@sony.com --- (In reply to Keiichi Fujino from comment #1) > Hi. > > Does this problem affect only when using SlowQueryReport? > According to my initial investigation, the same problems seems to be occurred > even

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #10 from Mark Thomas --- Progress. The quotes are retained and the unit test is removing them before comparing the actual and expected values. That also looks very deliberate. I need to remind myself why that is the case. The handl

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #9 from Mark Thomas --- Definitely not a Spring issue. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-uns

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #8 from Markus Malkusch --- Also please let me know if you agree that I can finally close the issue on Spring's side. -- You are receiving this mail because: You are the assignee for the bug. --

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #7 from Markus Malkusch --- I looked also a bit deeper. Parsing that header enters org.apache.tomcat.util.http.parser.Cookie.readQuotedString(ByteBuffer), which if I understand correctly will always return the value including the qu

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #6 from Mark Thomas --- Let me take a closer look. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubsc

svn commit: r1784991 - in /tomcat/site/trunk: docs/security-6.html docs/security-7.html docs/security-8.html docs/security-9.html xdocs/security-6.xml xdocs/security-7.xml xdocs/security-8.xml xdocs/s

2017-03-01 Thread csutherl
Author: csutherl Date: Wed Mar 1 16:56:54 2017 New Revision: 1784991 URL: http://svn.apache.org/viewvc?rev=1784991&view=rev Log: Fixing typo on the security pages. Modified: tomcat/site/trunk/docs/security-6.html tomcat/site/trunk/docs/security-7.html tomcat/site/trunk/docs/security-

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 Markus Malkusch changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|WORKSFOR

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #4 from Markus Malkusch --- Actually I also don't think Spring is messing around with Cookie parsing, but something is. The test cases are fine. I double checked by comparing Tomcat's log on the receiver side. Tomcat receives the Co

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 --- Comment #3 from Mark Thomas --- I tested 9.0.x trunk and 8.5.x trunk. Both worked as expected. Personally, I'd start with your test case and look at exactly what header values are being sent. I'd be surprised if Spring Boot is playing a ro

svn commit: r1784981 - in /tomcat/tc8.5.x/trunk: ./ test/org/apache/tomcat/util/http/TestCookies.java

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 16:05:07 2017 New Revision: 1784981 URL: http://svn.apache.org/viewvc?rev=1784981&view=rev Log: Add a test case based on the report in bug 60788. Modified: tomcat/tc8.5.x/trunk/ (props changed) tomcat/tc8.5.x/trunk/test/org/apache/tomcat/util/http/TestCook

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 Markus Malkusch changed: What|Removed |Added URL||https://github.com/spring-p

[Bug 60784] HTTP status line tests ignore trailing space after status code in status line (incomplete RFC test)

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

svn commit: r1784969 - in /tomcat/tc7.0.x/trunk: ./ test/org/apache/catalina/connector/TestKeepAliveCount.java test/org/apache/catalina/startup/SimpleHttpClient.java webapps/docs/changelog.xml

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 15:20:31 2017 New Revision: 1784969 URL: http://svn.apache.org/viewvc?rev=1784969&view=rev Log: Update all unit tests that test the HTTP status line to check for the required space after the status code. Patch provided by Michael Osipov. Modified: tomcat/tc7.0.

svn commit: r1784966 - in /tomcat/tc8.0.x/trunk: ./ test/org/apache/catalina/connector/TestKeepAliveCount.java test/org/apache/catalina/startup/SimpleHttpClient.java webapps/docs/changelog.xml

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 15:17:30 2017 New Revision: 1784966 URL: http://svn.apache.org/viewvc?rev=1784966&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 Update all unit tests that test the HTTP status line to check for the required space after the status code. Patch

svn commit: r1784964 - in /tomcat/tc8.5.x/trunk: ./ test/org/apache/catalina/connector/TestKeepAliveCount.java test/org/apache/catalina/startup/SimpleHttpClient.java test/org/apache/coyote/http2/Http2

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 15:15:41 2017 New Revision: 1784964 URL: http://svn.apache.org/viewvc?rev=1784964&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 Update all unit tests that test the HTTP status line to check for the required space after the status code. Patch

svn commit: r1784963 - in /tomcat/trunk: test/org/apache/catalina/connector/TestKeepAliveCount.java test/org/apache/catalina/startup/SimpleHttpClient.java test/org/apache/coyote/http2/Http2TestBase.ja

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 15:12:38 2017 New Revision: 1784963 URL: http://svn.apache.org/viewvc?rev=1784963&view=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 Update all unit tests that test the HTTP status line to check for the required space after the status code. Patch

[Bug 60784] HTTP status line tests ignore trailing space after status code in status line (incomplete RFC test)

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 --- Comment #2 from Michael Osipov <1983-01...@gmx.net> --- (In reply to Mark Thomas from comment #1) > The tests in question aren't (explicitly) testing the format of the status > line. They are testing that the correct status code is returned.

Re: svn commit: r1784959 - /tomcat/tc8.5.x/trunk/java/org/apache/catalina/security/SecurityClassLoad.java

2017-03-01 Thread Coty Sutherland
On Wed, Mar 1, 2017 at 9:55 AM, Mark Thomas wrote: > On 01/03/17 14:54, csuth...@apache.org wrote: >> Author: csutherl >> Date: Wed Mar 1 14:54:38 2017 >> New Revision: 1784959 >> >> URL: http://svn.apache.org/viewvc?rev=1784959&view=rev >> Log: >> Follow up to r1784807. >> Removing PushToken cla

Re: svn commit: r1784959 - /tomcat/tc8.5.x/trunk/java/org/apache/catalina/security/SecurityClassLoad.java

2017-03-01 Thread Mark Thomas
On 01/03/17 14:54, csuth...@apache.org wrote: > Author: csutherl > Date: Wed Mar 1 14:54:38 2017 > New Revision: 1784959 > > URL: http://svn.apache.org/viewvc?rev=1784959&view=rev > Log: > Follow up to r1784807. > Removing PushToken class reference as it was deleted by r1784807. Nice catch. Tha

svn commit: r1784959 - /tomcat/tc8.5.x/trunk/java/org/apache/catalina/security/SecurityClassLoad.java

2017-03-01 Thread csutherl
Author: csutherl Date: Wed Mar 1 14:54:38 2017 New Revision: 1784959 URL: http://svn.apache.org/viewvc?rev=1784959&view=rev Log: Follow up to r1784807. Removing PushToken class reference as it was deleted by r1784807. Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/security/Security

[Bug 60784] HTTP status line tests ignore trailing space after status code in status line (incomplete RFC test)

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60784 --- Comment #1 from Mark Thomas --- The tests in question aren't (explicitly) testing the format of the status line. They are testing that the correct status code is returned. I don't see any harm in expanding the tests to check for the space.

[Bug 60788] Cookies value contains quotes when the Cookie header contains $Version=1 and the header's value is enclosed by quotes

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60788 Mark Thomas changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEW

svn commit: r1784956 - /tomcat/trunk/test/org/apache/tomcat/util/http/TestCookies.java

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 14:49:59 2017 New Revision: 1784956 URL: http://svn.apache.org/viewvc?rev=1784956&view=rev Log: Add a test case based on the report in bug 60788. Modified: tomcat/trunk/test/org/apache/tomcat/util/http/TestCookies.java Modified: tomcat/trunk/test/org/apache/tom

Re: Default character encoding to be UTF-8 for 9.0.x

2017-03-01 Thread Enrico Olivelli
My two cents... It would be great! The backport to 8.5 too I always need to add some filter or some lines of code to every servlet. Enrico Il mer 1 mar 2017, 14:31 Mark Thomas ha scritto: > Servlet 4.0 adds the ability to set a default request character encoding > and a default response charac

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

2017-03-01 Thread Bill Barker
ump/public/workspace/objenesis/main/target/objenesis-2.6-SNAPSHOT.jar -Dexamples.sources.skip=true -Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20170301.jar -Dtest.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20170301/bin/openssl -Dex

svn commit: r1784933 - in /tomcat/site/trunk: docs/security-7.html docs/security-8.html xdocs/security-7.xml xdocs/security-8.xml

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 13:48:39 2017 New Revision: 1784933 URL: http://svn.apache.org/viewvc?rev=1784933&view=rev Log: Add info on CVE-2017-6056 to the not a vulnerability in Tomcat section Modified: tomcat/site/trunk/docs/security-7.html tomcat/site/trunk/docs/security-8.html

Default character encoding to be UTF-8 for 9.0.x

2017-03-01 Thread Mark Thomas
Servlet 4.0 adds the ability to set a default request character encoding and a default response character encoding in web.xml and programmatically. The spec allows containers to specify their own defaults via vendor specific config. We can do this via conf/web.xml. Therefore, I would like to propo

svn commit: r1784929 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/ java/org/apache/catalina/connector/ java/org/apache/catalina/core/ java/org/apache/catalina/servlet4preview/ java/org/apac

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 13:23:36 2017 New Revision: 1784929 URL: http://svn.apache.org/viewvc?rev=1784929&view=rev Log: Servlet 4.0 New methods on the ServletContext to enable the default request and response character encodings to be set per web application. Deployment descriptor changes

svn commit: r1784926 - in /tomcat/trunk: java/javax/servlet/ java/javax/servlet/resources/ java/org/apache/catalina/ java/org/apache/catalina/connector/ java/org/apache/catalina/core/ java/org/apache/

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 13:10:42 2017 New Revision: 1784926 URL: http://svn.apache.org/viewvc?rev=1784926&view=rev Log: Servlet 4.0 New elements for the deployment descriptor and new methods on the ServletContext to enable the default request and response character encodings to be set per

svn commit: r1784911 - /tomcat/trunk/java/org/apache/catalina/core/ApplicationContextFacade.java

2017-03-01 Thread markt
Author: markt Date: Wed Mar 1 11:08:26 2017 New Revision: 1784911 URL: http://svn.apache.org/viewvc?rev=1784911&view=rev Log: Fix copy/paste error Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationContextFacade.java Modified: tomcat/trunk/java/org/apache/catalina/core/Applica

[Bug 60764] SlowQueryReport causing connection leak

2017-03-01 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=60764 Keiichi Fujino changed: What|Removed |Added OS||All --- Comment #1 from Keiichi Fujin