[Bug 66179] NPE when retrieving locale from request

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66179 Mark Thomas changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #8 from Mark

[Bug 66174] Pound sign £ in passwords does not work with LDAP realm

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66174 Mark Thomas changed: What|Removed |Added Resolution|--- |INVALID Status|NEEDINFO

[GitHub] [tomcat-jakartaee-migration] wmccusker commented on a diff in pull request #33: fixes #32 check if file name ends with manifest name

2022-08-22 Thread GitBox
wmccusker commented on code in PR #33: URL: https://github.com/apache/tomcat-jakartaee-migration/pull/33#discussion_r951564802 ## src/test/java/org/apache/tomcat/jakartaee/ManifestConverterTest.java: ## @@ -0,0 +1,16 @@ +package org.apache.tomcat.jakartaee; + +import static

[GitHub] [tomcat-jakartaee-migration] wmccusker commented on a diff in pull request #33: fixes #32 check if file name ends with manifest name

2022-08-22 Thread GitBox
wmccusker commented on code in PR #33: URL: https://github.com/apache/tomcat-jakartaee-migration/pull/33#discussion_r951564802 ## src/test/java/org/apache/tomcat/jakartaee/ManifestConverterTest.java: ## @@ -0,0 +1,16 @@ +package org.apache.tomcat.jakartaee; + +import static

[GitHub] [tomcat-jakartaee-migration] wmccusker commented on a diff in pull request #33: fixes #32 check if file name ends with manifest name

2022-08-22 Thread GitBox
wmccusker commented on code in PR #33: URL: https://github.com/apache/tomcat-jakartaee-migration/pull/33#discussion_r951565338 ## src/test/java/org/apache/tomcat/jakartaee/ManifestConverterTest.java: ## @@ -0,0 +1,16 @@ +package org.apache.tomcat.jakartaee; + +import static

[GitHub] [tomcat-jakartaee-migration] wmccusker commented on a diff in pull request #33: fixes #32 check if file name ends with manifest name

2022-08-22 Thread GitBox
wmccusker commented on code in PR #33: URL: https://github.com/apache/tomcat-jakartaee-migration/pull/33#discussion_r951564802 ## src/test/java/org/apache/tomcat/jakartaee/ManifestConverterTest.java: ## @@ -0,0 +1,16 @@ +package org.apache.tomcat.jakartaee; + +import static

[Bug 66120] j_security_check returns 408 if j_security_check request lands on different tomcat server from original server

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66120 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[tomcat] branch 8.5.x updated: Fix BZ 66120

2022-08-22 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 e1e55f954a Fix BZ 66120 e1e55f954a is described

[tomcat] branch 9.0.x updated: Fix BZ 66120

2022-08-22 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 15be93dbae Fix BZ 66120 15be93dbae is described

[tomcat] branch 10.0.x updated: Fix BZ 66120

2022-08-22 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.0.x by this push: new c2493ab782 Fix BZ 66120 c2493ab782 is described

[tomcat] branch main updated: Fix BZ 66120

2022-08-22 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 d69935a118 Fix BZ 66120 d69935a118 is described

[Bug 66120] j_security_check returns 408 if j_security_check request lands on different tomcat server from original server

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66120 --- Comment #6 from Mark Thomas --- Slight change of plan. 10.1.x will just fix this (the 10.1.x releases are still milestones). 10.0.x and earlier will fix this but sending the new messages will be controlled by configuration and disabled

JDK 19 first Release Candidates!

2022-08-22 Thread David Delabassee
Greetings! I hope you had a chance to take some time off. On our side, and despite the summer vacation, everything is on track for the Java 19 GA release on September 20th with JDK 19 now in the Release Candidate Phase [1]. If you haven't done so yet, it is time to start testing your

[tomcat] branch 8.5.x updated: Log a warning rather than throw for an unrecognised action type

2022-08-22 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 aef3b97177 Log a warning rather than throw for an

[tomcat] branch 9.0.x updated: Log a warning rather than throw for an unrecognised action type

2022-08-22 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 e200fa31de Log a warning rather than throw for an

[tomcat] branch 10.0.x updated: Log a warning rather than throw for an unrecognised action type

2022-08-22 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.0.x by this push: new 62769e9ea5 Log a warning rather than throw for

[tomcat] branch main updated: Log a warning rather than throw for an unrecognised action type

2022-08-22 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 3e19abcd5d Log a warning rather than throw for an

[tomcat] branch 8.5.x updated: Clean-up - no functional change

2022-08-22 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 2e899f9e5f Clean-up - no functional change

[tomcat] branch main updated: Clean-up - no functional change

2022-08-22 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 fda1d323b6 Clean-up - no functional change

[tomcat] branch 9.0.x updated: Clean-up - no functional change

2022-08-22 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 6f15fcd19c Clean-up - no functional change

[tomcat] branch 10.0.x updated: Clean-up - no functional change

2022-08-22 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.0.x by this push: new 0f7ba7ccf5 Clean-up - no functional change

[Bug 66209] CPU regression when classpath Bloom filters are active

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66209 --- Comment #9 from John Engebretson --- Sounds like we have general agreement on updating the configuration switch to contain more than just true/false, so the remaining questions are: 1. Do we indeed agree? 2. Should I do the updated patch,

Re: jakarta.el, useless getResource?

2022-08-22 Thread Romain Manni-Bucau
+1 To answer the proxy reference: it affects other cases - loading classes from a "database", proxies is just a well known case I used to illustrate my point. By contract a classloader is not always an URLClassLoader which is the assumption of the impl right now. Also CDS changes the perf there

Re: jakarta.el, useless getResource?

2022-08-22 Thread Mark Thomas
On 22/08/2022 11:48, Mark Thomas wrote: On 22/08/2022 10:20, Romain Manni-Bucau wrote: So overall I wonder if this check can be dropped now we have concurrent classloaders and cache almost everywhere. If not, should the missed items be cached in some (webapp) classloader to help to exit

Re: jakarta.el, useless getResource?

2022-08-22 Thread Mark Thomas
On 22/08/2022 10:20, Romain Manni-Bucau wrote: Hi all, I just spotted that in jakarta.el.ImportHandler#findClass there is a cl.getResource(path) == null (similar code exists when there is a security manager). One line later there is a clazz = cl.loadClass(name);. I assume the intent is to

[Bug 66179] NPE when retrieving locale from request

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66179 simon.lu...@sap.com changed: What|Removed |Added Status|NEEDINFO|NEW --- Comment #7 from

[Bug 66179] NPE when retrieving locale from request

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66179 --- Comment #6 from simon.lu...@sap.com --- Created attachment 38372 --> https://bz.apache.org/bugzilla/attachment.cgi?id=38372=edit RequestTracingValve.java -- You are receiving this mail because: You are the assignee for the bug.

[GitHub] [tomcat] aooohan opened a new pull request, #546: Bugfix for 66233 - Response error msg when the number of Cookie exceeds maxCookieCount

2022-08-22 Thread GitBox
aooohan opened a new pull request, #546: URL: https://github.com/apache/tomcat/pull/546 Fix [BZ66233](https://bz.apache.org/bugzilla/show_bug.cgi?id=66233) -- 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

[Bug 66233] when set maxCookieCount 1, request has 5 cookies, the response message is "null"

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66233 --- Comment #1 from Han Li --- Sounds great. Thanks for your feedback. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail:

jakarta.el, useless getResource?

2022-08-22 Thread Romain Manni-Bucau
Hi all, I just spotted that in jakarta.el.ImportHandler#findClass there is a cl.getResource(path) == null (similar code exists when there is a security manager). One line later there is a clazz = cl.loadClass(name);. I assume the intent is to check the class is physically accessible but I have a

[tomcat] branch 8.5.x updated: Preparation for fixing BZ 66120

2022-08-22 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 e1eb363192 Preparation for fixing BZ 66120

[tomcat] branch 9.0.x updated: Preparation for fixing BZ 66120

2022-08-22 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 4acf70326a Preparation for fixing BZ 66120

[tomcat] branch main updated: Preparation for fixing BZ 66120

2022-08-22 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 65d9de34ac Preparation for fixing BZ 66120

[tomcat] branch 10.0.x updated: Preparation for fixing BZ 66120

2022-08-22 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.0.x by this push: new 2b03b7f29c Preparation for fixing BZ 66120

[Bug 66233] New: when set maxCookieCount 1, request has 5 cookies, the response message is "null"

2022-08-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=66233 Bug ID: 66233 Summary: when set maxCookieCount 1, request has 5 cookies, the response message is "null" Product: Tomcat 10 Version: unspecified Hardware: All