[tomcat] branch 8.5.x updated: Refactor. Narrow scope of try block.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 3425355 Refactor. Narrow scope of try block. 34253

[tomcat] branch master updated: Refactor. Narrow scope of try block.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 42aaf6d Refactor. Narrow scope of try block. 42a

[Bug 63324] CrawlerSessionManagerValve is getting put into Session, which causes problems when serializing sessions

2019-04-30 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=63324 Martin L changed: What|Removed |Added CC||martin.lemanski@willhaben.a

[tomcat] branch master updated: Simplify code, cleanup and improve comments

2019-04-30 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new de57438 Simplify code, cleanup and improve commen

Re: catalina.nonblocking.TestNonBlockingAPI

2019-04-30 Thread Mark Thomas
On 30/04/2019 12:48, Rémy Maucherat wrote: > Hi, > > There's something odd with this test: > https://ci.apache.org/projects/tomcat/tomcat9/logs/4265/TEST-org.apache.catalina.nonblocking.TestNonBlockingAPI.APR.txt > https://ci.apache.org/projects/tomcat/tomcat9/logs/4265/TEST-org.apache.catalina.no

[tomcat] branch master updated: Add --no-jmx flag

2019-04-30 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 21ca769 Add --no-jmx flag 21ca769 is described be

catalina.nonblocking.TestNonBlockingAPI

2019-04-30 Thread Rémy Maucherat
Hi, There's something odd with this test: https://ci.apache.org/projects/tomcat/tomcat9/logs/4265/TEST-org.apache.catalina.nonblocking.TestNonBlockingAPI.APR.txt https://ci.apache.org/projects/tomcat/tomcat9/logs/4265/TEST-org.apache.catalina.nonblocking.TestNonBlockingAPI.NIO.txt https://ci.apach

[GitHub] [tomcat] markt-asf commented on issue #161: Early check end of input condition to avoid ArrayIndexOutOfBoundsExceptions

2019-04-30 Thread GitBox
markt-asf commented on issue #161: Early check end of input condition to avoid ArrayIndexOutOfBoundsExceptions URL: https://github.com/apache/tomcat/pull/161#issuecomment-487916490 Thanks for the PR. I applied (rather than merged it) along with a update to the changelog that included credi

[GitHub] [tomcat] markt-asf closed pull request #161: Early check end of input condition to avoid ArrayIndexOutOfBoundsExceptions

2019-04-30 Thread GitBox
markt-asf closed pull request #161: Early check end of input condition to avoid ArrayIndexOutOfBoundsExceptions URL: https://github.com/apache/tomcat/pull/161 This is an automated message from the Apache Git Service. To resp

[tomcat] branch 7.0.x updated: Refactor Hostname validation to improve performance.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 7.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/7.0.x by this push: new 5c31b92 Refactor Hostname validation to improve pe

[tomcat] branch 8.5.x updated: Refactor Hostname validation to improve performance.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 2ef9f33 Refactor Hostname validation to improve pe

[tomcat] branch master updated: Refactor Hostname validation to improve performance.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 5baec17 Refactor Hostname validation to improve

buildbot success in on tomcat-trunk

2019-04-30 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while building tomcat. Full details are available at: https://ci.apache.org/builders/tomcat-trunk/builds/4264 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: silvanus_ubuntu Build Reason: The AnyBranchSchedul

[GitHub] [tomcat] markt-asf commented on issue #157: https://bz.apache.org/bugzilla/show_bug.cgi?id=63333

2019-04-30 Thread GitBox
markt-asf commented on issue #157: https://bz.apache.org/bugzilla/show_bug.cgi?id=6 URL: https://github.com/apache/tomcat/pull/157#issuecomment-487898377 I've applied this. I didn't merge the PR directly as I wanted to tweak some of the formatting but the functional aspects remained un

[GitHub] [tomcat] markt-asf closed pull request #157: https://bz.apache.org/bugzilla/show_bug.cgi?id=63333

2019-04-30 Thread GitBox
markt-asf closed pull request #157: https://bz.apache.org/bugzilla/show_bug.cgi?id=6 URL: https://github.com/apache/tomcat/pull/157 This is an automated message from the Apache Git Service. To respond to the message, ple

[Bug 63333] JAASRealm needs to override isAvailable method to prevent LockOutRealm to lock the user in case JAAS login modules are unavailable

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

[tomcat] branch 8.5.x updated: Fix indents (automated via Eclipse). No functional change.

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new ed56580 Fix indents (automated via Eclipse). No fu

[tomcat] branch 8.5.x updated: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63333

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new f83d0e1 Fix https://bz.apache.org/bugzilla/show_bu

Re: Missing git commit emails

2019-04-30 Thread Mark Thomas
On 30/04/2019 09:17, Rainer Jung wrote: > What could we do to avoid those summary emails, more precisely to > reliably get the detail emails? Currently a review via email often is > not possible due to the missing detail emails. I think it is related to pushing multiple commits at once. Best bet

[tomcat] branch master updated (f1cf343 -> 5b68fa0)

2019-04-30 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git. from f1cf343 Cleanup the preface read code new 0bba249 Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=6

[tomcat] branch master updated: Cleanup the preface read code

2019-04-30 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new f1cf343 Cleanup the preface read code f1cf343 is

buildbot failure in on tomcat-trunk

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

svn commit: r1858412 - in /tomcat/site/trunk: docs/ xdocs/stylesheets/

2019-04-30 Thread markt
Author: markt Date: Tue Apr 30 09:27:15 2019 New Revision: 1858412 URL: http://svn.apache.org/viewvc?rev=1858412&view=rev Log: Accessibility improvements Modified: tomcat/site/trunk/docs/bugreport.html tomcat/site/trunk/docs/ci.html tomcat/site/trunk/docs/conference.html tomcat/si

[tomcat] branch master updated: Async version of HTTP/2 preface and initial frame read

2019-04-30 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 50fbf98 Async version of HTTP/2 preface and initi

[tomcat] branch master updated: Cleanup

2019-04-30 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 2eb64d1 Cleanup 2eb64d1 is described below commi

Missing git commit emails

2019-04-30 Thread Rainer Jung
What could we do to avoid those summary emails, more precisely to reliably get the detail emails? Currently a review via email often is not possible due to the missing detail emails. I wonder whether the occurence of these "no details, instead just a summary" emails has anything to do with the