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
diff --git a/debian/changelog b/debian/changelog
index 6299cce..6a8ffcc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
1apache2 (2.4.57-2ubuntu2.3) mantic; urgency=medium
2
3 * d/c/m/setenvif.conf, d/p/fix-dolphin-to-delete-webdav-dirs.patch: Add
4 dolphin and Konqueror/5 careful redirection so that directories can be
5 deleted via webdav.
6 (LP: #1927742)
7
8 -- Bryce Harrington <bryce@canonical.com> Wed, 24 Jan 2024 22:51:25 -0800
9
1apache2 (2.4.57-2ubuntu2.2) mantic; urgency=medium10apache2 (2.4.57-2ubuntu2.2) mantic; urgency=medium
211
3 * d/icons/ubuntu-logo.png: add Ubuntu image for welcome page (LP: #1947459).12 * d/icons/ubuntu-logo.png: add Ubuntu image for welcome page (LP: #1947459).
diff --git a/debian/config-dir/mods-available/setenvif.conf b/debian/config-dir/mods-available/setenvif.conf
index 8bba04c..8b96bb9 100644
--- a/debian/config-dir/mods-available/setenvif.conf
+++ b/debian/config-dir/mods-available/setenvif.conf
@@ -24,3 +24,5 @@ BrowserMatch "^gvfs/1" redirect-carefully
24BrowserMatch "^XML Spy" redirect-carefully24BrowserMatch "^XML Spy" redirect-carefully
25BrowserMatch "^Dreamweaver-WebDAV-SCM1" redirect-carefully25BrowserMatch "^Dreamweaver-WebDAV-SCM1" redirect-carefully
26BrowserMatch " Konqueror/4" redirect-carefully26BrowserMatch " Konqueror/4" redirect-carefully
27BrowserMatch " Konqueror/5" redirect-carefully
28BrowserMatch " dolphin/" redirect-carefully
diff --git a/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch b/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch
27new file mode 10064429new file mode 100644
index 0000000..1fac833
--- /dev/null
+++ b/debian/patches/fix-dolphin-to-delete-webdav-dirs.patch
@@ -0,0 +1,16 @@
1Description: Recognize dolphin and Konqueror/5 browsers for webdav.
2 Without these settings, these browsers will be sent redirects on
3 URLs for directories specified without a trailing '/'. That can
4 prevent webdav DELETE operations from working.
5Author: Michal Maloszewski <michal.maloszewski@canonical.com>
6Bug: https://bz.apache.org/bugzilla/show_bug.cgi?id=67039
7Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/apache2/+bug/1927742
8Last-Update: 2024-01-16
9--- a/docs/conf/extra/httpd-dav.conf.in
10+++ b/docs/conf/extra/httpd-dav.conf.in
11@@ -48,3 +48,5 @@
12 BrowserMatch "^XML Spy" redirect-carefully
13 BrowserMatch "^Dreamweaver-WebDAV-SCM1" redirect-carefully
14 BrowserMatch " Konqueror/4" redirect-carefully
15+BrowserMatch " Konqueror/5" redirect-carefully
16+BrowserMatch " dolphin/" redirect-carefully
diff --git a/debian/patches/series b/debian/patches/series
index 041c4e7..ef52d02 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
1fix-dolphin-to-delete-webdav-dirs.patch
1fhs_compliance.patch2fhs_compliance.patch
2no_LD_LIBRARY_PATH.patch3no_LD_LIBRARY_PATH.patch
3suexec-CVE-2007-1742.patch4suexec-CVE-2007-1742.patch

Subscribers

People subscribed via source and target branches