[Bug 57870] backport GzipOutputFilter #doWrite to Tomcat 7 to call GZIPOutputStream to enable flushing via reflection when running on Java 7+

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57870 j...@yahoo.com changed: What|Removed |Added CC||j...@yahoo.com -- You are receiving t

[Bug 57870] New: backport GzipOutputFilter #doWrite to Tomcat 7 to call GZIPOutputStream to enable flushing via reflection when running on Java 7+

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57870 Bug ID: 57870 Summary: backport GzipOutputFilter #doWrite to Tomcat 7 to call GZIPOutputStream to enable flushing via reflection when running on Java 7+ Product: Tomcat 7

buildbot success in ASF Buildbot on tomcat-7-trunk

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

buildbot exception in ASF Buildbot on tomcat-7-trunk

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

Re: svn commit: r1676393 - in /tomcat/trunk: java/org/apache/el/parser/AstValue.java test/org/apache/el/TestMethodExpressionImpl.java

2015-04-28 Thread Christopher Schultz
Mark, On 4/28/15 5:26 PM, Mark Thomas wrote: > On 28/04/2015 22:21, Christopher Schultz wrote: >> Mark, >> >> On 4/27/15 7:01 PM, ma...@apache.org wrote: >>> Author: markt >>> Date: Mon Apr 27 23:01:16 2015 >>> New Revision: 1676393 >>> >>> URL: http://svn.apache.org/r1676393 >>> Log: >>> Add some

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

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57556 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|REOPENED

svn commit: r1676636 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/el/parser/AstValue.java

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:30:30 2015 New Revision: 1676636 URL: http://svn.apache.org/r1676636 Log: Use a new object every time for security. Review by schultz. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/el/parser/AstValue.java Propchan

svn commit: r1676635 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/el/parser/AstValue.java

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:29:32 2015 New Revision: 1676635 URL: http://svn.apache.org/r1676635 Log: Use a new object every time for security. Review by schultz. Modified: tomcat/tc8.0.x/trunk/ (props changed) tomcat/tc8.0.x/trunk/java/org/apache/el/parser/AstValue.java Propchan

svn commit: r1676634 - /tomcat/trunk/java/org/apache/el/parser/AstValue.java

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:28:26 2015 New Revision: 1676634 URL: http://svn.apache.org/r1676634 Log: Use a new object every time for security. Review by schultz. Modified: tomcat/trunk/java/org/apache/el/parser/AstValue.java Modified: tomcat/trunk/java/org/apache/el/parser/AstValue.ja

Re: svn commit: r1676393 - in /tomcat/trunk: java/org/apache/el/parser/AstValue.java test/org/apache/el/TestMethodExpressionImpl.java

2015-04-28 Thread Mark Thomas
On 28/04/2015 22:21, Christopher Schultz wrote: > Mark, > > On 4/27/15 7:01 PM, ma...@apache.org wrote: >> Author: markt >> Date: Mon Apr 27 23:01:16 2015 >> New Revision: 1676393 >> >> URL: http://svn.apache.org/r1676393 >> Log: >> Add some comments to clarify behaviour. >> Review by schultz re o

[Bug 56438] If jar scan does not find context config or TLD config, log a message

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56438 --- Comment #10 from Mark Thomas --- Thanks for the patch. It has been applied to 7.0.x/trunk for 7.0.62 onwards with the following changes: - renamed objCallBack to tldCallBack - fixed TldJarScannerCallback so it only reported no TLD was found

Re: svn commit: r1676393 - in /tomcat/trunk: java/org/apache/el/parser/AstValue.java test/org/apache/el/TestMethodExpressionImpl.java

2015-04-28 Thread Christopher Schultz
Mark, On 4/27/15 7:01 PM, ma...@apache.org wrote: > Author: markt > Date: Mon Apr 27 23:01:16 2015 > New Revision: 1676393 > > URL: http://svn.apache.org/r1676393 > Log: > Add some comments to clarify behaviour. > Review by schultz re object allocation > > Modified: > tomcat/trunk/java/org/a

svn commit: r1676633 - in /tomcat/tc7.0.x/trunk: java/org/apache/catalina/startup/LocalStrings.properties java/org/apache/catalina/startup/TldConfig.java webapps/docs/changelog.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:20:00 2015 New Revision: 1676633 URL: http://svn.apache.org/r1676633 Log: Partial fix for https://bz.apache.org/bugzilla/show_bug.cgi?id=56438 Add logging for unnecessary TLD scans in JARs Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/Local

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

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:04:52 2015 New Revision: 1676631 URL: http://svn.apache.org/r1676631 Log: Further fix for https://bz.apache.org/bugzilla/show_bug.cgi?id=57556 Only include a trailing separator in the real path if the requested path included one. Modified: tomcat/tc8.0.x/tru

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

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 21:03:04 2015 New Revision: 1676630 URL: http://svn.apache.org/r1676630 Log: Further fix for https://bz.apache.org/bugzilla/show_bug.cgi?id=57556 Only include a trailing separator in the real path if the requested path included one. Modified: tomcat/trunk/java/

[Bug 57802] Compatibility with ICEfaces 1.8 over JSF 1.1 apps

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57802 --- Comment #12 from Mark Thomas --- OK. No rush. We don't normally close things until there has been at least a month of inactivity. Worst case if this gets closed before you have a test case you can always re-open it when you have the test ca

[Bug 57802] Compatibility with ICEfaces 1.8 over JSF 1.1 apps

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57802 --- Comment #11 from Patrick WENDJI --- Sorry for the silence. I'm a bit busy right now since I'm building the test case part time. I'll come back to you once it's done. -- You are receiving this mail because: You are the assignee for the bug

buildbot exception in ASF Buildbot on tomcat-8-trunk

2015-04-28 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/228 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranc

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

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57556 --- Comment #8 from Mark Thomas --- (In reply to Chuck Caldarale from comment #7) > (In reply to Christopher Schultz from comment #6) > > But my position is that this method should unconditionally return null. > > +1 on that. Anyone using thi

[Bug 57802] Compatibility with ICEfaces 1.8 over JSF 1.1 apps

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57802 --- Comment #10 from Mark Thomas --- Without a test case this is going to get resolved as INVALID. -- You are receiving this mail because: You are the assignee for the bug.

[Bug 57863] Can't get the matched pattern in RewriteValve

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57863 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

svn commit: r1676619 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/catalina/valves/rewrite/ test/org/apache/catalina/valves/rewrite/ webapps/docs/

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 20:37:37 2015 New Revision: 1676619 URL: http://svn.apache.org/r1676619 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57863 Correct RewriteMap support Based on a patch provided by Tatsuya Bessho. Added: tomcat/tc8.0.x/trunk/test/org/apache/catalina/va

svn commit: r1676615 - in /tomcat/trunk: java/org/apache/catalina/valves/rewrite/Substitution.java test/org/apache/catalina/valves/rewrite/TestRewriteValve.java test/org/apache/catalina/valves/rewrite

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 20:35:21 2015 New Revision: 1676615 URL: http://svn.apache.org/r1676615 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57863 Correct RewriteMap support Based on a patch provided by Tatsuya Bessho. Added: tomcat/trunk/test/org/apache/catalina/valves/rewr

buildbot success in ASF Buildbot on tomcat-8-trunk

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

buildbot success in ASF Buildbot on tomcat-trunk

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

[Bug 57864] channelSendOptions attribute is not able to be configured in hexadecimal

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

svn commit: r1676558 - in /tomcat/tc6.0.x/trunk/webapps/docs: changelog.xml config/cluster.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 15:08:58 2015 New Revision: 1676558 URL: http://svn.apache.org/r1676558 Log: CTR: Docs only Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Make it clearer that hex values are not valid for cluster send options Based on a patch by Kyohei Nakamura Modified:

svn commit: r1676557 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/cluster.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 15:04:56 2015 New Revision: 1676557 URL: http://svn.apache.org/r1676557 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Make it clearer that hex values are not valid for cluster send options Based on a patch by Kyohei Nakamura Modified: tomcat/tc7.

svn commit: r1676556 - in /tomcat/tc8.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/cluster.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 15:03:29 2015 New Revision: 1676556 URL: http://svn.apache.org/r1676556 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Make it clearer that hex values are not valid for cluster send options Based on a patch by Kyohei Nakamura Modified: tomcat/tc8.

svn commit: r1676552 - /tomcat/trunk/webapps/docs/config/cluster.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 14:55:13 2015 New Revision: 1676552 URL: http://svn.apache.org/r1676552 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Make it clearer that hex values are not valid for cluster send options Based on a patch by Kyohei Nakamura Modified: tomcat/trun

buildbot exception in ASF Buildbot on tomcat-8-trunk

2015-04-28 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/226 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranc

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

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 13:37:38 2015 New Revision: 1676532 URL: http://svn.apache.org/r1676532 Log: Clearer error message when trying to build a full release build on a non-Windows platform without Wine. Modified: tomcat/tc8.0.x/trunk/ (props changed) tomcat/tc8.0.x/trunk/build

buildbot success in ASF Buildbot on tomcat-8-trunk

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

[Bug 57707] Execute failed: java.io.IOException: Cannot run program "wine"

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57707 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|REOPENED

svn commit: r1676525 - /tomcat/trunk/build.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 12:58:38 2015 New Revision: 1676525 URL: http://svn.apache.org/r1676525 Log: Fix for https://bz.apache.org/bugzilla/show_bug.cgi?id=57707 Clearer error message when trying to build a full release build on a non-Windows platform without Wine. Modified: tomcat/tr

[Bug 57866] FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 j...@yahoo.com changed: What|Removed |Added Resolution|--- |WONTFIX Status|NEW

[Bug 57837] add "text/css" to default of compressableMimeType

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 --- Comment #9 from Violeta Georgieva --- +1 for javascript -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr.

[Bug 57866] FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 --- Comment #3 from Konstantin Kolinko --- 1. If Tomcat did call "def.setLevel(Deflater.DEFAULT_COMPRESSION);" but that call did not re-enable compression, that would be a bug in Deflater class. >From Tomcat point of view that is a WONTFIX. It

[Bug 57837] add "text/css" to default of compressableMimeType

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 --- Comment #8 from romain.manni-bucau --- +1 for javascript as well -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-

[Bug 57837] add "text/css" to default of compressableMimeType

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 --- Comment #7 from mgrigorov --- I agree with Rainer that JavaScript should be whitelisted as well. @Mark: do you see problem by adding text/javascript and application/javascript to the list? -- You are receiving this mail because: You are

[Bug 57866] FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 j...@yahoo.com changed: What|Removed |Added Status|NEEDINFO|NEW --- Comment #2 from j...@yahoo.com

[Bug 57866] FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 Konstantin Kolinko changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #1 from Kons

buildbot exception in ASF Buildbot on tomcat-7-trunk

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

[Bug 57837] add "text/css" to default of compressableMimeType

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

svn commit: r1676484 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/coyote/http11/AbstractHttp11Protocol.java webapps/docs/changelog.xml webapps/docs/config/http.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 11:23:38 2015 New Revision: 1676484 URL: http://svn.apache.org/r1676484 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 Add text/css to the default for compressableMimeType Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk

svn commit: r1676483 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/coyote/http11/AbstractHttp11Protocol.java webapps/docs/changelog.xml webapps/docs/config/http.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 11:23:02 2015 New Revision: 1676483 URL: http://svn.apache.org/r1676483 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 Add text/css to the default for compressableMimeType Modified: tomcat/tc8.0.x/trunk/ (props changed) tomcat/tc8.0.x/trunk

svn commit: r1676479 - /tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 11:19:22 2015 New Revision: 1676479 URL: http://svn.apache.org/r1676479 Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=57837 Add text/css to the default for compressableMimeType Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol

svn commit: r1676477 - in /tomcat/trunk: java/org/apache/coyote/http11/AbstractHttp11Protocol.java java/org/apache/coyote/http11/Http11Processor.java webapps/docs/config/http.xml

2015-04-28 Thread markt
Author: markt Date: Tue Apr 28 11:18:32 2015 New Revision: 1676477 URL: http://svn.apache.org/r1676477 Log: Refactor compressableMimeType Convert comma separated string to string array once and re-use it rather than converting for every new Processor Modified: tomcat/trunk/java/org/apache/co

Fwd: [Bug 56438] If jar scan does not find context config or TLD config, log a message

2015-04-28 Thread Pravallika Peddi
Hi Mark, As you mentioned in issue thread, i have provided fix for TLD scan to inline Tomcat7 with 8.0 and trunk. Please review the patch and let me know if you see any problem. regards, pravallika -- Forwarded message -- From: Date: Tue, Apr 28, 2015 at 2:52 PM Subject: [Bug 564

[Bug 57866] FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 j...@yahoo.com changed: What|Removed |Added CC||j...@yahoo.com -- You are receiving t

[Bug 57866] New: FlushableGZIPOutputStream only compresses first flush()

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57866 Bug ID: 57866 Summary: FlushableGZIPOutputStream only compresses first flush() Product: Tomcat 7 Version: 7.0.54 Hardware: PC OS: Linux Status

[Bug 57865] session.invalidate does not work when during startup war file is deployed

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57865 Mark Thomas changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #2 from Mark Thomas

[Bug 56438] If jar scan does not find context config or TLD config, log a message

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56438 VIN changed: What|Removed |Added Attachment #32639|0 |1 is obsolete|

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

2015-04-28 Thread Bill Barker
ump/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-201504

[Bug 57823] Server hangs if there is a colon on the start of the class path

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57823 --- Comment #4 from gregh --- Agreed, there would be far too many permutations to check for, but not knowing that the colon causes it to hang is the key. Without debugging tomcat I was stuck as it would not start and nothing in the logs. Mayb

[Bug 57865] session.invalidate does not work when during startup war file is deployed

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57865 --- Comment #1 from Alessandro Trolli --- same webapp and configuration is working properly with version 7.0.59 -- You are receiving this mail because: You are the assignee for the bug. ---

[Bug 57865] New: session.invalidate does not work when during startup war file is deployed

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57865 Bug ID: 57865 Summary: session.invalidate does not work when during startup war file is deployed Product: Tomcat 7 Version: 7.0.61 Hardware: PC OS: Linux

[Bug 57864] New: channelSendOptions attribute is not able to be configured in hexadecimal

2015-04-28 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=57864 Bug ID: 57864 Summary: channelSendOptions attribute is not able to be configured in hexadecimal Product: Tomcat 8 Version: trunk Hardware: All OS: All