From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: DDNS: add new Provider key-sytems.net (domaindiscount24)
Date: Thu, 28 Nov 2019 11:20:49 +0000
Message-ID: <4143661B-9526-49CE-AECD-7FD2E03D0DF7@ipfire.org>
In-Reply-To: <81dc1ed3-1179-42c6-784a-15f64a79508c@gmx.at>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8671625287273569491=="
List-Id: <development.lists.ipfire.org>

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

Hello Christof,

> On 28 Nov 2019, at 10:45, Christof Weniger <ChristofWeniger(a)gmx.at> wrote:
>=20
> Hi,
>=20
> I hope this is the correct way to submit this patch.

Yes, you found the right place.

It would have been better to post the patch inline (and not as an attachment)=
, so that I could have commented on it.

> https://www.domaindiscount24.com/ has its own ddns service runnning,
> which (for me) gets rid of the necessity of having to use an extra
> service for that.
>=20
> I tested the following patch on my system at home, and attached it to
> this mail.
>=20
> I started my quest at the community forum:
> https://community.ipfire.org/t/adding-new-ddns-provider/428/2
>=20
> Christof
>=20
>=20
>=20
>=20
>=20
> <add_key_systems_ddns.patch>

I will make an exception here now and still give you my thoughts :)

It looks like this is very close to the DynDNS protocol, so you can re-use th=
at as some other providers do.

Unfortunately this provider has no list of any possible responses - or maybe =
I have overlooked it. That can however be tested to check if those are still =
compatible with DynDNS.

Finally, you are setting the IP addresses to =E2=80=9Cauto=E2=80=9D which pro=
bably is not a good idea. I would prefer that it is explicitly being set. The=
re is a function for it that finds out which one is the correct IP address.

In the end I think this provider could look like =E2=80=9Cmyonlineportal=E2=
=80=9D: https://git.ipfire.org/?p=3Dddns.git;a=3Dblob;f=3Dsrc/ddns/providers.=
py;h=3D661fbcc57a5aecba0d958ec05c26296b3cef0d70;hb=3DHEAD#l1274

Best,
-Michael

--===============8671625287273569491==--