Re: Cookie parsing and upcoming updates to RFC6265

2024-08-16 Thread Mark Thomas

On 14/08/2024 19:12, Konstantin Kolinko wrote:




I think that
1) We would better switch to "ignore" mode right now, in all supported versions.


Based on past experience I am extremely hesitant to change anything 
related to cookie handling behaviour unless we have to. I'd prefer to 
use "name" as the default more for 9.0.x and 10.1.x.


I'm prepared to be convinced otherwise though.



2) The "empty name" option seems to be the correct behaviour,
But as the majority of web applications would not need this feature,
I think that\ "ignore" would better remain the default behaviour.


No objections to that.

I'm going to start working on this "noEqualsCookie" option. Suggestions 
for a better name still welcome.


Mark

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



(tomcat) branch 9.0.x updated: Update excludes list

2024-08-16 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 414d50614d Update excludes list
414d50614d is described below

commit 414d50614dc6c82971b391620bd6366c424e13ab
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:10:05 2024 +0100

Update excludes list
---
 res/rat/rat-excludes.txt | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index ea3d721a98..aeea015af8 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -46,7 +46,7 @@
 
   - files used simply to ensure directories are not empty
 
-  - *.bmp, *.dia files are binary
+  - *.bmp, *.dia and *.vpd files are binary
 (*.gif, *.jpg are also binary, but are automatically detected by RAT as
 ones, so no explicit configuration is needed)
 
@@ -100,6 +100,7 @@ output/dist/src/test/webapp/bug53257/*.txt
 output/dist/src/test/webapp/bug53257/foo bar/foobar.txt
 output/dist/src/test/webapp/bug66609/a&a.txt
 output/dist/src/test/webapp/bug66609/b'b.txt
+output/dist/src/test/webapp/jsp/ok.html
 output/dist/src/test/webapp-fragments/WEB-INF/classes/*.txt
 output/dist/src/test/webresources/dir1/d1/d1-f1.txt
 output/dist/src/test/webresources/dir1/d2/d2-f1.txt
@@ -118,6 +119,7 @@ test/webapp/bug53257/*.txt
 test/webapp/bug53257/foo bar/foobar.txt
 test/webapp/bug66609/a&a.txt
 test/webapp/bug66609/b'b.txt
+test/webapp/jsp/ok.html
 test/webapp-fragments/WEB-INF/classes/*.txt
 test/webresources/dir1/d1/d1-f1.txt
 test/webresources/dir1/d2/d2-f1.txt
@@ -205,6 +207,7 @@ output/dist/temp/safeToDelete.tmp
 
 **/*.bmp
 **/*.dia
+**/*.vpd
 
 **/*.md
 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



(tomcat) branch 10.1.x updated: Update excludes list

2024-08-16 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 cfde3f9794 Update excludes list
cfde3f9794 is described below

commit cfde3f9794cd8a9874717079fe94624227979ed3
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:10:05 2024 +0100

Update excludes list
---
 res/rat/rat-excludes.txt | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index f4dc296306..e4460c1f65 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -46,7 +46,7 @@
 
   - files used simply to ensure directories are not empty
 
-  - *.bmp, *.dia files are binary
+  - *.bmp, *.dia and *.vpd files are binary
 (*.gif, *.jpg are also binary, but are automatically detected by RAT as
 ones, so no explicit configuration is needed)
 
@@ -100,6 +100,7 @@ output/dist/src/test/webapp/bug53257/*.txt
 output/dist/src/test/webapp/bug53257/foo bar/foobar.txt
 output/dist/src/test/webapp/bug66609/a&a.txt
 output/dist/src/test/webapp/bug66609/b'b.txt
+output/dist/src/test/webapp/jsp/ok.html
 output/dist/src/test/webapp-fragments/WEB-INF/classes/*.txt
 output/dist/src/test/webresources/dir1/d1/d1-f1.txt
 output/dist/src/test/webresources/dir1/d2/d2-f1.txt
@@ -118,6 +119,7 @@ test/webapp/bug53257/*.txt
 test/webapp/bug53257/foo bar/foobar.txt
 test/webapp/bug66609/a&a.txt
 test/webapp/bug66609/b'b.txt
+test/webapp/jsp/ok.html
 test/webapp-fragments/WEB-INF/classes/*.txt
 test/webresources/dir1/d1/d1-f1.txt
 test/webresources/dir1/d2/d2-f1.txt
@@ -248,6 +250,7 @@ output/dist/temp/safeToDelete.tmp
 
 **/*.bmp
 **/*.dia
+**/*.vpd
 
 **/*.md
 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



(tomcat) branch main updated: Update excludes list

2024-08-16 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 9474cd9fe5 Update excludes list
9474cd9fe5 is described below

commit 9474cd9fe5b2bd7c5541b556a5cfb2d8fe9c4530
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:10:05 2024 +0100

Update excludes list
---
 res/rat/rat-excludes.txt | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index 16cac64276..6d6957b924 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -46,7 +46,7 @@
 
   - files used simply to ensure directories are not empty
 
-  - *.bmp, *.dia files are binary
+  - *.bmp, *.dia and *.vpd files are binary
 (*.gif, *.jpg are also binary, but are automatically detected by RAT as
 ones, so no explicit configuration is needed)
 
@@ -99,6 +99,7 @@ output/dist/src/test/webapp/bug53257/*.txt
 output/dist/src/test/webapp/bug53257/foo bar/foobar.txt
 output/dist/src/test/webapp/bug66609/a&a.txt
 output/dist/src/test/webapp/bug66609/b'b.txt
+output/dist/src/test/webapp/jsp/ok.html
 output/dist/src/test/webapp-fragments/WEB-INF/classes/*.txt
 output/dist/src/test/webresources/dir1/d1/d1-f1.txt
 output/dist/src/test/webresources/dir1/d2/d2-f1.txt
@@ -117,6 +118,7 @@ test/webapp/bug53257/*.txt
 test/webapp/bug53257/foo bar/foobar.txt
 test/webapp/bug66609/a&a.txt
 test/webapp/bug66609/b'b.txt
+test/webapp/jsp/ok.html
 test/webapp-fragments/WEB-INF/classes/*.txt
 test/webresources/dir1/d1/d1-f1.txt
 test/webresources/dir1/d2/d2-f1.txt
@@ -253,6 +255,7 @@ output/dist/temp/safeToDelete.tmp
 
 **/*.bmp
 **/*.dia
+**/*.vpd
 
 **/*.md
 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



(tomcat) branch 11.0.x updated: Update excludes list

2024-08-16 Thread markt
This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 11.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/11.0.x by this push:
 new a401439abc Update excludes list
a401439abc is described below

commit a401439abc695ea8ee6e3cbb41431a4e60619753
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:10:05 2024 +0100

Update excludes list
---
 res/rat/rat-excludes.txt | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index c185941722..b6d7c64e7d 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -46,7 +46,7 @@
 
   - files used simply to ensure directories are not empty
 
-  - *.bmp, *.dia files are binary
+  - *.bmp, *.dia and *.vpd files are binary
 (*.gif, *.jpg are also binary, but are automatically detected by RAT as
 ones, so no explicit configuration is needed)
 
@@ -99,6 +99,7 @@ output/dist/src/test/webapp/bug53257/*.txt
 output/dist/src/test/webapp/bug53257/foo bar/foobar.txt
 output/dist/src/test/webapp/bug66609/a&a.txt
 output/dist/src/test/webapp/bug66609/b'b.txt
+output/dist/src/test/webapp/jsp/ok.html
 output/dist/src/test/webapp-fragments/WEB-INF/classes/*.txt
 output/dist/src/test/webresources/dir1/d1/d1-f1.txt
 output/dist/src/test/webresources/dir1/d2/d2-f1.txt
@@ -117,6 +118,7 @@ test/webapp/bug53257/*.txt
 test/webapp/bug53257/foo bar/foobar.txt
 test/webapp/bug66609/a&a.txt
 test/webapp/bug66609/b'b.txt
+test/webapp/jsp/ok.html
 test/webapp-fragments/WEB-INF/classes/*.txt
 test/webresources/dir1/d1/d1-f1.txt
 test/webresources/dir1/d2/d2-f1.txt
@@ -253,6 +255,7 @@ output/dist/temp/safeToDelete.tmp
 
 **/*.bmp
 **/*.dia
+**/*.vpd
 
 **/*.md
 


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



(tomcat) branch main updated: Additional excludes for Jakarta EE 11 resources

2024-08-16 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 cdb1d507af Additional excludes for Jakarta EE 11 resources
cdb1d507af is described below

commit cdb1d507af991bcb286cc9ae63646267394dd0ad
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:21:07 2024 +0100

Additional excludes for Jakarta EE 11 resources
---
 res/rat/rat-excludes.txt | 12 
 1 file changed, 12 insertions(+)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index 6d6957b924..86afc41f78 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -181,6 +181,7 @@ output/classes/jakarta/servlet/resources/javaee_7.xsd
 output/classes/jakarta/servlet/resources/javaee_8.xsd
 output/classes/jakarta/servlet/resources/jakartaee_9.xsd
 output/classes/jakarta/servlet/resources/jakartaee_10.xsd
+output/classes/jakarta/servlet/resources/jakartaee_11.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_2.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_3.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_4.xsd
@@ -194,29 +195,35 @@ output/classes/jakarta/servlet/resources/jsp_2_2.xsd
 output/classes/jakarta/servlet/resources/jsp_2_3.xsd
 output/classes/jakarta/servlet/resources/jsp_3_0.xsd
 output/classes/jakarta/servlet/resources/jsp_3_1.xsd
+output/classes/jakarta/servlet/resources/jsp_4_0.xsd
 output/classes/jakarta/servlet/resources/web-app_3_0.xsd
 output/classes/jakarta/servlet/resources/web-app_3_1.xsd
 output/classes/jakarta/servlet/resources/web-app_4_0.xsd
 output/classes/jakarta/servlet/resources/web-app_5_0.xsd
 output/classes/jakarta/servlet/resources/web-app_6_0.xsd
+output/classes/jakarta/servlet/resources/web-app_6_1.xsd
 output/classes/jakarta/servlet/resources/web-common_3_0.xsd
 output/classes/jakarta/servlet/resources/web-common_3_1.xsd
 output/classes/jakarta/servlet/resources/web-common_4_0.xsd
 output/classes/jakarta/servlet/resources/web-common_5_0.xsd
 output/classes/jakarta/servlet/resources/web-common_6_0.xsd
+output/classes/jakarta/servlet/resources/web-common_6_1.xsd
 output/classes/jakarta/servlet/resources/web-fragment_3_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_3_1.xsd
 output/classes/jakarta/servlet/resources/web-fragment_4_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_5_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_6_0.xsd
+output/classes/jakarta/servlet/resources/web-fragment_6_1.xsd
 output/classes/jakarta/servlet/resources/web-jsptaglibrary_3_0.xsd
 output/classes/jakarta/servlet/resources/web-jsptaglibrary_3_1.xsd
+output/classes/jakarta/servlet/resources/web-jsptaglibrary_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_5.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_6.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_7.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_8.xsd
 output/dist/src/java/jakarta/servlet/resources/jakartaee_9.xsd
 output/dist/src/java/jakarta/servlet/resources/jakartaee_10.xsd
+output/dist/src/java/jakarta/servlet/resources/jakartaee_11.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_2.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_3.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_4.xsd
@@ -229,23 +236,28 @@ output/dist/src/java/jakarta/servlet/resources/jsp_2_2.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_2_3.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_3_1.xsd
+output/dist/src/java/jakarta/servlet/resources/jsp_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_6_0.xsd
+output/dist/src/java/jakarta/servlet/resources/web-app_6_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_6_0.xsd
+output/dist/src/java/jakarta/servlet/resources/web-common_6_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_6_0.xsd
+output/dist/src/java/jakarta/se

(tomcat) branch 11.0.x updated: Additional excludes for Jakarta EE 11 resources

2024-08-16 Thread markt
This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 11.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/11.0.x by this push:
 new d31cd4cfdd Additional excludes for Jakarta EE 11 resources
d31cd4cfdd is described below

commit d31cd4cfdd9f4cf188c0b49c1f2d5f72e49b12e2
Author: Mark Thomas 
AuthorDate: Fri Aug 16 11:21:07 2024 +0100

Additional excludes for Jakarta EE 11 resources
---
 res/rat/rat-excludes.txt | 12 
 1 file changed, 12 insertions(+)

diff --git a/res/rat/rat-excludes.txt b/res/rat/rat-excludes.txt
index b6d7c64e7d..5dd744ec9b 100644
--- a/res/rat/rat-excludes.txt
+++ b/res/rat/rat-excludes.txt
@@ -181,6 +181,7 @@ output/classes/jakarta/servlet/resources/javaee_7.xsd
 output/classes/jakarta/servlet/resources/javaee_8.xsd
 output/classes/jakarta/servlet/resources/jakartaee_9.xsd
 output/classes/jakarta/servlet/resources/jakartaee_10.xsd
+output/classes/jakarta/servlet/resources/jakartaee_11.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_2.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_3.xsd
 output/classes/jakarta/servlet/resources/javaee_web_services_1_4.xsd
@@ -194,29 +195,35 @@ output/classes/jakarta/servlet/resources/jsp_2_2.xsd
 output/classes/jakarta/servlet/resources/jsp_2_3.xsd
 output/classes/jakarta/servlet/resources/jsp_3_0.xsd
 output/classes/jakarta/servlet/resources/jsp_3_1.xsd
+output/classes/jakarta/servlet/resources/jsp_4_0.xsd
 output/classes/jakarta/servlet/resources/web-app_3_0.xsd
 output/classes/jakarta/servlet/resources/web-app_3_1.xsd
 output/classes/jakarta/servlet/resources/web-app_4_0.xsd
 output/classes/jakarta/servlet/resources/web-app_5_0.xsd
 output/classes/jakarta/servlet/resources/web-app_6_0.xsd
+output/classes/jakarta/servlet/resources/web-app_6_1.xsd
 output/classes/jakarta/servlet/resources/web-common_3_0.xsd
 output/classes/jakarta/servlet/resources/web-common_3_1.xsd
 output/classes/jakarta/servlet/resources/web-common_4_0.xsd
 output/classes/jakarta/servlet/resources/web-common_5_0.xsd
 output/classes/jakarta/servlet/resources/web-common_6_0.xsd
+output/classes/jakarta/servlet/resources/web-common_6_1.xsd
 output/classes/jakarta/servlet/resources/web-fragment_3_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_3_1.xsd
 output/classes/jakarta/servlet/resources/web-fragment_4_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_5_0.xsd
 output/classes/jakarta/servlet/resources/web-fragment_6_0.xsd
+output/classes/jakarta/servlet/resources/web-fragment_6_1.xsd
 output/classes/jakarta/servlet/resources/web-jsptaglibrary_3_0.xsd
 output/classes/jakarta/servlet/resources/web-jsptaglibrary_3_1.xsd
+output/classes/jakarta/servlet/resources/web-jsptaglibrary_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_5.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_6.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_7.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_8.xsd
 output/dist/src/java/jakarta/servlet/resources/jakartaee_9.xsd
 output/dist/src/java/jakarta/servlet/resources/jakartaee_10.xsd
+output/dist/src/java/jakarta/servlet/resources/jakartaee_11.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_2.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_3.xsd
 output/dist/src/java/jakarta/servlet/resources/javaee_web_services_1_4.xsd
@@ -229,23 +236,28 @@ output/dist/src/java/jakarta/servlet/resources/jsp_2_2.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_2_3.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/jsp_3_1.xsd
+output/dist/src/java/jakarta/servlet/resources/jsp_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-app_6_0.xsd
+output/dist/src/java/jakarta/servlet/resources/web-app_6_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-common_6_0.xsd
+output/dist/src/java/jakarta/servlet/resources/web-common_6_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_3_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_3_1.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_4_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_5_0.xsd
 output/dist/src/java/jakarta/servlet/resources/web-fragment_6_0.xsd
+output/dist/src/java/jakart

Re: Cookie parsing and upcoming updates to RFC6265

2024-08-16 Thread Tim Funk
How about  missingEqualsCookie="allow | ignore"?

By using [allow | ignore] instead of yes/no, it opens the door to
additional behaviors. (such as reject which triggers a http error)

-Tim

On Fri, Aug 16, 2024 at 4:33 AM Mark Thomas  wrote:

> On 14/08/2024 19:12, Konstantin Kolinko wrote:
>
> 
>
> I'm going to start working on this "noEqualsCookie" option. Suggestions
> for a better name still welcome.
>
>


[PR] Fix proxying null value result set [tomcat]

2024-08-16 Thread via GitHub


htdebeer opened a new pull request, #745:
URL: https://github.com/apache/tomcat/pull/745

   When invoking a method on a proxied statement returns `null`, that 
statement's proxy should also return `null`.
   
   In particular, `Statement#getResultSet` should not return a `ResultSetProxy` 
when the proxied statement returns `null`. Otherwise, that `ResultSet` proxy 
has a `null` delegate and calling methods on the proxy that lead to calling 
methods on the delegate could result in a `SqlException` with message 
"ResultSet closed."
   
   We found this issue because our Spring + MyBatis application started 
throwing `SqlException`s when calling stored procedures after upgrading to 
Tomcat v. 10.1.28, whereas it didn't in version 10.1.26.
   
   Looking in the MyBatis code, it checks if a `Statement`'s `ResultSet` is 
`null` or not. If it isn't, and it isn't when the `ResultSet` is proxied with a 
`null` delegate`, it later calls method `ResultSet#getMetaData()`. And that now 
responds with `SqlException` with message "ResultSet closed.".
   
   I think the new behavior was introduced in 
https://github.com/apache/tomcat/pull/742.


-- 
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.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 69268] New: Oklahoma's Trusted Roofing Partner

2024-08-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69268

Bug ID: 69268
   Summary: Oklahoma's Trusted Roofing Partner
   Product: Tomcat Native
   Version: unspecified
  Hardware: PC
Status: NEW
  Severity: normal
  Priority: P2
 Component: Documentation
  Assignee: dev@tomcat.apache.org
  Reporter: goalcreat...@gmail.com
  Target Milestone: ---

Proudly serving Oklahoma, Loomis Construction is more than just your average
roofing company. We’re committed to delivering top-notch craftsmanship,
exceptional customer service, and durable roofing solutions that stand the test
of time.

Whether it’s your home or business, we understand the unique challenges of
Oklahoma’s weather—from strong winds to hail storms. That’s why we go above and
beyond, providing comprehensive inspections and repairs that ensure your roof
remains in top condition year-round.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Bug 69268] Oklahoma's Trusted Roofing Partner

2024-08-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69268

loomis construction and roofing  changed:

   What|Removed |Added

 OS||All
URL||https://loomisroofpro.com/

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Cookie parsing and upcoming updates to RFC6265

2024-08-16 Thread Mark Thomas

On 16/08/2024 13:40, Tim Funk wrote:

How about  missingEqualsCookie="allow | ignore"?


The proposed options were:
- ignore
- name
- value


By using [allow | ignore] instead of yes/no, it opens the door to
additional behaviors. (such as reject which triggers a http error)


Agreed.

Mark

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org