[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#892734: marked as done (stretch-pu: package dehydrated/0.3.1-3+deb9u2)

Your message dated Sat, 14 Jul 2018 11:21:20 +0100
with message-id <1531563680.2095.30.camel@adam-barratt.org.uk>
and subject line Closing bugs for updates included in 9.5
has caused the Debian Bug report #892734,
regarding stretch-pu: package dehydrated/0.3.1-3+deb9u2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
892734: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=892734
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Tags: stretch
User: release.debian.org@packages.debian.org
Usertags: pu


Let's Encrypt decided to add a redirect in their API, and it turns out
not all clients could cope with it seamlessly.

https://bugs.debian.org/892723

Attached, a tested patch fixing the bug.

Given that the updater is just about terribily broken by this bug, I'd
like to ask for a push to stable-updates.

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

diffstat for dehydrated-0.3.1 dehydrated-0.3.1

 changelog                                                         |    8 ++
 patches/Update-the-default-License-Subscriber-Agreement-URL.patch |    6 +-
 patches/follow-location-on-http-get-requests.patch                |   27 ++++++++++
 patches/series                                                    |    1 
 4 files changed, 39 insertions(+), 3 deletions(-)

diff -Nru dehydrated-0.3.1/debian/changelog dehydrated-0.3.1/debian/changelog
--- dehydrated-0.3.1/debian/changelog	2017-11-18 14:00:07.000000000 +0100
+++ dehydrated-0.3.1/debian/changelog	2018-03-12 11:48:10.000000000 +0100
@@ -1,3 +1,11 @@
+dehydrated (0.3.1-3+deb9u2) stretch; urgency=medium
+
+  * Add patch from upstream to follow redirects on HTTP GET.
+    This fixes an error when creating the fullchain.pem after the LE API
+    introduced a new redirect.  Closes: #892723
+
+ -- Mattia Rizzolo <mattia@debian.org>  Mon, 12 Mar 2018 11:48:10 +0100
+
 dehydrated (0.3.1-3+deb9u1) stretch; urgency=medium
 
   * Update the default License Subscriber Agreement URL.  Closes: #881974
diff -Nru dehydrated-0.3.1/debian/patches/follow-location-on-http-get-requests.patch dehydrated-0.3.1/debian/patches/follow-location-on-http-get-requests.patch
--- dehydrated-0.3.1/debian/patches/follow-location-on-http-get-requests.patch	1970-01-01 01:00:00.000000000 +0100
+++ dehydrated-0.3.1/debian/patches/follow-location-on-http-get-requests.patch	2018-03-12 11:45:54.000000000 +0100
@@ -0,0 +1,27 @@
+From: Lukas Schauer <lukas@schauer.so>
+Date: Sat, 3 Feb 2018 22:03:58 +0100
+Subject: follow location on http get-requests
+
+For more info:
+https://community.letsencrypt.org/t/dehydrated-caused-rate-limits-to-be-reached/52477
+
+Ogigin: upstream, https://github.com/lukas2511/dehydrated/commit/7a0e71c6c2ccc6e98abca5ea1c7de28053e90c02
+Bug-Debian: https://bugs.debian.org/892723
+Signed-off-by: Mattia Rizzolo <mattia@debian.org>
+---
+ dehydrated | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/dehydrated b/dehydrated
+index 882c6bd..a0dbf04 100755
+--- a/dehydrated
++++ b/dehydrated
+@@ -337,7 +337,7 @@ http_request() {
+     statuscode="$(curl ${ip_version:-} -s -w "%{http_code}" -o "${tempcont}" "${2}" -I)"
+     curlret="${?}"
+   elif [[ "${1}" = "get" ]]; then
+-    statuscode="$(curl ${ip_version:-} -s -w "%{http_code}" -o "${tempcont}" "${2}")"
++    statuscode="$(curl ${ip_version:-} -L -s -w "%{http_code}" -o "${tempcont}" "${2}")"
+     curlret="${?}"
+   elif [[ "${1}" = "post" ]]; then
+     statuscode="$(curl ${ip_version:-} -s -w "%{http_code}" -o "${tempcont}" "${2}" -d "${3}")"
diff -Nru dehydrated-0.3.1/debian/patches/series dehydrated-0.3.1/debian/patches/series
--- dehydrated-0.3.1/debian/patches/series	2017-11-18 14:00:07.000000000 +0100
+++ dehydrated-0.3.1/debian/patches/series	2018-03-12 11:45:54.000000000 +0100
@@ -5,3 +5,4 @@
 honor-config-if-the-user-provided-one-to-letsencrypt.sh-w.patch
 Support-both-config.sh-and-config-as-config-filenames-for.patch
 Update-the-default-License-Subscriber-Agreement-URL.patch
+follow-location-on-http-get-requests.patch
diff -Nru dehydrated-0.3.1/debian/patches/Update-the-default-License-Subscriber-Agreement-URL.patch dehydrated-0.3.1/debian/patches/Update-the-default-License-Subscriber-Agreement-URL.patch
--- dehydrated-0.3.1/debian/patches/Update-the-default-License-Subscriber-Agreement-URL.patch	2017-11-18 14:00:07.000000000 +0100
+++ dehydrated-0.3.1/debian/patches/Update-the-default-License-Subscriber-Agreement-URL.patch	2018-03-12 11:45:54.000000000 +0100
@@ -5,9 +5,9 @@
 Closes: #881974
 Signed-off-by: Mattia Rizzolo <mattia@debian.org>
 ---
- dehydrated            | 2 +-
- docs/examples/config  | 4 ++--
- 3 files changed, 4 insertions(+), 3 deletions(-)
+ dehydrated           | 2 +-
+ docs/examples/config | 4 ++--
+ 2 files changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/dehydrated b/dehydrated
 index 7b88ae9..882c6bd 100755

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message --- 
Version: 9.5

Hi,

The update referenced by each of these bugs was included in this
morning's stretch point release.

Regards,

Adam

--- End Message ---
Reply to: