Re: Buildbot failure in on tomcat-11.0.x

2024-06-12 Thread Igal Sapir
On Wed, Jun 12, 2024, 01:32 Mark Thomas wrote: > On 12/06/2024 01:46, Igal Sapir wrote: > > On Tue, Jun 11, 2024 at 5:32 PM wrote: > > > >> Build status: BUILD FAILED: failed Snapshot deployed to ASF Maven > snapshot > >> repository (failure) > >> Worker used: bb_worker2_ubuntu > >> URL:

[Bug 69131] allowCorsPreflight="filter" does not work with filter mappings other than "/*" for the CorsFilter

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69131 --- Comment #4 from Mark Thomas --- That seems reasonable. Maybe move the method to o.a.catalina.util.FilterUtil It isn't a perfect solution but it is an improvement on what we have now. -- You are receiving this mail because: You are the

Re: [PR] Support jar resources in bloom filter [tomcat]

2024-06-12 Thread via GitHub
markt-asf commented on PR #730: URL: https://github.com/apache/tomcat/pull/730#issuecomment-2163058083 Thanks for the PR. Applied manually with a few tweaks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Support jar resources in bloom filter [tomcat]

2024-06-12 Thread via GitHub
markt-asf closed pull request #730: Support jar resources in bloom filter URL: https://github.com/apache/tomcat/pull/730 -- 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

Buildbot failure in on tomcat-11.0.x

2024-06-12 Thread buildbot
Build status: BUILD FAILED: Snapshot deployed to ASF Maven snapshot repository (failure) Worker used: bb_worker2_ubuntu URL: https://ci2.apache.org/#builders/112/builds/1134 Blamelist: Mark Thomas Build Text: Snapshot deployed to ASF Maven snapshot repository (failure) Status Detected: new

(tomcat) branch 9.0.x updated: Fix static resources in a JAR file when using a bloom filter.

2024-06-12 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 84f13f968f Fix static resources in a JAR file when

(tomcat) branch 10.1.x updated: Fix static resources in a JAR file when using a bloom filter.

2024-06-12 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 a048642095 Fix static resources in a JAR file

(tomcat) branch main updated: Fix static resources in a JAR file when using a bloom filter.

2024-06-12 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 b3a6834ede Fix static resources in a JAR file when

Re: [PR] Configurable HTML subtitle in Manager web application [tomcat]

2024-06-12 Thread via GitHub
markt-asf commented on PR #724: URL: https://github.com/apache/tomcat/pull/724#issuecomment-2162824078 Thanks for the PR. I have applied a variation of this that uses a servlet init parameter rather than a context init parameter since a) the configuration is specific to the servlet and b)

Re: [PR] Configurable HTML subtitle in Manager web application [tomcat]

2024-06-12 Thread via GitHub
markt-asf closed pull request #724: Configurable HTML subtitle in Manager web application URL: https://github.com/apache/tomcat/pull/724 -- 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

(tomcat) branch 9.0.x updated: Add the ability to set a sub-title for the Manager main page

2024-06-12 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 0cf53edcdf Add the ability to set a sub-title for

(tomcat) branch 10.1.x updated: Add the ability to set a sub-title for the Manager main page

2024-06-12 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 992ef01394 Add the ability to set a sub-title

(tomcat) branch main updated: Add the ability to set a sub-title for the Manager main page

2024-06-12 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 c1573e4191 Add the ability to set a sub-title for

[Bug 69131] allowCorsPreflight="filter" does not work with filter mappings other than "/*" for the CorsFilter

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69131 --- Comment #3 from Robert Rodewald --- How about making the static method matchFiltersURL of ApplicationFilterFactory public and using it in the AuthenticatorBase? In general there should be a very small amount of mappings for this one

Buildbot success in on tomcat-11.0.x

2024-06-12 Thread buildbot
Build status: Build succeeded! Worker used: bb_worker2_ubuntu URL: https://ci2.apache.org/#builders/112/builds/1132 Blamelist: Igal Sapir , remm Build Text: build successful Status Detected: restored build Build Source Stamp: [branch main] 204f0fe75d1bdae7a071ac2a49ecd42fc033afe6 Steps:

(tomcat) branch 9.0.x updated: Make (Connection)Handler accessible through API from AbstractProtocol#getHandler to ultimately access RequestGroupInfo (getGlobal)

2024-06-12 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 7b341f6292 Make (Connection)Handler accessible

(tomcat) branch 10.1.x updated: Make (Connection)Handler accessible through API from AbstractProtocol#getHandler to ultimately access RequestGroupInfo (getGlobal)

2024-06-12 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 2373486bc0 Make (Connection)Handler accessible

[Bug 69133] Add property for max queue size to AbstractProtocol

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69133 --- Comment #2 from Moritz Halbritter --- That was fast. Thanks a lot! -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail:

(tomcat) branch main updated: Make (Connection)Handler accessible through API from AbstractProtocol#getHandler to ultimately access RequestGroupInfo (getGlobal)

2024-06-12 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 9c65807659 Make (Connection)Handler accessible

Re: [PR] Make ConnectionHandler accessible through API from AbstractProtocol to ultimately acess RequestGroupInfo [tomcat]

2024-06-12 Thread via GitHub
markt-asf merged PR #711: URL: https://github.com/apache/tomcat/pull/711 -- 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:

Re: [PR] Make ConnectionHandler accessible through API from AbstractProtocol to ultimately acess RequestGroupInfo [tomcat]

2024-06-12 Thread via GitHub
rmannibucau commented on code in PR #711: URL: https://github.com/apache/tomcat/pull/711#discussion_r1636187299 ## java/org/apache/coyote/AbstractProtocol.java: ## @@ -769,7 +769,7 @@ private void logPortOffset() { // ---

(tomcat) branch 9.0.x updated: Remove attribute duplication

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 f870a75635 Remove attribute duplication f870a75635

(tomcat) branch 10.1.x updated: Remove attribute duplication

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 e1f905b7f9 Remove attribute duplication

(tomcat) branch main updated: Remove attribute duplication

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 3e5fae1eac Remove attribute duplication 3e5fae1eac is

[Bug 69133] Add property for max queue size to AbstractProtocol

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69133 Remy Maucherat changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

(tomcat) branch 9.0.x updated: Add task queue size configuration on the Connector element

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 8feb72bab9 Add task queue size configuration on the

(tomcat) branch 10.1.x updated: Add task queue size configuration on the Connector element

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 4bddc82b64 Add task queue size configuration on

(tomcat) branch main updated: Add task queue size configuration on the Connector element

2024-06-12 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 204f0fe75d Add task queue size configuration on the

Re: [PR] Make ConnectionHandler accessible through API from AbstractProtocol to ultimately acess RequestGroupInfo [tomcat]

2024-06-12 Thread via GitHub
markt-asf commented on code in PR #711: URL: https://github.com/apache/tomcat/pull/711#discussion_r1636159961 ## java/org/apache/coyote/AbstractProtocol.java: ## @@ -769,7 +769,7 @@ private void logPortOffset() { // --- Connection

[Bug 69131] allowCorsPreflight="filter" does not work with filter mappings other than "/*" for the CorsFilter

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69131 --- Comment #2 from Mark Thomas --- Going back to that conversation, the reason we went with "/*" only was that regarding supporting anything else: "Not easily, no. You'd essentially have to recreate large chunks of ApplicationFilterFactory."

Re: Buildbot failure in on tomcat-11.0.x

2024-06-12 Thread Mark Thomas
On 12/06/2024 01:46, Igal Sapir wrote: On Tue, Jun 11, 2024 at 5:32 PM wrote: Build status: BUILD FAILED: failed Snapshot deployed to ASF Maven snapshot repository (failure) Worker used: bb_worker2_ubuntu URL: https://ci2.apache.org/#builders/112/builds/1130 Blamelist: Igal Sapir Build Text:

[Bug 69133] Add property for max queue size to AbstractProtocol

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69133 Andy Wilkinson changed: What|Removed |Added CC||andy.wilkin...@broadcom.com -- You

[Bug 69016] HttpURLConnection getOutputStream createCapacityException

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69016 Mark Thomas changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEEDINFO

[Bug 69133] New: Add property for max queue size to AbstractProtocol

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69133 Bug ID: 69133 Summary: Add property for max queue size to AbstractProtocol Product: Tomcat 10 Version: 10.1.24 Hardware: PC OS: Linux Status: NEW

[Bug 68932] Occasional 404 or 500 errors

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=68932 Mark Thomas changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEEDINFO

[Bug 69121] Async requests are being forwarded to the default error page when the connection with the server is lost

2024-06-12 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69121 Mark Thomas changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #1 from Mark