Re: [PATCH] ProviderDDNSS: Fix unhandled exception on update

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1236 bytes --]

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

Did you send this to any of the people affected by the problem to test it?

-Michael

> On 25 May 2020, at 17:05, Stefan Schantl <stefan.schantl(a)ipfire.org> wrote:
> 
> In python > 3.3 the getheader() attribute is not longer part of the
> urllib.response response.info() object. It is part of response object
> and so directly can be accessed.
> 
> Fixes #12328
> 
> Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> ---
> src/ddns/providers.py | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index f1fed22..46d8a67 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -642,10 +642,8 @@ class DDNSProviderDDNSS(DDNSProvider):
> 		response = self.send_request(self.url, data=data)
> 
> 		# This provider sends the response code as part of the header.
> -		header = response.info()
> -
> 		# Get status information from the header.
> -		output = header.getheader('ddnss-response')
> +		output = response.getheader('ddnss-response')
> 
> 		# Handle success messages.
> 		if output == "good" or output == "nochg":
> -- 
> 2.26.1
>