From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] ddns: Import upstream patch for NoIP.com
Date: Wed, 06 Nov 2019 15:36:15 +0000
Message-ID: <5CB0350E-E396-4035-87C5-107E49A41B02@ipfire.org>
In-Reply-To: <20191105182341.5129-1-stefan.schantl@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4665115486531973075=="
List-Id: <development.lists.ipfire.org>

--===============4665115486531973075==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi,

You have already merged this patch, so I commented in Bugzilla.

However, here is it again for the record:

You cannot change the handle because all existing configurations - which argu=
ably don=E2=80=99t work anyways now - won=E2=80=99t pick this provider to run=
 an update.

-Michael

> On 5 Nov 2019, at 18:23, Stefan Schantl <stefan.schantl(a)ipfire.org> wrote:
>=20
> Reference: #11561.
>=20
> Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> ---
> lfs/ddns                                      |  3 +
> .../ddns-012-noip-rename-provider.patch       | 65 +++++++++++++++++++
> 2 files changed, 68 insertions(+)
> create mode 100644 src/patches/ddns-012-noip-rename-provider.patch
>=20
> diff --git a/lfs/ddns b/lfs/ddns
> index a6ca2e35d..8bfa92f77 100644
> --- a/lfs/ddns
> +++ b/lfs/ddns
> @@ -71,6 +71,9 @@ $(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> 	@$(PREBUILD)
> 	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar axf $(DIR_DL)/$(DL_FILE)
>=20
> +	# Add upstream patch for fixing noip.com
> +	cd $(DIR_APP) && patch -Np1 < $(DIR_SRC)/src/patches/ddns-012-noip-rename=
-provider.patch
> +
> 	cd $(DIR_APP) && [ -x "configure" ] || sh ./autogen.sh
> 	cd $(DIR_APP) && ./configure \
> 		--prefix=3D/usr \
> diff --git a/src/patches/ddns-012-noip-rename-provider.patch b/src/patches/=
ddns-012-noip-rename-provider.patch
> new file mode 100644
> index 000000000..acc5e66a7
> --- /dev/null
> +++ b/src/patches/ddns-012-noip-rename-provider.patch
> @@ -0,0 +1,65 @@
> +commit bfdba55a5eeb3dac5ed35ff72b6c74b941da6445
> +Author: Stefan Schantl <stefan.schantl(a)ipfire.org>
> +Date:   Tue Nov 5 19:09:53 2019 +0100
> +
> +    No-IP.com: Rename provider and handle to NoIP.com
> +   =20
> +    Fixes #11561.
> +   =20
> +    Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
> +
> +diff --git a/README b/README
> +index 76f63bc..c75c448 100644
> +--- a/README
> ++++ b/README
> +@@ -75,7 +75,7 @@ SUPPORTED PROVIDERS:
> + 	loopia.se
> + 	myonlineportal.net
> + 	namecheap.com
> +-	no-ip.com
> ++	noip.com
> + 	now-dns.com
> + 	nsupdate.info
> + 	opendns.com
> +diff --git a/ddns.conf.sample b/ddns.conf.sample
> +index 5b3b845..528684f 100644
> +--- a/ddns.conf.sample
> ++++ b/ddns.conf.sample
> +@@ -119,8 +119,8 @@
> + # provider =3D namecheap.com
> + # password =3D pass
> +=20
> +-# [test.no-ip.org]
> +-# provider =3D no-ip.com
> ++# [test.noip.org]
> ++# provider =3D noip.com
> + # username =3D user
> + # password =3D pass
> +=20
> +diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> +index f3c62c1..4789107 100644
> +--- a/src/ddns/providers.py
> ++++ b/src/ddns/providers.py
> +@@ -1344,16 +1344,16 @@ class DDNSProviderNamecheap(DDNSResponseParserXML,=
 DDNSProvider):
> +=20
> +=20
> + class DDNSProviderNOIP(DDNSProtocolDynDNS2, DDNSProvider):
> +-	handle    =3D "no-ip.com"
> +-	name      =3D "No-IP"
> +-	website   =3D "http://www.no-ip.com/"
> ++	handle    =3D "noip.com"
> ++	name      =3D "NoIP"
> ++	website   =3D "http://www.noip.com/"
> + 	protocols =3D ("ipv4",)
> +=20
> + 	# Information about the format of the HTTP request is to be found
> +-	# here: http://www.no-ip.com/integrate/request and
> +-	# here: http://www.no-ip.com/integrate/response
> ++	# here: http://www.noip.com/integrate/request and
> ++	# here: http://www.noip.com/integrate/response
> +=20
> +-	url =3D "http://dynupdate.no-ip.com/nic/update"
> ++	url =3D "http://dynupdate.noip.com/nic/update"
> +=20
> + 	def prepare_request_data(self, proto):
> + 		assert proto =3D=3D "ipv4"
> --=20
> 2.20.1
>=20


--===============4665115486531973075==--