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

Proposed by Bryce Harrington
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merge reported by: Bryce Harrington
Merged at revision: aa3cb2b14e99f9e4c6f07c8b7fa722fde5e98d9a
Proposed branch: ~bryce/ubuntu/+source/apache2:sru-lp1927742-mantic
Merge into: ubuntu/+source/apache2:ubuntu/mantic-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
Andreas Hasenack Approve
git-ubuntu bot Approve
Canonical Server packageset reviewers Pending
Canonical Server Reporter Pending
Canonical Server Core Reviewers Pending
Review via email: mp+459607@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.57-2ubuntu2.3~mantic1
    + ✅ apache2 on mantic for amd64 @ 29.01.24 20:58:30 Log️ 🗒️
    + ✅ apache2 on mantic for armhf @ 29.01.24 21:18:49 Log️ 🗒️
    + ✅ apache2 on mantic for ppc64el @ 29.01.24 20:58:35 Log️ 🗒️
    + ✅ apache2 on mantic for s390x @ 29.01.24 20:59:25 Log️ 🗒️

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

+1 with the added DEP3 headers

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
Andreas Hasenack (ahasenack) :
review: Needs Fixing
Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks for catching that. Fix pushed.

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

+1

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

Successfully signed dsc, buildinfo, changes files
tee: /home/bryce/pkg/Apache2/sru-lp1927742/upload_reply.txt: No such file or directory
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 6299cce..6a8ffcc 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,12 @@
6+apache2 (2.4.57-2ubuntu2.3) mantic; 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 22:51:25 -0800
14+
15 apache2 (2.4.57-2ubuntu2.2) mantic; urgency=medium
16
17 * d/icons/ubuntu-logo.png: add Ubuntu image for welcome page (LP: #1947459).
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 041c4e7..ef52d02 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