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

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

[Bug 60469] RealmBase#hasRole() imposes too much boilerplate code if principal is not of type GenericPrincipal

https://bz.apache.org/bugzilla/show_bug.cgi?id=60469 --- Comment #7 from Michael Osipov <1983-01...@gmx.net> --- (In reply to Mark Thomas from comment #6) > Fixed in: > - trunk for 9.0.0.M18 onwards > - 8.5.x for 8.5.12 onwards Per(In reply to Mark Thomas from comment #6) > Fixed in: > - trunk

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

/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-20170303.jar -Dtest.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20170303/bin/openssl -Dexecute.test.nio=false

[Bug 60808] ServletRequest.getParameterMap() not fully immutable

https://bz.apache.org/bugzilla/show_bug.cgi?id=60808 --- Comment #4 from Woonsan Ko --- I've created a new PR: https://github.com/apache/tomcat/pull/47, following Mark's suggestion to use containment instead of inheritance. Indeed, it helps reduce code size and safer in the

[GitHub] tomcat pull request #47: Make ParameterMap immutable using containment inste...

GitHub user woonsan opened a pull request: https://github.com/apache/tomcat/pull/47 Make ParameterMap immutable using containment instead of inheritance. You can merge this pull request into a Git repository by running: $ git pull https://github.com/woonsan/tomcat

[GitHub] tomcat pull request #46: Bug_60808: ensure immutability of ParameterMap.

Github user woonsan closed the pull request at: https://github.com/apache/tomcat/pull/46 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] tomcat issue #46: Bug_60808: ensure immutability of ParameterMap.

Github user woonsan commented on the issue: https://github.com/apache/tomcat/pull/46 I'll propose a new PR using containment, not inheritance, as @markt-asf suggested. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[Bug 60808] ServletRequest.getParameterMap() not fully immutable

https://bz.apache.org/bugzilla/show_bug.cgi?id=60808 --- Comment #3 from Woonsan Ko --- Hi Mark, Yes, that makes more sense to use containment instead of extending. Even if newer JVM has more new API support in the future, ParameterMap implementing j.u.Map with a contained

Re: JDK 9 EA Build 159 and JDK 8u152 is available on java.net

Thanks Mark. On 03/03/2017 14:54, Mark Thomas wrote: On 03/03/17 13:12, Rory O'Donnell wrote: Hi Mark, There is a second bug that we would like to get fix confirmation on: JDK-8172726 : ForkJoin common pool retains a reference to the thread context class loader Fix confirmed with b159.

buildbot success in on tomcat-trunk

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/2145 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchScheduler

svn commit: r1785318 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/realm/RealmBase.java

Author: markt Date: Fri Mar 3 15:00:42 2017 New Revision: 1785318 URL: http://svn.apache.org/viewvc?rev=1785318=rev Log: Use inherited Javadoc where available. Add missing @return Modified: tomcat/tc8.5.x/trunk/ (props changed)

svn commit: r1785317 - /tomcat/trunk/java/org/apache/catalina/realm/RealmBase.java

Author: markt Date: Fri Mar 3 14:59:41 2017 New Revision: 1785317 URL: http://svn.apache.org/viewvc?rev=1785317=rev Log: Use inherited Javadoc where available. Add missing @return Modified: tomcat/trunk/java/org/apache/catalina/realm/RealmBase.java Modified:

Re: JDK 9 EA Build 159 and JDK 8u152 is available on java.net

On 03/03/17 13:12, Rory O'Donnell wrote: > Hi Mark, > > There is a second bug that we would like to get fix confirmation on: > > JDK-8172726 : ForkJoin common pool retains a reference to the thread > context class loader Fix confirmed with b159. >> Hi Mark, >> >> *JDK 9 Early Access* b159

buildbot failure in on tomcat-trunk

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/2144 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchScheduler

[Bug 60469] RealmBase#hasRole() imposes too much boilerplate code if principal is not of type GenericPrincipal

https://bz.apache.org/bugzilla/show_bug.cgi?id=60469 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED

svn commit: r1785312 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/realm/RealmBase.java webapps/docs/changelog.xml

Author: markt Date: Fri Mar 3 14:25:06 2017 New Revision: 1785312 URL: http://svn.apache.org/viewvc?rev=1785312=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60469 Refactor RealmBase for better code re-use when implementing Realms that use a custom Principal. Modified:

svn commit: r1785310 - in /tomcat/trunk: java/org/apache/catalina/realm/RealmBase.java webapps/docs/changelog.xml

Author: markt Date: Fri Mar 3 14:24:16 2017 New Revision: 1785310 URL: http://svn.apache.org/viewvc?rev=1785310=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60469 Refactor RealmBase for better code re-use when implementing Realms that use a custom Principal. Modified:

Re: JDK 9 EA Build 159 and JDK 8u152 is available on java.net

Hi Mark, There is a second bug that we would like to get fix confirmation on: JDK-8172726 : ForkJoin common pool retains a reference to the thread context class loader Thanks,Rory On 03/03/2017 11:23, Rory O'Donnell wrote: Hi Mark, *JDK 9 Early Access* b159

[Bug 60560] Support systemd/inetd style socket activation

https://bz.apache.org/bugzilla/show_bug.cgi?id=60560 --- Comment #4 from Mark Thomas --- Documentation is in trunk/webapps/docs -- You are receiving this mail because: You are the assignee for the bug. - To

[Bug 60751] Add 100.64/10 to the default internalProxies

https://bz.apache.org/bugzilla/show_bug.cgi?id=60751 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED

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

https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED

svn commit: r1785278 - in /tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/http: TestCookiesDefaultSysProps.java TestCookiesNoFwdStrictSysProps.java TestCookiesNoStrictNamingSysProps.java TestCookies

Author: markt Date: Fri Mar 3 12:07:02 2017 New Revision: 1785278 URL: http://svn.apache.org/viewvc?rev=1785278=rev Log: Remove unnecessary (according to Eclipse) @SuppressWarnings("javadoc") Modified: tomcat/tc8.0.x/trunk/test/org/apache/tomcat/util/http/TestCookiesDefaultSysProps.java

svn commit: r1785277 - in /tomcat/trunk/test/org/apache/tomcat/util/http: TestCookiesDefaultSysProps.java TestCookiesNoFwdStrictSysProps.java TestCookiesNoStrictNamingSysProps.java TestCookiesStrictSy

Author: markt Date: Fri Mar 3 12:04:21 2017 New Revision: 1785277 URL: http://svn.apache.org/viewvc?rev=1785277=rev Log: Update reference to correct location of static fields. Remove unnecessary @SuppressWarnings Modified:

svn commit: r1785273 - in /tomcat/tc8.0.x/trunk: ./ java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java test/org/apache/tomcat/util/scan/TestAbstractInputStreamJar.java webapps/docs/changelog

Author: markt Date: Fri Mar 3 11:56:14 2017 New Revision: 1785273 URL: http://svn.apache.org/viewvc?rev=1785273=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Correct a bug in the handling of JARs in unpacked WARs that meant multiple attempts to read the same entry from a JAR

svn commit: r1785272 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java test/org/apache/tomcat/util/scan/TestAbstractInputStreamJar.java webapps/docs/changelog

Author: markt Date: Fri Mar 3 11:55:27 2017 New Revision: 1785272 URL: http://svn.apache.org/viewvc?rev=1785272=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Correct a bug in the handling of JARs in unpacked WARs that meant multiple attempts to read the same entry from a JAR

svn commit: r1785271 - in /tomcat/trunk: java/org/apache/tomcat/util/scan/AbstractInputStreamJar.java test/org/apache/tomcat/util/scan/TestAbstractInputStreamJar.java webapps/docs/changelog.xml

Author: markt Date: Fri Mar 3 11:54:46 2017 New Revision: 1785271 URL: http://svn.apache.org/viewvc?rev=1785271=rev Log: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60798 Correct a bug in the handling of JARs in unpacked WARs that meant multiple attempts to read the same entry from a JAR

JDK 9 EA Build 159 and JDK 8u152 is available on java.net

Hi Mark, *JDK 9 Early Access* b159 is available on java.net, summary of changes are listed here . Can you confirm fix in b159 for JDK-8175261 : Per-protocol cache setting not working for JAR

[Bug 60808] ServletRequest.getParameterMap() not fully immutable

https://bz.apache.org/bugzilla/show_bug.cgi?id=60808 --- Comment #2 from Mark Thomas --- Generally, I think it would be safer (in terms of future additions to Map and changes to the implementation of LinkedHashMap) and probably require less code if ParameterMap used composition

svn commit: r1785252 - /tomcat/trunk/java/javax/servlet/http/PushBuilder.java

Author: markt Date: Fri Mar 3 09:21:18 2017 New Revision: 1785252 URL: http://svn.apache.org/viewvc?rev=1785252=rev Log: ws Modified: tomcat/trunk/java/javax/servlet/http/PushBuilder.java Modified: tomcat/trunk/java/javax/servlet/http/PushBuilder.java URL:

svn commit: r1785248 - in /tomcat/tc8.5.x/trunk: ./ java/org/apache/catalina/core/ApplicationPushBuilder.java java/org/apache/catalina/core/LocalStrings.properties java/org/apache/catalina/servlet4pre

Author: markt Date: Fri Mar 3 08:56:59 2017 New Revision: 1785248 URL: http://svn.apache.org/viewvc?rev=1785248=rev Log: Servlet 4.0 Additional validation for HTTP method used for server push Modified: tomcat/tc8.5.x/trunk/ (props changed)

svn commit: r1785245 - in /tomcat/trunk/java: javax/servlet/http/PushBuilder.java org/apache/catalina/core/ApplicationPushBuilder.java org/apache/catalina/core/LocalStrings.properties

Author: markt Date: Fri Mar 3 08:55:24 2017 New Revision: 1785245 URL: http://svn.apache.org/viewvc?rev=1785245=rev Log: Servlet 4.0 Additional validation for HTTP method used for server push Modified: tomcat/trunk/java/javax/servlet/http/PushBuilder.java

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

/main/target/objenesis-2.6-SNAPSHOT.jar -Dexamples.sources.skip=true -Dcommons-daemon.jar=/srv/gump/public/workspace/apache-commons/daemon/dist/commons-daemon-20170303.jar -Dtest.openssl.path=/srv/gump/public/workspace/openssl-1.0.2/dest-20170303/bin/openssl -Dexecute.test.nio=false -Dhamcrest.jar