svn commit: r1687369 - /tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 21:01:24 2015 New Revision: 1687369 URL: http://svn.apache.org/r1687369 Log: Remove unnecessary code Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java URL:

svn commit: r1687361 - /tomcat/trunk/java/org/apache/coyote/http2/AbstractStream.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 20:26:17 2015 New Revision: 1687361 URL: http://svn.apache.org/r1687361 Log: Fix typo Modified: tomcat/trunk/java/org/apache/coyote/http2/AbstractStream.java Modified: tomcat/trunk/java/org/apache/coyote/http2/AbstractStream.java URL: http://svn.apache.org/vie

svn commit: r1687352 - /tomcat/trunk/res/findbugs/filter-false-positives.xml

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 19:45:46 2015 New Revision: 1687352 URL: http://svn.apache.org/r1687352 Log: Add some more false positives Modified: tomcat/trunk/res/findbugs/filter-false-positives.xml Modified: tomcat/trunk/res/findbugs/filter-false-positives.xml URL: http://svn.apache.org/

svn commit: r1687351 - /tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 19:44:51 2015 New Revision: 1687351 URL: http://svn.apache.org/r1687351 Log: Correct boundary condition Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java URL

svn commit: r1687350 - /tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 19:37:02 2015 New Revision: 1687350 URL: http://svn.apache.org/r1687350 Log: Fix logic Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java URL: http://svn.apa

buildbot exception in ASF Buildbot on tomcat-8-trunk

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

buildbot success in ASF Buildbot on tomcat-8-trunk

2015-06-24 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/302 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranch

buildbot failure in ASF Buildbot on tomcat-7-trunk

2015-06-24 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/689 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSch

svn commit: r1687349 - /tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/PrincipalGroupCallback.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 19:26:00 2015 New Revision: 1687349 URL: http://svn.apache.org/r1687349 Log: Fix copy/paste error spotted by FindBugs Modified: tomcat/trunk/java/org/apache/catalina/authenticator/jaspic/PrincipalGroupCallback.java Modified: tomcat/trunk/java/org/apache/catal

svn commit: r1687346 - /tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 19:10:57 2015 New Revision: 1687346 URL: http://svn.apache.org/r1687346 Log: Fix FindBugs warnings. activeRemoteStreamCount wasn't thread safe. Modified: tomcat/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java Modified: tomcat/trunk/java/org/apache/c

buildbot failure in ASF Buildbot on tomcat-trunk

2015-06-24 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/1454 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSchedu

svn commit: r1687342 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/filters/RemoteIpFilter.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 18:45:59 2015 New Revision: 1687342 URL: http://svn.apache.org/r1687342 Log: (empty) Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/catalina/filters/RemoteIpFilter.java Propchange: tomcat/tc7.0.x/trunk/ --

svn commit: r1687341 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/filters/RemoteIpFilter.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 18:45:20 2015 New Revision: 1687341 URL: http://svn.apache.org/r1687341 Log: Fix FindBugs warning. Don't use != with Strings Modified: tomcat/tc8.0.x/trunk/ (props changed) tomcat/tc8.0.x/trunk/java/org/apache/catalina/filters/RemoteIpFilter.java Propchan

svn commit: r1687340 - /tomcat/trunk/java/org/apache/catalina/filters/RemoteIpFilter.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 18:44:18 2015 New Revision: 1687340 URL: http://svn.apache.org/r1687340 Log: Fix FindBugs warning. Don't use != with Strings Modified: tomcat/trunk/java/org/apache/catalina/filters/RemoteIpFilter.java Modified: tomcat/trunk/java/org/apache/catalina/filters/Remo

[Bug 57546] Memory Leak in SecureNioChannel

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57546 --- Comment #14 from Mark Thomas --- (In reply to Paul Gaylie from comment #13) > Created attachment 32854 [details] > Screen Shot from hprof > > Guys, Please consider using a more gender neutral form of address when addressing the Tomcat com

[Bug 57546] Memory Leak in SecureNioChannel

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57546 Paul Gaylie changed: What|Removed |Added CC||paul.gay...@gmail.com --- Comment #13 fr

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 Violeta Georgieva changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

svn commit: r1687334 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/core/ContainerBase.java webapps/docs/changelog.xml

2015-06-24 Thread violetagg
Author: violetagg Date: Wed Jun 24 17:37:39 2015 New Revision: 1687334 URL: http://svn.apache.org/r1687334 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 Merged revision 1687268 from tomcat/trunk: ADD_CHILD_EVENT must be sent even when there are failures during child's start, thus

svn commit: r1687333 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/core/ContainerBase.java webapps/docs/changelog.xml

2015-06-24 Thread violetagg
Author: violetagg Date: Wed Jun 24 17:33:17 2015 New Revision: 1687333 URL: http://svn.apache.org/r1687333 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 Merged revision 1687268 from tomcat/trunk: ADD_CHILD_EVENT must be sent even when there are failures during child's start, thus

Re: [Bug 58031] Posting data exceeding maxPostSize should result in HTTP 413.

2015-06-24 Thread Violeta Georgieva
2015-06-24 18:37 GMT+03:00 Christopher Schultz : > > Mark, > > On 6/24/15 9:57 AM, Mark Thomas wrote: > > On 24/06/2015 14:43, Violeta Georgieva wrote: > >> Hi Mark, > >> > >> 2015-06-24 16:27 GMT+03:00 : > >>> > >>> https://bz.apache.org/bugzilla/show_bug.cgi?id=58031 > >>> > >>> --- Comment #10 f

Re: [Bug 58031] Posting data exceeding maxPostSize should result in HTTP 413.

2015-06-24 Thread Christopher Schultz
Mark, On 6/24/15 9:57 AM, Mark Thomas wrote: > On 24/06/2015 14:43, Violeta Georgieva wrote: >> Hi Mark, >> >> 2015-06-24 16:27 GMT+03:00 : >>> >>> https://bz.apache.org/bugzilla/show_bug.cgi?id=58031 >>> >>> --- Comment #10 from Mark Thomas --- >>> I've patched trunk (9.0.x) to make the reason f

Re: Time for Tomcat 7.0.63?

2015-06-24 Thread Christopher Schultz
Violeta, On 6/24/15 2:34 AM, Violeta Georgieva wrote: > Hi, > > 2015-06-23 20:53 GMT+03:00 Christopher Schultz > : >> >> All, >> >> Some recent bugs have been fixed and there is some community demand for >> 7.0.63. >> >> e.g. >> https://bz.apache.org/bugzilla/show_bug.cgi?id=57783 >> >> Violetta,

buildbot success in ASF Buildbot on tomcat-trunk

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

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

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 14:15:32 2015 New Revision: 1687292 URL: http://svn.apache.org/r1687292 Log: Correct error code Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java URL: http://

[Bug 58074] Memory leak - session serialization

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=58074 --- Comment #2 from qed...@gmail.com --- (In reply to Mark Thomas from comment #1) > A profiler confirms that there is no memory leak here. weak references != > memory leaks. GC will clear them as/if necessary to relieve memory pressure. I must

buildbot failure in ASF Buildbot on tomcat-trunk

2015-06-24 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/1452 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSchedu

Re: [Bug 58031] Posting data exceeding maxPostSize should result in HTTP 413.

2015-06-24 Thread Mark Thomas
On 24/06/2015 14:43, Violeta Georgieva wrote: > Hi Mark, > > 2015-06-24 16:27 GMT+03:00 : >> >> https://bz.apache.org/bugzilla/show_bug.cgi?id=58031 >> >> --- Comment #10 from Mark Thomas --- >> I've patched trunk (9.0.x) to make the reason for the failure available > as a >> request attribute th

svn commit: r1687278 - in /tomcat/trunk/test/org/apache/coyote/http2: Http2TestBase.java TestHttp2Section_5_2.java TestHttp2Section_5_3.java TestHttp2Section_5_5.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 13:55:44 2015 New Revision: 1687278 URL: http://svn.apache.org/r1687278 Log: Make writing the settings frame unit tests easier. Modified: tomcat/trunk/test/org/apache/coyote/http2/Http2TestBase.java tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_

Re: [Bug 58031] Posting data exceeding maxPostSize should result in HTTP 413.

2015-06-24 Thread Violeta Georgieva
Hi Mark, 2015-06-24 16:27 GMT+03:00 : > > https://bz.apache.org/bugzilla/show_bug.cgi?id=58031 > > --- Comment #10 from Mark Thomas --- > I've patched trunk (9.0.x) to make the reason for the failure available as a > request attribute that the FailedRequestFilter can then use to provide a better

svn commit: r1687268 - /tomcat/trunk/java/org/apache/catalina/core/ContainerBase.java

2015-06-24 Thread violetagg
Author: violetagg Date: Wed Jun 24 13:40:34 2015 New Revision: 1687268 URL: http://svn.apache.org/r1687268 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 ADD_CHILD_EVENT must be sent even when there are failures during child's start, thus MapperListener will be added as Container/L

buildbot success in ASF Buildbot on tomcat-trunk

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

[Bug 58031] Posting data exceeding maxPostSize should result in HTTP 413.

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=58031 --- Comment #10 from Mark Thomas --- I've patched trunk (9.0.x) to make the reason for the failure available as a request attribute that the FailedRequestFilter can then use to provide a better status code to the client. I'll give it a few day

svn commit: r1687261 - in /tomcat/trunk: java/org/apache/catalina/ java/org/apache/catalina/connector/ java/org/apache/catalina/filters/ java/org/apache/tomcat/util/http/ test/org/apache/catalina/conn

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 13:26:06 2015 New Revision: 1687261 URL: http://svn.apache.org/r1687261 Log: Make the (first) reason parameter parsing failed available as a request attribute and then use it to provide a better status code via the FailedRequstFilter (if configured). Modified:

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 --- Comment #8 from Remy Maucherat --- Would be fine then, but +1 for the finally. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 --- Comment #7 from Violeta Georgieva --- (In reply to Mark Thomas from comment #5) > I'm wary of changing the order since it might break something else. How about > a > finally block? +1 (In reply to Remy Maucherat from comment #6) > I agre

buildbot failure in ASF Buildbot on tomcat-trunk

2015-06-24 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/1449 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSchedu

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 --- Comment #6 from Remy Maucherat --- I agree it will have side effects, like maybe mapping requests while the context is being deployed ? A finally seems better but even that would still change the current behavior (previously requests would

[Bug 58052] RewriteValve: Implement additional RewriteRule directive capabilities

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=58052 --- Comment #5 from Remy Maucherat --- I have documented explicitly the difference with the current mod_rewrite in that area. -- You are receiving this mail because: You are the assignee for the bug. -

svn commit: r1687251 - /tomcat/tc8.0.x/trunk/webapps/docs/rewrite.xml

2015-06-24 Thread remm
Author: remm Date: Wed Jun 24 12:34:07 2015 New Revision: 1687251 URL: http://svn.apache.org/r1687251 Log: Port 58052: Clarify that some newer additions from the mod_rewrite documentation are not supported in Tomcat. Modified: tomcat/tc8.0.x/trunk/webapps/docs/rewrite.xml Modified: tomcat/t

svn commit: r1687250 - /tomcat/trunk/webapps/docs/rewrite.xml

2015-06-24 Thread remm
Author: remm Date: Wed Jun 24 12:33:24 2015 New Revision: 1687250 URL: http://svn.apache.org/r1687250 Log: 58052: Clarify that some newer additions from the mod_rewrite documentation are not supported in Tomcat. Modified: tomcat/trunk/webapps/docs/rewrite.xml Modified: tomcat/trunk/webapps/

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 --- Comment #5 from Mark Thomas --- I'm wary of changing the order since it might break something else. How about a finally block? -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57700] Exception in a LifecycleListener on Context makes webapp unreachable in spite of subsequent successful startups

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57700 --- Comment #4 from Violeta Georgieva --- Hi, I succeeded to reproduce the scenario on all Tomcat versions. The issue is the following: - The implementation of Manager web app invokes HostConfig to deploy the web application. ... at org.a

buildbot exception in ASF Buildbot on tomcat-trunk

2015-06-24 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/1448 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSc

svn commit: r1687242 - /tomcat/trunk/java/org/apache/catalina/connector/Request.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 11:58:57 2015 New Revision: 1687242 URL: http://svn.apache.org/r1687242 Log: Differentiate between failure modes. Handling is currently identical but it won't be once BZ 58031 is addressed. Modified: tomcat/trunk/java/org/apache/catalina/connector/Request.java

[Bug 58074] Memory leak - session serialization

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=58074 Mark Thomas changed: What|Removed |Added Resolution|--- |INVALID Status|NEW

[Bug 58074] New: Memory leak - session serialization

2015-06-24 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=58074 Bug ID: 58074 Summary: Memory leak - session serialization Product: Tomcat 7 Version: 7.0.62 Hardware: PC Status: NEW Severity: minor Priority: P2 Co

buildbot success in ASF Buildbot on tomcat-trunk

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

svn commit: r1687193 - /tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_6_4.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 08:02:34 2015 New Revision: 1687193 URL: http://svn.apache.org/r1687193 Log: Add unit tests for rst frames Added: tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_6_4.java (with props) Added: tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Secti

svn commit: r1687184 - in /tomcat/trunk/test/org/apache/coyote/http2: Http2TestBase.java TestHttp2Section_6_2.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 07:53:55 2015 New Revision: 1687184 URL: http://svn.apache.org/r1687184 Log: Use constants Modified: tomcat/trunk/test/org/apache/coyote/http2/Http2TestBase.java tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_6_2.java Modified: tomcat/trunk/test

svn commit: r1687183 - /tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_6_3.java

2015-06-24 Thread markt
Author: markt Date: Wed Jun 24 07:53:20 2015 New Revision: 1687183 URL: http://svn.apache.org/r1687183 Log: Add unit tests for priority frames Added: tomcat/trunk/test/org/apache/coyote/http2/TestHttp2Section_6_3.java (with props) Added: tomcat/trunk/test/org/apache/coyote/http2/TestHttp2