Merge ~bryce/ubuntu/+source/apache2:sru-lp1927742-noble into ubuntu/+source/apache2:ubuntu/noble-devel

Proposed by Bryce Harrington
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merge reported by: Bryce Harrington
Merged at revision: 551e81cc3f955f6a36b780d1c1cfdbb6a14d3467
Proposed branch: ~bryce/ubuntu/+source/apache2:sru-lp1927742-noble
Merge into: ubuntu/+source/apache2:ubuntu/noble-devel
Diff against target: 58 lines (+28/-0)
4 files modified
debian/changelog (+9/-0)
debian/config-dir/mods-available/setenvif.conf (+2/-0)
debian/patches/fix-dolphin-to-delete-webdav-dirs.patch (+16/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Andreas Hasenack Approve
Canonical Server packageset reviewers Pending
Canonical Server Reporter Pending
Canonical Server Core Reviewers Pending
Review via email: mp+459606@code.launchpad.net

Description of the change

Adds a fix for a KDE issue with apache2's webdav. This carries the work by Michal, which while his approach was accurate I found through testing was missing the actual fix being installed. I've corrected that and verified the testing using both curl and dolphin as clients against each of the 4 series being targeted as webdav servers.

Bug: https://bugs.launchpad.net/ubuntu/+source/apache2/+bug/1927742
PPA: https://launchpad.net/~bryce/+archive/ubuntu/apache2-sru-lp1927742/

I've just kicked off a fresh set of DEP8 runs against the PPA, but as the DEP8 tests don't cover this case I don't expect them to be of much interest.

To post a comment you must log in.
Revision history for this message
Bryce Harrington (bryce) wrote :

  - apache2/2.4.58-1ubuntu2~noble1
    + ✅ apache2 on noble for amd64 @ 29.01.24 21:00:15 Log️ 🗒️
    + ✅ apache2 on noble for arm64 @ 29.01.24 21:24:00 Log️ 🗒️
    + ✅ apache2 on noble for armhf @ 29.01.24 20:43:57 Log️ 🗒️
    + ✅ apache2 on noble for ppc64el @ 29.01.24 20:59:55 Log️ 🗒️
    + ✅ apache2 on noble for s390x @ 29.01.24 21:09:26 Log️ 🗒️

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1

I took the liberty of "fixing" the curl test case in the bug:

a. curl -I -X DELETE -A 'prefix dolphin' http://davtest-<codename>.lxd/example/test

to

a. curl -I -X DELETE -A 'prefix dolphin/' http://davtest-<codename>.lxd/example/test

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: bryce, ahasenack
Uploaders: bryce, ahasenack
MP auto-approved

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks, uploaded for noble:

Vcs-Git: https://git.launchpad.net/~bryce/ubuntu/+source/apache2
Vcs-Git-Commit: 551e81cc3f955f6a36b780d1c1cfdbb6a14d3467
Vcs-Git-Ref: refs/heads/sru-lp1927742-noble
$ dput ubuntu ../apache2_2.4.58-1ubuntu2_source.changes
gpg: ../apache2_2.4.58-1ubuntu2_source.changes: Valid signature from E603B2578FB8F0FB
gpg: ../apache2_2.4.58-1ubuntu2.dsc: Valid signature from E603B2578FB8F0FB
D: Setting host argument.
Checking signature on .changes
Checking signature on .dsc
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading apache2_2.4.58-1ubuntu2.dsc: done.
  Uploading apache2_2.4.58-1ubuntu2.debian.tar.xz: done.
  Uploading apache2_2.4.58-1ubuntu2_source.buildinfo: done.
  Uploading apache2_2.4.58-1ubuntu2_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index fe2f3bc..69fae98 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,12 @@
6+apache2 (2.4.58-1ubuntu2) noble; urgency=medium
7+
8+ * d/c/m/setenvif.conf, d/p/fix-dolphin-to-delete-webdav-dirs.patch: Add
9+ dolphin and Konqueror/5 careful redirection so that directories can be
10+ deleted via webdav.
11+ (LP: #1927742)
12+
13+ -- Bryce Harrington <bryce@canonical.com> Wed, 24 Jan 2024 14:00:03 -0800
14+
15 apache2 (2.4.58-1ubuntu1) noble; urgency=medium
16
17 * Merge with Debian unstable (LP: #2040357). Remaining changes:
18diff --git a/debian/config-dir/mods-available/setenvif.conf b/debian/config-dir/mods-available/setenvif.conf
19index 8bba04c..8b96bb9 100644
20--- a/debian/config-dir/mods-available/setenvif.conf
21+++ b/debian/config-dir/mods-available/setenvif.conf
22@@ -24,3 +24,5 @@ BrowserMatch "^gvfs/1" redirect-carefully
23 BrowserMatch "^XML Spy" redirect-carefully
24 BrowserMatch "^Dreamweaver-WebDAV-SCM1" redirect-carefully
25 BrowserMatch " Konqueror/4" redirect-carefully
26+BrowserMatch " Konqueror/5" redirect-carefully
27+BrowserMatch " dolphin/" redirect-carefully
28diff --git a/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch b/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch
29new file mode 100644
30index 0000000..1fac833
31--- /dev/null
32+++ b/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch
33@@ -0,0 +1,16 @@
34+Description: Recognize dolphin and Konqueror/5 browsers for webdav.
35+ Without these settings, these browsers will be sent redirects on
36+ URLs for directories specified without a trailing '/'. That can
37+ prevent webdav DELETE operations from working.
38+Author: Michal Maloszewski <michal.maloszewski@canonical.com>
39+Bug: https://bz.apache.org/bugzilla/show_bug.cgi?id=67039
40+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/apache2/+bug/1927742
41+Last-Update: 2024-01-16
42+--- a/docs/conf/extra/httpd-dav.conf.in
43++++ b/docs/conf/extra/httpd-dav.conf.in
44+@@ -48,3 +48,5 @@
45+ BrowserMatch "^XML Spy" redirect-carefully
46+ BrowserMatch "^Dreamweaver-WebDAV-SCM1" redirect-carefully
47+ BrowserMatch " Konqueror/4" redirect-carefully
48++BrowserMatch " Konqueror/5" redirect-carefully
49++BrowserMatch " dolphin/" redirect-carefully
50diff --git a/debian/patches/series b/debian/patches/series
51index d2c00e2..ae14d4c 100644
52--- a/debian/patches/series
53+++ b/debian/patches/series
54@@ -1,3 +1,4 @@
55+fix-dolphin-to-delete-webdav-dirs.patch
56 fhs_compliance.patch
57 no_LD_LIBRARY_PATH.patch
58 suexec-CVE-2007-1742.patch

Subscribers

People subscribed via source and target branches