[Bug 66533] Wrapping Requests or Responses in Valves is not possible

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66533 --- Comment #2 from Wolfgang Illmeyer --- Ok, after playing around with Valves for a little, I would advise against actually using HttpServletRequest & HttpServletResponse and instead opt for extracting an interface from the currently used Requ

[Bug 66541] CachedResource for OSGi URL resources changes URL hashing behavior & exacerbates DNS issues

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66541 --- Comment #7 from Mark Thomas --- (In reply to Tom Whitmore from comment #6) > Thanks. The "wrapped one" is the outer one; I use wrapped & underlying as a > frequent terminology, it works well & is clear semantically. On that we are going to

[tomcat] branch main updated: Fix logic bug - with thanks to Tom Whitmore.

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new e4c0f1fe3e Fix logic bug - with thanks to Tom Whitmor

[tomcat] branch 10.1.x updated: Fix logic bug - with thanks to Tom Whitmore.

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new af62dea83f Fix logic bug - with thanks to Tom Whi

[tomcat] branch 9.0.x updated: Fix logic bug - with thanks to Tom Whitmore.

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new baeeddce3b Fix logic bug - with thanks to Tom Whitm

[tomcat] branch 8.5.x updated: Fix logic bug - with thanks to Tom Whitmore.

2023-03-27 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 f1295ac538 Fix logic bug - with thanks to Tom Whitm

[tomcat] branch main updated: Improve naming - based a patch by Tom Whitmore

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 71b6644bce Improve naming - based a patch by Tom Whit

[tomcat] branch 10.1.x updated: Improve naming - based a patch by Tom Whitmore

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new da5d1eef78 Improve naming - based a patch by Tom

[tomcat] branch 9.0.x updated: Improve naming - based a patch by Tom Whitmore

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 9626aa78f7 Improve naming - based a patch by Tom Wh

[tomcat] branch 8.5.x updated: Improve naming - based a patch by Tom Whitmore

2023-03-27 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 23cc5bcdb0 Improve naming - based a patch by Tom Wh

[tomcat] branch main updated: Fix comment typo

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 628bb0ee10 Fix comment typo 628bb0ee10 is described b

[tomcat] branch 10.1.x updated: Fix comment typo

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new af4a4dbe5a Fix comment typo af4a4dbe5a is describ

[tomcat] branch 9.0.x updated: Fix comment typo

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 3578c93597 Fix comment typo 3578c93597 is described

[tomcat] branch 8.5.x updated: Fix comment typo

2023-03-27 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 6d3415e090 Fix comment typo 6d3415e090 is described

[Bug 66541] CachedResource for OSGi URL resources changes URL hashing behavior & exacerbates DNS issues

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66541 Mark Thomas changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Bug 66544] java.lang.ArrayIndexOutOfBoundsException @ rg.apache.coyote.http11.Http11Processor.service Error processing request

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66544 --- Comment #2 from Mark Thomas --- I don't see how this could be a Tomcat bug. All the indications are that this is a JVM bug. I won't close this just yet, but it is likely to be closed in the next few days. -- You are receiving this mail be

[GitHub] [tomcat] markt-asf merged pull request #604: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent

2023-03-27 Thread via GitHub
markt-asf merged PR #604: URL: https://github.com/apache/tomcat/pull/604 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.o

[tomcat] branch main updated: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 0ac33836c3 Fix Bug 66543 - Wrong log message used in

[tomcat] branch 9.0.x updated: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 9f00d9fba7 Fix Bug 66543 - Wrong log message used

[tomcat] branch 8.5.x updated: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent

2023-03-27 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 4b201dff58 Fix Bug 66543 - Wrong log message used

[Bug 66543] Misleading error log in fireRequestDestroyEvent

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66543 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[tomcat] branch 10.1.x updated: Fix Bug 66543 - Wrong log message used in StandardContext#fireRequestDestroyEvent

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new e7c450e994 Fix Bug 66543 - Wrong log message use

[tomcat] branch main updated: Fix BZ 66542 - Update references to JavaMail to use Jakarta Mail

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new a6339b1405 Fix BZ 66542 - Update references to JavaMa

[tomcat] branch 10.1.x updated: Fix BZ 66542 - Update references to JavaMail to use Jakarta Mail

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new b441ffe9e4 Fix BZ 66542 - Update references to Ja

[Bug 66542] JavaMail specification links are outdated in documentation

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66542 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 66505] Calling [asyncPostProcess()] is not valid for a request with Async state [ERROR]

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66505 Mark Thomas changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEEDINFO

[Bug 66541] CachedResource for OSGi URL resources changes URL hashing behavior & exacerbates DNS issues

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66541 --- Comment #9 from Tom Whitmore --- Great, thanks Mark. * I think 'cacheURL' and 'resourceURL' sound reasonably clearly distinct. * I agree the 'url == cacheURL' logic is necessary; relative resource URL objects can be created inheriting the

[Bug 66541] CachedResource for OSGi URL resources changes URL hashing behavior & exacerbates DNS issues

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66541 --- Comment #10 from Mark Thomas --- I have updated https://people.apache.org/~markt/dev/v8.5.88-dev/ with a new test / dev build. Same caveats apply. -- You are receiving this mail because: You are the assignee for the bug. -

[Bug 66508] Tomcat after a GC pause causes the HTTP threads to be blocked to acquire a semaphore to process WebSockets connection closure.

2023-03-27 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66508 --- Comment #18 from Mark Thomas --- Thanks for the Connector settings. The stack traces will definitely be useful as it looks as if you are using useAsyncIO="true". If correct, I definitely want to see how this is being triggered. As an alter

[tomcat] 01/01: Further fix for BZ 66508

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch fix-bz-66508 in repository https://gitbox.apache.org/repos/asf/tomcat.git commit f09171e654dfedcdf9657973427bd9c62b732ad0 Author: Mark Thomas AuthorDate: Fri Mar 24 17:21:04 2023 + Furthe

[tomcat] branch fix-bz-66508 created (now f09171e654)

2023-03-27 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch fix-bz-66508 in repository https://gitbox.apache.org/repos/asf/tomcat.git at f09171e654 Further fix for BZ 66508 This branch includes the following new commits: new f09171e654 Further

[GitHub] [tomcat] markt-asf opened a new pull request, #605: Further fix for BZ 66508

2023-03-27 Thread via GitHub
markt-asf opened a new pull request, #605: URL: https://github.com/apache/tomcat/pull/605 https://bz.apache.org/bugzilla/show_bug.cgi?id=66508 Avoid deadlock for close messages when WsRemoteEndpointImplServer.endMessage() for a previous message is processed on a container thread

[GitHub] [tomcat] markt-asf commented on pull request #605: Further fix for BZ 66508

2023-03-27 Thread via GitHub
markt-asf commented on PR #605: URL: https://github.com/apache/tomcat/pull/605#issuecomment-1486301345 Sorry. Branch in wrong repo. I'll resubmit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [tomcat] markt-asf closed pull request #605: Further fix for BZ 66508

2023-03-27 Thread via GitHub
markt-asf closed pull request #605: Further fix for BZ 66508 URL: https://github.com/apache/tomcat/pull/605 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-ma

[GitHub] [tomcat] markt-asf opened a new pull request, #606: Further fix for BZ 66508

2023-03-27 Thread via GitHub
markt-asf opened a new pull request, #606: URL: https://github.com/apache/tomcat/pull/606 https://bz.apache.org/bugzilla/show_bug.cgi?id=66508 Avoid deadlock for close messages when WsRemoteEndpointImplServer.endMessage() for a previous message is processed on a container thread

Re: [GitHub] [tomcat] markt-asf opened a new pull request, #606: Further fix for BZ 66508

2023-03-27 Thread Mark Thomas
On 28/03/2023 07:44, markt-asf (via GitHub) wrote: markt-asf opened a new pull request, #606: URL: https://github.com/apache/tomcat/pull/606 https://bz.apache.org/bugzilla/show_bug.cgi?id=66508 I'd really like to get this fixed for the April releases but the fix is non-trivial. Review wo