[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 --- Comment #5 from Michael Osipov --- How to proceed here since the interface needs a change, should be go with default methods? -- You are receiving this mail because: You are the assignee for the bug. --

[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 --- Comment #4 from Michael Osipov --- (In reply to Christopher Schultz from comment #1) > (In reply to Michael Osipov from comment #0) > > if (!readOnly || !resource.isReadOnly()) { > > Should this be an && instead of || ? > > I would think

[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 --- Comment #3 from Remy Maucherat --- (In reply to Mark Thomas from comment #2) > Woot! I get to mention De Morgan's laws! > > We can use either of > > !(A || B) > !A && !B > > I think the second for is easier to understand in this case but

Re: Buildbot failure in on tomcat-12.0.x

2024-10-02 Thread Rémy Maucherat
On Wed, Oct 2, 2024 at 5:38 PM wrote: > > Build status: BUILD FAILED: failed compile (failure) > Worker used: bb_worker2_ubuntu > URL: https://ci2.apache.org/#builders/120/builds/102 > Blamelist: Mark Thomas > Build Text: failed compile (failure) > Status Detected: new failure > Build Source Stam

Buildbot success in on tomcat-12.0.x

2024-10-02 Thread buildbot
Build status: Build succeeded! Worker used: bb_worker2_ubuntu URL: https://ci2.apache.org/#builders/120/builds/103 Blamelist: Mark Thomas , remm Build Text: build successful Status Detected: restored build Build Source Stamp: [branch main] ada067f939896c05be372b1712e40208ecdccadc Steps: worke

[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 --- Comment #2 from Mark Thomas --- Woot! I get to mention De Morgan's laws! We can use either of !(A || B) !A && !B I think the second for is easier to understand in this case but I think an argument could be made for either. -- You are r

(tomcat-training) 02/02: Clarify this plan was for Jakatrta EE 11

2024-10-02 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-training.git commit aae58eb8616c60dc43d2a8f0a95bed56b356c82a Author: Mark Thomas AuthorDate: Wed Oct 2 16:41:23 2024 +0100 Clarif

(tomcat-training) 01/02: Add code example for HttpSession.getAccessor()

2024-10-02 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-training.git commit 1ad620eb298db6fc62d93fe3ec698d38cae20b8a Author: Mark Thomas AuthorDate: Wed Oct 2 16:40:18 2024 +0100 Add co

(tomcat-training) branch main updated (09558ba -> aae58eb)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat-training.git from 09558ba Updates after a couple of practice runs new 1ad620e Add code example for HttpSession.getAccessor(

Buildbot failure in on tomcat-12.0.x

2024-10-02 Thread buildbot
Build status: BUILD FAILED: failed compile (failure) Worker used: bb_worker2_ubuntu URL: https://ci2.apache.org/#builders/120/builds/102 Blamelist: Mark Thomas Build Text: failed compile (failure) Status Detected: new failure Build Source Stamp: [branch main] b446e2b121151b3889fcbf0d8c16a92282b117

[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 --- Comment #1 from Christopher Schultz --- (In reply to Michael Osipov from comment #0) > if (!readOnly || !resource.isReadOnly()) { Should this be an && instead of || ? I would think that DELETE is only allowed if both all-of-webdav is writ

[Bug 69365] Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 Michael Osipov changed: What|Removed |Added CC||ch...@christopherschultz.ne

[Bug 69365] New: Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69365 Bug ID: 69365 Summary: Expose option for WebResource to determine it is read-only/read-write for DefaultServlet/WebdavServlet Product: Tomcat 9 Version: 9.0.95 Hardware:

(tomcat) branch 9.0.x updated: Add SSL session tracking test

2024-10-02 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 bff8ca00a0 Add SSL session tracking test bff8ca00a0

(tomcat) branch 10.1.x updated: Add SSL session tracking test

2024-10-02 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 d5755ed94c Add SSL session tracking test d5755ed94

(tomcat) branch 11.0.x updated: Add SSL session tracking test

2024-10-02 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm 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 aa8cd8f284 Add SSL session tracking test aa8cd8f28

(tomcat) branch main updated (b446e2b121 -> ada067f939)

2024-10-02 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from b446e2b121 Update Chinese translations (Ch_jp) add ada067f939 Add SSL session tracking test No new revisions were ad

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Michael Osipov
On 2024/10/02 14:30:52 Christopher Schultz wrote: > All, > > On 10/2/24 06:21, Mark Thomas wrote: > > -1 veto > > > > This change is not compliant with RFC 9110. > > > > [...] > > > > Generally, SC_METHOD_NOT_ALLOWED is probably the right status code as > > the most likely failure mode is a l

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Michael Osipov
On 2024/10/02 14:33:28 Christopher Schultz wrote: > Michael, > > On 10/2/24 07:00, Michael Osipov wrote: > > [...] > > > >> $ curl --negotiate -u : -X DELETE > >> 'https://example.com/backend-dev/dav/log/sadfs' --verbose > >> ...* Request completely sent off > >> < HTTP/1.1 405 > > Method not

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #13 from Michael Osipov --- (In reply to Michael Osipov from comment #12) > (In reply to Christopher Schultz from comment #11) > > (In reply to Mark Thomas from comment #6) > > > Final paragraph of 9.6.1. > > > > Also, the complete

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #12 from Michael Osipov --- (In reply to Christopher Schultz from comment #11) > (In reply to Mark Thomas from comment #6) > > Final paragraph of 9.6.1. > > Also, the complete example in 9.6.2 includes this specific case, and only

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #11 from Christopher Schultz --- (In reply to Mark Thomas from comment #6) > Final paragraph of 9.6.1. Also, the complete example in 9.6.2 includes this specific case, and only includes the file and not its parent directory in the

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Christopher Schultz
Michael, On 10/2/24 07:00, Michael Osipov wrote: > [...] > $ curl --negotiate -u : -X DELETE 'https://example.com/backend-dev/dav/log/sadfs' --verbose ...* Request completely sent off < HTTP/1.1 405 Method not allowed: DELETE < Date: Wed, 02 Oct 2024 10:59:10 GMT < Server: Apache < X-Frame

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Christopher Schultz
All, On 10/2/24 06:21, Mark Thomas wrote: -1 veto This change is not compliant with RFC 9110. > > [...] Generally, SC_METHOD_NOT_ALLOWED is probably the right status code as the most likely failure mode is a lack of permissions. I just wonder if there are circumstances where we would want

(tomcat) branch 9.0.x updated: Improvements to Chinese translations by Ch_jp

2024-10-02 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 0de3537921 Improvements to Chinese translations by

(tomcat) branch 9.0.x updated: Remove unused translations

2024-10-02 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 246abd207c Remove unused translations 246abd207c is

(tomcat) branch 9.0.x updated: Improvements to Japanese translations by tak7iji

2024-10-02 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 eb98277a06 Improvements to Japanese translations by

(tomcat) branch 9.0.x updated: Improvements to French translations. (remm)

2024-10-02 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 bd42dfac62 Improvements to French translations. (re

(tomcat) branch 10.1.x updated: Improvements to Chinese translations by Ch_jp

2024-10-02 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 bfd4f44abe Improvements to Chinese translations b

(tomcat) branch 10.1.x updated: Remove unused translations

2024-10-02 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 4a448ee606 Remove unused translations 4a448ee606

(tomcat) branch 10.1.x updated: Improvements to Japanese translations by tak7iji

2024-10-02 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 c759a7cca7 Improvements to Japanese translations

(tomcat) branch 10.1.x updated: Improvements to French translations. (remm)

2024-10-02 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 206518f9a1 Improvements to French translations. (

(tomcat) branch 11.0.x updated: Improvements to Chinese translations by Ch_jp

2024-10-02 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 7b01ab71c4 Improvements to Chinese translations b

(tomcat) branch 11.0.x updated: Removed unused translation

2024-10-02 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 7354ed05d8 Removed unused translation 7354ed05d8

(tomcat) branch 11.0.x updated: Improvements to Japanese translations by tak7iji

2024-10-02 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 595753a482 Improvements to Japanese translations

(tomcat) branch 11.0.x updated: Update French translations (remm)

2024-10-02 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 ab64878f20 Update French translations (remm) ab64

(tomcat) branch 11.0.x updated: Remove. Only need Export in main.

2024-10-02 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 73924b9937 Remove. Only need Export in main. 7392

(tomcat) 03/03: Update Chinese translations (Ch_jp)

2024-10-02 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 commit b446e2b121151b3889fcbf0d8c16a92282b117ce Author: Mark Thomas AuthorDate: Wed Oct 2 14:58:30 2024 +0100 Update Chinese

(tomcat) 01/03: Update Japanese translations (tak7iji)

2024-10-02 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 commit 07e85517c675d4129689bc170001efa8fde3ed5a Author: Mark Thomas AuthorDate: Wed Oct 2 14:57:17 2024 +0100 Update Japanese

(tomcat) 02/03: Remove unused translation

2024-10-02 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 commit 27fc32d89904232ab9cf03fecd9648d63312000c Author: Mark Thomas AuthorDate: Wed Oct 2 14:57:43 2024 +0100 Remove unused t

(tomcat) branch main updated (859bdd6a07 -> b446e2b121)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from 859bdd6a07 Updates to French translations (remm) new 07e85517c6 Update Japanese translations (tak7iji) new 27fc

(tomcat) branch main updated: Updates to French translations (remm)

2024-10-02 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 859bdd6a07 Updates to French translations (remm) 859b

(tomcat) branch main updated (b03fde0383 -> 5b0b742e1a)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from b03fde0383 Make WebdavServlet properly return the Allow header on NOT_ALLOWED status add 5b0b742e1a Add comment not

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #10 from Michael Osipov --- (In reply to Mark Thomas from comment #9) > My reading of RFC 4918 is if we have a status code for baz it should be 424. > > Quoting from RFC 2119: >SHOULD NOT This phrase, or the phrase "NOT RECOM

[Bug 69364] SPAM SPAM SPAM SPAM

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69364 Chuck Caldarale changed: What|Removed |Added Summary|Godrej Neopolis Apartments |SPAM SPAM SPAM SPAM

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #9 from Mark Thomas --- My reading of RFC 4918 is if we have a status code for baz it should be 424. Quoting from RFC 2119: SHOULD NOT This phrase, or the phrase "NOT RECOMMENDED" mean that there may exist valid reasons in

Re: (tomcat) branch main updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread Michael Osipov
On 2024/10/02 11:58:12 Rémy Maucherat wrote: > On Wed, Oct 2, 2024 at 1:51 PM wrote: > > > > This is an automated email from the ASF dual-hosted git repository. > > > > michaelo pushed a commit to branch main > > in repository https://gitbox.apache.org/repos/asf/tomcat.git > > > > > > The followin

[Bug 69364] New: Godrej Neopolis Apartments

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69364 Bug ID: 69364 Summary: Godrej Neopolis Apartments Product: Tomcat 10 Version: 10.1.29 Hardware: PC Status: NEW Severity: normal Priority: P2 Componen

Re: (tomcat) branch main updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread Rémy Maucherat
On Wed, Oct 2, 2024 at 1:51 PM wrote: > > This is an automated email from the ASF dual-hosted git repository. > > michaelo 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: >

[Bug 69360] Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69360 --- Comment #8 from Michael Osipov --- Incorporated Mark's objection. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-

(tomcat) branch 9.0.x updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 76fb7ec803 Make WebdavServlet properly return th

(tomcat) branch 10.1.x updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 0207a660c7 Make WebdavServlet properly return

(tomcat) branch 11.0.x updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 33d26b4036 Make WebdavServlet properly return

(tomcat) branch main updated: Make WebdavServlet properly return the Allow header on NOT_ALLOWED status

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 b03fde0383 Make WebdavServlet properly return the

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #8 from Michael Osipov --- My counter-proposal for code clarity: > diff --git a/java/org/apache/catalina/servlets/WebdavServlet.java > b/java/org/apache/catalina/servlets/WebdavServlet.java > index e8cb718295..22df7343af 100644 > -

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #7 from Michael Osipov --- (In reply to Mark Thomas from comment #6) > Final paragraph of 9.6.1. I see: 424 (Failed Dependency) status codes SHOULD NOT be in the 207 (Multi- Status) response for DELETE. They can be safely le

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Mark Thomas
On 02/10/2024 12:00, Michael Osipov wrote: On 2024/10/02 10:41:38 Mark Thomas wrote: On 02/10/2024 11:30, Michael Osipov wrote: On 2024/10/02 10:21:07 Mark Thomas wrote: -1 veto This change is not compliant with RFC 9110. If Tomcat sends METHOD_NOT_ALLOWED it MUST also send an Allow header.

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #6 from Mark Thomas --- Final paragraph of 9.6.1. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Michael Osipov
On 2024/10/02 10:41:38 Mark Thomas wrote: > On 02/10/2024 11:30, Michael Osipov wrote: > > On 2024/10/02 10:21:07 Mark Thomas wrote: > >> -1 veto > >> > >> This change is not compliant with RFC 9110. > >> > >> If Tomcat sends METHOD_NOT_ALLOWED it MUST also send an Allow header. > >> > > > > >>>

(tomcat) branch 9.0.x updated (ab0cd03763 -> 2de1db532d)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git from ab0cd03763 Improve debug log in DefaultServlet/WebdavServlet new 3ae984c224 Updates prior to applying automated cod

(tomcat) 01/04: Updates prior to applying automated code formatting

2024-10-02 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 commit 3ae984c224e38ad8eae28ba6e3ffd53da95c2a35 Author: Mark Thomas AuthorDate: Wed Oct 2 10:57:22 2024 +0100 Updates prior

(tomcat) 02/04: Code clean-up - formatting. No functional change.

2024-10-02 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 commit 1b7f7cf931ae551ce096cba4150111b123096d40 Author: Mark Thomas AuthorDate: Wed Oct 2 11:00:34 2024 +0100 Code clean-up

(tomcat) 03/04: Simplest way to retain formatting

2024-10-02 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 commit 85cd50ceb16a4d6c7d9cf6105604d477c480f68f Author: Mark Thomas AuthorDate: Wed Oct 2 11:46:51 2024 +0100 Simplest way t

(tomcat) 01/04: Updates prior to applying automated code formatting

2024-10-02 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 commit 565b21dba656a519f99db3f58cc50b130fa49e3f Author: Mark Thomas AuthorDate: Wed Oct 2 10:57:22 2024 +0100 Updates prior

(tomcat) 03/04: Simplest way to retain formatting

2024-10-02 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 commit 754d07ebdbeeabebab800b5b469227f9e5e013e4 Author: Mark Thomas AuthorDate: Wed Oct 2 11:46:51 2024 +0100 Simplest way

(tomcat) 02/04: Code clean-up - formatting. No functional change.

2024-10-02 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 commit 9faa87378cb08943a56c255571823c1a3424196b Author: Mark Thomas AuthorDate: Wed Oct 2 11:00:34 2024 +0100 Code clean-up

(tomcat) branch 10.1.x updated (02dd078f1c -> 2b7c1cf8da)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git from 02dd078f1c Improve debug log in DefaultServlet/WebdavServlet new 565b21dba6 Updates prior to applying automated co

(tomcat) branch 11.0.x updated (1ecf6870a5 -> 352efca575)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch 11.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git from 1ecf6870a5 Improve debug log in DefaultServlet/WebdavServlet new ee15e7567e Updates prior to applying automated co

(tomcat) 03/04: Simplest way to retain formatting

2024-10-02 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 commit 492dd4b7fc95fce0e91a043d41049960cf2c6369 Author: Mark Thomas AuthorDate: Wed Oct 2 11:46:51 2024 +0100 Simplest way

(tomcat) 02/04: Code clean-up - formatting. No functional change.

2024-10-02 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 commit 51b9cd00e692f74767d7b98be1978a79b3545f6b Author: Mark Thomas AuthorDate: Wed Oct 2 11:00:34 2024 +0100 Code clean-up

(tomcat) 04/04: Re-apply code formatting - no functional change

2024-10-02 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 commit 352efca5757ede5cdc28975cdc5be640cf6d3735 Author: Mark Thomas AuthorDate: Wed Oct 2 11:54:13 2024 +0100 Re-apply code

(tomcat) 01/04: Updates prior to applying automated code formatting

2024-10-02 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 commit ee15e7567e15d82d04d220ce643d7a955f0e9534 Author: Mark Thomas AuthorDate: Wed Oct 2 10:57:22 2024 +0100 Updates prior

(tomcat) branch main updated (eed64ff580 -> d605c66e90)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from eed64ff580 Code clean-up - formatting. No functional change. new e0d8de393c Simplest way to retain formatting n

(tomcat) 02/02: Re-apply code formatting - no functional change

2024-10-02 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 commit d605c66e907b2e2c1a30669af40f0ed1bf725af5 Author: Mark Thomas AuthorDate: Wed Oct 2 11:47:53 2024 +0100 Re-apply code f

(tomcat) 01/02: Simplest way to retain formatting

2024-10-02 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 commit e0d8de393c1aace3f8c0de353030a61862bfaf5e Author: Mark Thomas AuthorDate: Wed Oct 2 11:46:51 2024 +0100 Simplest way to

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #5 from Michael Osipov --- (In reply to Mark Thomas from comment #4) > See RFC 4918 section 9.6.1. I think this should remain as is. Are you referring to: If any resource identified by a member URL cannot be deleted, then all

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Mark Thomas
On 02/10/2024 11:30, Michael Osipov wrote: On 2024/10/02 10:21:07 Mark Thomas wrote: -1 veto This change is not compliant with RFC 9110. If Tomcat sends METHOD_NOT_ALLOWED it MUST also send an Allow header. diff --git a/java/org/apache/catalina/servlets/WebdavServlet.java b/java/org/apa

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Michael Osipov
On 2024/10/02 10:21:07 Mark Thomas wrote: > -1 veto > > This change is not compliant with RFC 9110. > > If Tomcat sends METHOD_NOT_ALLOWED it MUST also send an Allow header. > > That is why the default servlet has a dedicated method for sending > METHOD_NOT_ALLOWED. > > Further comments in-lin

(tomcat) 02/02: Code clean-up - formatting. No functional change.

2024-10-02 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 commit eed64ff5806705ae5779e225d8242be35a1812c5 Author: Mark Thomas AuthorDate: Wed Oct 2 11:00:34 2024 +0100 Code clean-up -

(tomcat) 01/02: Updates prior to applying automated code formatting

2024-10-02 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 commit 79234b2bb8cd9e1730b9ffac5183b69bcf7ce4ba Author: Mark Thomas AuthorDate: Wed Oct 2 10:57:22 2024 +0100 Updates prior t

(tomcat) branch main updated (6dfefeefbe -> eed64ff580)

2024-10-02 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from 6dfefeefbe Improve debug log in DefaultServlet/WebdavServlet new 79234b2bb8 Updates prior to applying automated code

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #4 from Mark Thomas --- See RFC 4918 section 9.6.1. I think this should remain as is. -- You are receiving this mail because: You are the assignee for the bug. -

Re: (tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread Mark Thomas
-1 veto This change is not compliant with RFC 9110. If Tomcat sends METHOD_NOT_ALLOWED it MUST also send an Allow header. That is why the default servlet has a dedicated method for sending METHOD_NOT_ALLOWED. Further comments in-line. Mark On 02/10/2024 10:07, micha...@apache.org wrote: T

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #3 from Remy Maucherat --- (In reply to Michael Osipov from comment #2) > (In reply to Remy Maucherat from comment #1) > > It's always being like that. The reason might have been to avoid being too > > redundant (if not deleting the

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #2 from Michael Osipov --- (In reply to Remy Maucherat from comment #1) > It's always being like that. The reason might have been to avoid being too > redundant (if not deleting the contents, the intermediate parent folder is > also

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 --- Comment #1 from Remy Maucherat --- It's always being like that. The reason might have been to avoid being too redundant (if not deleting the contents, the intermediate parent folder is also obviously not getting deleted) but it seems possib

(tomcat) branch 9.0.x updated: Improve debug log in DefaultServlet/WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 ab0cd03763 Improve debug log in DefaultServlet/W

(tomcat) branch 10.1.x updated: Improve debug log in DefaultServlet/WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 02dd078f1c Improve debug log in DefaultServlet

(tomcat) branch 11.0.x updated: Improve debug log in DefaultServlet/WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 1ecf6870a5 Improve debug log in DefaultServlet

(tomcat) branch main updated (142c45a427 -> 6dfefeefbe)

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git from 142c45a427 BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet add 6dfefeefbe Improv

[Bug 69363] WebdavServlet#parseProperties() incorrectly calculates href

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69363 Michael Osipov changed: What|Removed |Added CC||micha...@apache.org -- You are recei

[Bug 69363] New: WebdavServlet#parseProperties() incorrectly calculates href

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69363 Bug ID: 69363 Summary: WebdavServlet#parseProperties() incorrectly calculates href Product: Tomcat 9 Version: 9.0.95 Hardware: All OS: All Sta

[Bug 69362] Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 Michael Osipov changed: What|Removed |Added CC||micha...@apache.org -- You are recei

[Bug 69362] New: Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69362 Bug ID: 69362 Summary: Recursive nested collection DELETE not reflected in multi-status report from WebdavServlet Product: Tomcat 9 Version: 9.0.95 Hardware: All

[Bug 69359] WebdavServlet duplicates getRelativePath() method from super class with incorrect Javadoc

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69359 Michael Osipov changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug 69360] Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69360 Michael Osipov changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

(tomcat) branch 9.0.x updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 bb5bf0ac7a BZ 69360: Inconsistent DELETE behavio

(tomcat) branch 10.1.x updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 75b291f08e BZ 69360: Inconsistent DELETE behav

(tomcat) branch 11.0.x updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 d975c2b0f1 BZ 69360: Inconsistent DELETE behav

(tomcat) branch main updated: BZ 69360: Inconsistent DELETE behavior between DefaultServlet and WebdavServlet

2024-10-02 Thread michaelo
This is an automated email from the ASF dual-hosted git repository. michaelo 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 142c45a427 BZ 69360: Inconsistent DELETE behavior

  1   2   >