Re: [PATCH] ddns: Add provider Feste-IP.Net Signed-off-by: Martin Krieger <makrie@posteo.de> --- *IPv6 support included based on API description but not tested. *Test system is connected by VDSL with pure IPv4 (No IPv6 or dual stack). *Setup can handle IPv4, IPv6 or dual stack updates

Re: [PATCH] ddns: Add provider Feste-IP.Net Signed-off-by: Martin Krieger <makrie@posteo.de> --- *IPv6 support included based on API description but not tested. *Test system is connected by VDSL with pure IPv4 (No IPv6 or dual stack). *Setup can handle IPv4, IPv6 or dual stack updates

[Peter Müller]

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

Hi,

thanks for your reply.

First, please do not forget to keep the mailing list CC'ed. For transparency reasons, we have a policy here
stating to keep public conversations public (please refer to https://wiki.ipfire.org/devel/contact for details).

Second, your patches _did_ made it into patchwork. They can be found in the "ipfire" section - for your convenience,
here is a list of all active patches submitted by you patchwork currently knows about:
https://patchwork.ipfire.org/project/ipfire/list/?submitter=93

Once having logged in, you can mark patches as being "superseded", so they won't appear as being active anymore.

Third, I have seen your updated patch and will comment on it separately the other day.

Thanks, and best regards,
Peter Müller


> Hello Peter,
> 
> thanks for your reply. Sorry for confusion about the patch. I tried to fullfill the requirements, but due to my low experience on the git process it went accidantely wrong.
> 
> I will improve & resubmit my patch according to your comments.
> 
> What ist the right way to mark patches as obsolete or remove them? My patches doesn‘t seem to enter https://patchwork.ipfire.org.
> 
> Regards,
> 
> Martin Krieger
> 
> Von meinem iPad gesendet
> 
>> Am 17.05.2021 um 21:31 schrieb Peter Müller <peter.mueller(a)ipfire.org>:
>> Hi,
>>
>> thank you for providing this.
>>
>> I regret to inform you that
>>
>> (a) both this patch and it's precursor violate the syntax of patches applicable to git,
>>    having parts of the description and various tags mangled into the RFC 5322 / MIME
>>    "Subject" header.
>>
>>    You do not need to use "git send-email" as such (personally, I do not use it either,
>>    because my development environment cannot establish SMTP connections due to firewall
>>    policies). However, IPFire's development heavily relies on Git, so please figure out
>>    how to run the most basic commands such as "git commit" and "git format-patch" in
>>    order to properly submit your patches.
>>
>> (b) these patches each success each other, however missing appropriate version tags
>>    (such as [PATCH v2], [PATCH v3], etc.) in the RFC 5322 "Subject" header of their
>>    mails.
>>
>>    Since you did not mark the obsoleted patches as being superseded in https://patchwork.ipfire.org/,
>>    either, this makes it hard to keep track of the latest version of your patch. Eventually,
>>    we might apply, commend or reject based on the wrong one, because of simply having
>>    overlooked there is a new, fixed version of it available.
>>
>> Apart from that, it would be good to see this patch tested with IPv6 and dual-stack scenarios
>> as well - please see also my remark(s) on this below. If I understood the documentation of
>> that DDNS provider correctly, you do not really need to supply two IP addresses having distinct
>> families. Could you at least try executing this snippet with two different IPv4 addresses?
>>
>>> ---
>>> README                |  1 +
>>> src/ddns/providers.py | 27 +++++++++++++++++++++++++++
>>> 2 files changed, 28 insertions(+)
>>>
>>> diff --git a/README b/README
>>> index b6decb35c338..fa6ce5e598b8 100644
>>> --- a/README
>>> +++ b/README
>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>    easydns.com
>>>    enom.com
>>>    entrydns.net
>>> +    feste-ip.net
>>>    freedns.afraid.org
>>>    inwx.com|de|at|ch|es
>>>    itsdns.de
>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>> index 56e6620c78ab..009e2c9e337d 100644
>>> --- a/src/ddns/providers.py
>>> +++ b/src/ddns/providers.py
>>> @@ -1179,6 +1179,33 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>        raise DDNSUpdateError
>>>
>>>
>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>> +        handle    = "feste-ip.net"
>>> +        name      = "Feste-IP.Net"
>>> +        website   = "https://www.feste-ip.net/"
>>> +
>>> +        # Information about the format of the request is to be found
>>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>>> +
>>> +        myips = ("myip","myip2")
>>
>> Personal comment: Kinda ugly, but given the documentation of that provider, there is no
>> other choice to do this here...
>>
>>> +
>>> +        url = "https://members.feste-ip.net/nic/update/"
>>> +
>>> +        def update(self):
>>> +                data = {
>>> +                        "hostname" : self.hostname
>>> +                }
>>> +
>>> +                for proto in DDNSProvider.protocols:
>>> +                        idx = 0
>>> +                        tmpip = self.get_address(proto)
>>> +                        if tmpip:
>>> +                                data[self.myips[idx]] = tmpip
>>> +                                idx += 1
>>> +
>>> +                        self.send_request(data)
>>
>> What is happening here if $tmpip equals a Boolean False?
>>
>>> +
>>> +
>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>    handle    = "freedns.afraid.org"
>>>    name      = "freedns.afraid.org"
>>
>> Thanks, and best regards,
>> Peter Müller
> 

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Martin Krieger]

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

Hello Peter,

I marked the obsolete patches as "superseded".

Regards,

Martin Krieger

Peter Müller:
> Hi,
> 
> thanks for your reply.
> 
> First, please do not forget to keep the mailing list CC'ed. For transparency reasons, we have a policy here
> stating to keep public conversations public (please refer to https://wiki.ipfire.org/devel/contact for details).
> 
> Second, your patches _did_ made it into patchwork. They can be found in 
the "ipfire" section - for your convenience,
> here is a list of all active patches submitted by you patchwork currently knows about:
> https://patchwork.ipfire.org/project/ipfire/list/?submitter=93
> 
> Once having logged in, you can mark patches as being "superseded", so they won't appear as being active anymore.
> 
> Third, I have seen your updated patch and will comment on it separately 
the other day.
> 
> Thanks, and best regards,
> Peter Müller
> 
> 
>> Hello Peter,
>>
>> thanks for your reply. Sorry for confusion about the patch. I tried to 
fullfill the requirements, but due to my low experience on the git process it went accidantely wrong.
>>
>> I will improve & resubmit my patch according to your comments.
>>
>> What ist the right way to mark patches as obsolete or remove them? My patches doesn‘t seem to enter https://patchwork.ipfire.org.
>>
>> Regards,
>>
>> Martin Krieger
>>
>> Von meinem iPad gesendet
>>
>>> Am 17.05.2021 um 21:31 schrieb Peter Müller <peter.mueller(a)ipfire.org>:
>>> Hi,
>>>
>>> thank you for providing this.
>>>
>>> I regret to inform you that
>>>
>>> (a) both this patch and it's precursor violate the syntax of patches applicable to git,
>>>     having parts of the description and various tags mangled into the 
RFC 5322 / MIME
>>>     "Subject" header.
>>>
>>>     You do not need to use "git send-email" as such (personally, I do 
not use it either,
>>>     because my development environment cannot establish SMTP connections due to firewall
>>>     policies). However, IPFire's development heavily relies on Git, so please figure out
>>>     how to run the most basic commands such as "git commit" and "git format-patch" in
>>>     order to properly submit your patches.
>>>
>>> (b) these patches each success each other, however missing appropriate version tags
>>>     (such as [PATCH v2], [PATCH v3], etc.) in the RFC 5322 "Subject" header of their
>>>     mails.
>>>
>>>     Since you did not mark the obsoleted patches as being superseded in https://patchwork.ipfire.org/,
>>>     either, this makes it hard to keep track of the latest version of 
your patch. Eventually,
>>>     we might apply, commend or reject based on the wrong one, because 
of simply having
>>>     overlooked there is a new, fixed version of it available.
>>>
>>> Apart from that, it would be good to see this patch tested with IPv6 and dual-stack scenarios
>>> as well - please see also my remark(s) on this below. If I understood 
the documentation of
>>> that DDNS provider correctly, you do not really need to supply two IP 
addresses having distinct
>>> families. Could you at least try executing this snippet with two different IPv4 addresses?
>>>
>>>> ---
>>>> README                |  1 +
>>>> src/ddns/providers.py | 27 +++++++++++++++++++++++++++
>>>> 2 files changed, 28 insertions(+)
>>>>
>>>> diff --git a/README b/README
>>>> index b6decb35c338..fa6ce5e598b8 100644
>>>> --- a/README
>>>> +++ b/README
>>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>>     easydns.com
>>>>     enom.com
>>>>     entrydns.net
>>>> +    feste-ip.net
>>>>     freedns.afraid.org
>>>>     inwx.com|de|at|ch|es
>>>>     itsdns.de
>>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>>> index 56e6620c78ab..009e2c9e337d 100644
>>>> --- a/src/ddns/providers.py
>>>> +++ b/src/ddns/providers.py
>>>> @@ -1179,6 +1179,33 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>>         raise DDNSUpdateError
>>>>
>>>>
>>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>>> +        handle    = "feste-ip.net"
>>>> +        name      = "Feste-IP.Net"
>>>> +        website   = "https://www.feste-ip.net/"
>>>> +
>>>> +        # Information about the format of the request is to be found
>>>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>>>> +
>>>> +        myips = ("myip","myip2")
>>>
>>> Personal comment: Kinda ugly, but given the documentation of that provider, there is no
>>> other choice to do this here...
>>>
>>>> +
>>>> +        url = "https://members.feste-ip.net/nic/update/"
>>>> +
>>>> +        def update(self):
>>>> +                data = {
>>>> +                        "hostname" : self.hostname
>>>> +                }
>>>> +
>>>> +                for proto in DDNSProvider.protocols:
>>>> +                        idx = 0
>>>> +                        tmpip = self.get_address(proto)
>>>> +                        if tmpip:
>>>> +                                data[self.myips[idx]] = tmpip
>>>> +                                idx += 1
>>>> +
>>>> +                        self.send_request(data)
>>>
>>> What is happening here if $tmpip equals a Boolean False?
>>>
>>>> +
>>>> +
>>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>>     handle    = "freedns.afraid.org"
>>>>     name      = "freedns.afraid.org"
>>>
>>> Thanks, and best regards,
>>> Peter Müller
>>


[-- Attachment #2: OpenPGP_signature.sig --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH v6 1/1] ddns: Add provider Feste-IP.Net

[Michael Tremer]

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

Hello,

> On 20 May 2021, at 12:02, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> Hello Michael,
> 
> got it. You are right, sorry for confusion.
> 
> But I really read the process description and have read your comments as well as looking in providers.py for examples.
> 
> What should I do? Remove anything, improve it,...?

Yes, please improve the patch. If there is anything unclear, you should ask those questions first before sending a fixed version of that patch.

> Simply issue a request to you to implement that?

No, this won’t work because we don’t have an account with this provider to test it.

> One general question: Should a patch be send as uncommited diff from the staging area in before for revision instead of a commited one?

No, you should have a branch in your local git repository and commit to that. You can use “git commit —-amend” if you need to edit an existing commit.

Best,
-Michael

> 
> Regards,
> 
> Martin Krieger
> 
> 
> 
> Am 20. Mai 2021 12:00:26 MESZ schrieb Michael Tremer <michael.tremer(a)ipfire.org>:
> Hello Martin,
> 
> Thanks for working on ddns, but *please* read Peter’s email.
> 
> This is now the third patch (I don’t know why you are at number 6 already) and the idea of the review process is to submit something that at least works. You can ask all sorts of questions about anything you need to know, but you will have to listen to what people answer.
> 
> This patch is fundamentally broken. I will comment on that further below.
> 
> Secondly, we cannot even think about merging something that isn’t tested at all.
> 
> So I would urge you to look at any of the other providers, because what you are trying to implement has already been implemented here for example:
> 
>   https://git.ipfire.org/?p=ddns.git;a=blob;f=src/ddns/providers.py;h=56e6620c78ab3d9de1c945147f86e70a8d8614d7;hb=HEAD#l975
> 
> @Stefan: We might want to have a class variable in the DynDNS protocol that adds the IPv6 address to a field that is named in that variable. That way, we avoid any code duplication.
> 
> On 18 May 2021, at 19:49, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> From 37fb0237932fb12bdd635e9cb5e01b0bf9f03dda Mon Sep 17 00:00:00 2001
> From: Martin Krieger <makrie(a)posteo.de>
> Date: Tue, 18 May 2021 19:42:44 +0200
> Subject: [PATCH v6 1/1] ddns: Add provider Feste-IP.Net
> 
> Comments:
> Provider supports IPv4, IPv6 & DS (Dual-Stack)
> 
> Changelog:
> 
> 18.05.2021
> Improved failure handling.
> IPv6 & DS still not checked, because my network connection is pure IPv4.
> 
> 17.05.2021
> 5th attempt ([PATCH v5 1/1])
> 
> 16.05.2021
> 4th attempt ([PATCH v4 1/1])
> 
> 08.05.2021
> 3rd attempt ([PATCH v3 1/1])
> 
> 06.05.2021
> 2nd attempt ([PATCH v2 1/1])
> 
> 05.05.2021
> 1st attempt ([PATCH v1 1/1])
> 
> Signed-off-by: Martin Krieger <makrie(a)posteo.de>..
> README                |  1 +
> src/ddns/providers.py | 31 +++++++++++++++++++++++++++++++
> 2 files changed, 32 insertions(+)
> 
> diff --git a/README b/README
> index b6decb35c338..fa6ce5e598b8 100644
> --- a/README
> +++ b/README
> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
> 	easydns.com
> 	enom.com
> 	entrydns.net
> +	feste-ip.net
> 	freedns.afraid.org
> 	inwx.com|de|at|ch|es
> 	itsdns.de
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index 56e6620c78ab..b7e14482a0b1 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1179,6 +1179,37 @@ class DDNSProviderEntryDNS(DDNSProvider):
> 		raise DDNSUpdateError
> 
> 
> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    = "feste-ip.net"
> +        name      = "Feste-IP.Net"
> +        website   = "https://www.feste-ip.net/"
> +
> +        # Information about the format of the request is to be found
> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
> +
> +        myips = ("myip","myip2")
> +
> +        url = "https://members.feste-ip.net/nic/update/"
> +
> +        def update(self):
> +                data = {
> +                        "hostname" : self.hostname
> +                }
> +
> +                for proto in DDNSProvider.protocols:
> +                        idx = 0
> +                        tmpip = self.get_address(proto)
> +                        if tmpip:
> +                                data[self.myips[idx]] = tmpip
> +                                idx += 1
> 
> Setting idx is useless here. You will reset to zero in every iteration of the loop. Since it is being incremented last and then reset to zero, idx will *always* be zero.
> 
> +
> +                if self.myips[0] in data:
> +                        self.send_request(data)
> +                        return#
> 
> You will only send one request here and only if it contains an IPv4 address. Why? What happens to systems that only have IPv6?
> 
> -Michael
> 
> +
> +                raise DDNSUpdateError
> +
> +
> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
> 	handle    = "freedns.afraid.org"
> 	name      = "freedns.afraid.org"
> -- 
> 2.31.0
> 
> 
> 
> 
> 
> -- 
> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Martin Krieger]

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

Hello Michael,

this is my latest improvement:

 From c745be038550bf66f2c233ff9de1348bcfd99080 Mon Sep 17 00:00:00 2001
From: Martin Krieger <makrie(a)posteo.de>
Date: Tue, 18 May 2021 19:42:44 +0200
Subject: [PATCH] ddns: Add provider Feste-IP.Net

Provider supports IPv4, IPv6 & DS (Dual-Stack).
Required url parameter myip for IPv4 or IPv6.
Optional url parameter myip2 for DS which holds IPv4 or IPv6.
Url parameters myip & myip2 will not hold the same record type.
---
  README                |  1 +
  src/ddns/providers.py | 40 ++++++++++++++++++++++++++++++++++++++++
  2 files changed, 41 insertions(+)

diff --git a/README b/README
index b6decb35c338..fa6ce5e598b8 100644
--- a/README
+++ b/README
@@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
  	easydns.com
  	enom.com
  	entrydns.net
+	feste-ip.net
  	freedns.afraid.org
  	inwx.com|de|at|ch|es
  	itsdns.de
diff --git a/src/ddns/providers.py b/src/ddns/providers.py
index 56e6620c78ab..8c6da69b6e50 100644
--- a/src/ddns/providers.py
+++ b/src/ddns/providers.py
@@ -1179,6 +1179,46 @@ class DDNSProviderEntryDNS(DDNSProvider):
  		raise DDNSUpdateError


+class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
+        handle    = "feste-ip.net"
+        name      = "Feste-IP.Net"
+        website   = "https://www.feste-ip.net/"
+
+        # Information about the format of the request is to be found
+        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
+
+	# myip is required and holds IPv4 or IPv6
+	# myip2 is optional for DS and holds IPv4 or Ipv6 but opposite record 
type of myip
+        myips = ("myip","myip2")
+
+        url = "https://members.feste-ip.net/nic/update/"
+
+        def update(self):
+                data = {
+                        "hostname" : self.hostname
+                }
+
+		# idx sets index of myips and checks for required url parameter myip
+		idx = 0
+		# Loop through DDNSProvider.protocols list
+                for proto in DDNSProvider.protocols:
+                        idx = 0
+                        tmpip = self.get_address(proto)
+			# Save only valid address response of protocol proto to the actual 
url parameter...
+			# and increase idx
+                        if tmpip:
+                                data[self.myips[idx]] = tmpip
+                                idx += 1
+
+		# Send DynDNS2 update request if the required url parameter myip is given
+                if idx:
+                        self.send_request(data)
+                        return
+
+		# Raise error if the required if the required url parameter myip gets 
no valid address
+                raise DDNSUpdateError
+
+
  class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
  	handle    = "freedns.afraid.org"
  	name      = "freedns.afraid.org"
-- 
2.31.0

Based on your correct comment idx = 0 was on the wrong line, it moved 
outside the loop over the protocols (IPv4 & IPv6).

Pure IPv4 is tested & works as expected. As stated several times I have 
no option to test IPv6 and DS on my system.

The provider serves a test possibility 
(https://www.feste-ip.net/ddns-service/faq-ddns/):

"Kann Ich den Dienst testen ohne mich extra anzumelden?
Ja. Um zu testen ob Ihr Router bzw. Ihr Updateclient mit unserem Dienst 
funktioniert können Sie folgende Testdaten verwenden:

DNS-Name: test.feste-ip.net
HOST-ID : 13135
Passwort: 7sN2KS6L8W

Bitte beachten Sie dass diese Daten u.U. von mehreren Testern verwendet 
werden.
Ob Ihr Update erfolgreich war können Sie mit der zeitnahen Eingabe von 
"nslookup -query=txt test.feste-ip.net" prüfen."

Could please anybody in the development-team who has the right technical 
conditions (IPv6 or DS) do test for these cases? That would be nice. Thanks.

Should I mark my faulty patch on the ipfire git server as superseded?

Regards,

Martin Krieger

Michael Tremer:
> Hello,
> 
>> On 20 May 2021, at 12:02, Martin Krieger <makrie(a)posteo.de> wrote:
>>
>> Hello Michael,
>>
>> got it. You are right, sorry for confusion.
>>
>> But I really read the process description and have read your comments as well as looking in providers.py for examples.
>>
>> What should I do? Remove anything, improve it,...?
> 
> Yes, please improve the patch. If there is anything unclear, you should 
ask those questions first before sending a fixed version of that patch.
> 
>> Simply issue a request to you to implement that?
> 
> No, this won’t work because we don’t have an account with this provider to test it.
> 
>> One general question: Should a patch be send as uncommited diff from the staging area in before for revision instead of a commited one?
> 
> No, you should have a branch in your local git repository and commit to 
that. You can use “git commit —-amend” if you need to edit an existing commit.
> 
> Best,
> -Michael
> 
>>
>> Regards,
>>
>> Martin Krieger
>>
>>
>>
>> Am 20. Mai 2021 12:00:26 MESZ schrieb Michael Tremer <michael.tremer(a)ipfire.org>:
>> Hello Martin,
>>
>> Thanks for working on ddns, but *please* read Peter’s email.
>>
>> This is now the third patch (I don’t know why you are at number 6 already) and the idea of the review process is to submit something that at least works. You can ask all sorts of questions about anything you need to know, but you will have to listen to what people answer.
>>
>> This patch is fundamentally broken. I will comment on that further below.
>>
>> Secondly, we cannot even think about merging something that isn’t tested at all.
>>
>> So I would urge you to look at any of the other providers, because what you are trying to implement has already been implemented here for example:
>>
>>    https://git.ipfire.org/?p=ddns.git;a=blob;f=src/ddns/providers.py;h=56e6620c78ab3d9de1c945147f86e70a8d8614d7;hb=HEAD#l975
>>
>> @Stefan: We might want to have a class variable in the DynDNS protocol 
that adds the IPv6 address to a field that is named in that variable. That way, we avoid any code duplication.
>>
>> On 18 May 2021, at 19:49, Martin Krieger <makrie(a)posteo.de> wrote:
>>
>>  From 37fb0237932fb12bdd635e9cb5e01b0bf9f03dda Mon Sep 17 00:00:00 2001
>> From: Martin Krieger <makrie(a)posteo.de>
>> Date: Tue, 18 May 2021 19:42:44 +0200
>> Subject: [PATCH v6 1/1] ddns: Add provider Feste-IP.Net
>>
>> Comments:
>> Provider supports IPv4, IPv6 & DS (Dual-Stack)
>>
>> Changelog:
>>
>> 18.05.2021
>> Improved failure handling.
>> IPv6 & DS still not checked, because my network connection is pure IPv4.
>>
>> 17.05.2021
>> 5th attempt ([PATCH v5 1/1])
>>
>> 16.05.2021
>> 4th attempt ([PATCH v4 1/1])
>>
>> 08.05.2021
>> 3rd attempt ([PATCH v3 1/1])
>>
>> 06.05.2021
>> 2nd attempt ([PATCH v2 1/1])
>>
>> 05.05.2021
>> 1st attempt ([PATCH v1 1/1])
>>
>> Signed-off-by: Martin Krieger <makrie(a)posteo.de>..
>> README                |  1 +
>> src/ddns/providers.py | 31 +++++++++++++++++++++++++++++++
>> 2 files changed, 32 insertions(+)
>>
>> diff --git a/README b/README
>> index b6decb35c338..fa6ce5e598b8 100644
>> --- a/README
>> +++ b/README
>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>> 	easydns.com
>> 	enom.com
>> 	entrydns.net
>> +	feste-ip.net
>> 	freedns.afraid.org
>> 	inwx.com|de|at|ch|es
>> 	itsdns.de
>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>> index 56e6620c78ab..b7e14482a0b1 100644
>> --- a/src/ddns/providers.py
>> +++ b/src/ddns/providers.py
>> @@ -1179,6 +1179,37 @@ class DDNSProviderEntryDNS(DDNSProvider):
>> 		raise DDNSUpdateError
>>
>>
>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>> +        handle    = "feste-ip.net"
>> +        name      = "Feste-IP.Net"
>> +        website   = "https://www.feste-ip.net/"
>> +
>> +        # Information about the format of the request is to be found
>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>> +
>> +        myips = ("myip","myip2")
>> +
>> +        url = "https://members.feste-ip.net/nic/update/"
>> +
>> +        def update(self):
>> +                data = {
>> +                        "hostname" : self.hostname
>> +                }
>> +
>> +                for proto in DDNSProvider.protocols:
>> +                        idx = 0
>> +                        tmpip = self.get_address(proto)
>> +                        if tmpip:
>> +                                data[self.myips[idx]] = tmpip
>> +                                idx += 1
>>
>> Setting idx is useless here. You will reset to zero in every iteration 
of the loop. Since it is being incremented last and then reset to zero, idx will *always* be zero.
>>
>> +
>> +                if self.myips[0] in data:
>> +                        self.send_request(data)
>> +                        return#
>>
>> You will only send one request here and only if it contains an IPv4 address. Why? What happens to systems that only have IPv6?
>>
>> -Michael
>>
>> +
>> +                raise DDNSUpdateError
>> +
>> +
>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>> 	handle    = "freedns.afraid.org"
>> 	name      = "freedns.afraid.org"
>> -- 
>> 2.31.0
>>
>>
>>
>>
>>
>> -- 
>> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
> 


[-- Attachment #2: OpenPGP_signature.sig --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Michael Tremer]

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

Hello Martin,

> On 20 May 2021, at 17:51, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> Hello Michael,
> 
> this is my latest improvement:
> 
> From c745be038550bf66f2c233ff9de1348bcfd99080 Mon Sep 17 00:00:00 2001
> From: Martin Krieger <makrie(a)posteo.de>
> Date: Tue, 18 May 2021 19:42:44 +0200
> Subject: [PATCH] ddns: Add provider Feste-IP.Net
> 
> Provider supports IPv4, IPv6 & DS (Dual-Stack).
> Required url parameter myip for IPv4 or IPv6.
> Optional url parameter myip2 for DS which holds IPv4 or IPv6.
> Url parameters myip & myip2 will not hold the same record type.
> ---
> README                |  1 +
> src/ddns/providers.py | 40 ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 41 insertions(+)
> 
> diff --git a/README b/README
> index b6decb35c338..fa6ce5e598b8 100644
> --- a/README
> +++ b/README
> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
> 	easydns.com
> 	enom.com
> 	entrydns.net
> +	feste-ip.net
> 	freedns.afraid.org
> 	inwx.com|de|at|ch|es
> 	itsdns.de
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index 56e6620c78ab..8c6da69b6e50 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1179,6 +1179,46 @@ class DDNSProviderEntryDNS(DDNSProvider):
> 		raise DDNSUpdateError
> 
> 
> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    = "feste-ip.net"
> +        name      = "Feste-IP.Net"
> +        website   = "https://www.feste-ip.net/"
> +
> +        # Information about the format of the request is to be found
> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
> +
> +	# myip is required and holds IPv4 or IPv6
> +	# myip2 is optional for DS and holds IPv4 or Ipv6 but opposite record type of myip
> +        myips = ("myip","myip2")
> +
> +        url = "https://members.feste-ip.net/nic/update/"
> +
> +        def update(self):
> +                data = {
> +                        "hostname" : self.hostname
> +                }
> +
> +		# idx sets index of myips and checks for required url parameter myip
> +		idx = 0
> +		# Loop through DDNSProvider.protocols list
> +                for proto in DDNSProvider.protocols:
> +                        idx = 0
> +                        tmpip = self.get_address(proto)
> +			# Save only valid address response of protocol proto to the actual url parameter...
> +			# and increase idx
> +                        if tmpip:
> +                                data[self.myips[idx]] = tmpip
> +                                idx += 1
> +
> +		# Send DynDNS2 update request if the required url parameter myip is given
> +                if idx:
> +                        self.send_request(data)
> +                        return
> +
> +		# Raise error if the required if the required url parameter myip gets no valid address
> +                raise DDNSUpdateError
> +
> +
> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
> 	handle    = "freedns.afraid.org"
> 	name      = "freedns.afraid.org"
> -- 
> 2.31.0
> 
> Based on your correct comment idx = 0 was on the wrong line, it moved outside the loop over the protocols (IPv4 & IPv6).

You ignored all technical advice that I have given to you. There are solutions already in the same file that you simply need to copy and paste. They are tested, clean and just work.

Your implementation is not very Pythonic and raises a random error if no IP address could be determined.

> Pure IPv4 is tested & works as expected. As stated several times I have no option to test IPv6 and DS on my system.
> 
> The provider serves a test possibility (https://www.feste-ip.net/ddns-service/faq-ddns/):
> 
> "Kann Ich den Dienst testen ohne mich extra anzumelden?
> Ja. Um zu testen ob Ihr Router bzw. Ihr Updateclient mit unserem Dienst funktioniert können Sie folgende Testdaten verwenden:
> 
> DNS-Name: test.feste-ip.net
> HOST-ID : 13135
> Passwort: 7sN2KS6L8W
> 
> Bitte beachten Sie dass diese Daten u.U. von mehreren Testern verwendet werden.
> Ob Ihr Update erfolgreich war können Sie mit der zeitnahen Eingabe von "nslookup -query=txt test.feste-ip.net" prüfen."
> 
> Could please anybody in the development-team who has the right technical conditions (IPv6 or DS) do test for these cases? That would be nice. Thanks.
> 
> Should I mark my faulty patch on the ipfire git server as superseded?

Yes, please.

> Regards,
> 
> Martin Krieger
> 
> Michael Tremer:
>> Hello,
>>> On 20 May 2021, at 12:02, Martin Krieger <makrie(a)posteo.de> wrote:
>>> 
>>> Hello Michael,
>>> 
>>> got it. You are right, sorry for confusion.
>>> 
>>> But I really read the process description and have read your comments as well as looking in providers.py for examples.
>>> 
>>> What should I do? Remove anything, improve it,...?
>> Yes, please improve the patch. If there is anything unclear, you should 
> ask those questions first before sending a fixed version of that patch.
>>> Simply issue a request to you to implement that?
>> No, this won’t work because we don’t have an account with this provider to test it.
>>> One general question: Should a patch be send as uncommited diff from the staging area in before for revision instead of a commited one?
>> No, you should have a branch in your local git repository and commit to 
> that. You can use “git commit —-amend” if you need to edit an existing commit.
>> Best,
>> -Michael
>>> 
>>> Regards,
>>> 
>>> Martin Krieger
>>> 
>>> 
>>> 
>>> Am 20. Mai 2021 12:00:26 MESZ schrieb Michael Tremer <michael.tremer(a)ipfire.org>:
>>> Hello Martin,
>>> 
>>> Thanks for working on ddns, but *please* read Peter’s email.
>>> 
>>> This is now the third patch (I don’t know why you are at number 6 already) and the idea of the review process is to submit something that at least works. You can ask all sorts of questions about anything you need to know, but you will have to listen to what people answer.
>>> 
>>> This patch is fundamentally broken. I will comment on that further below.
>>> 
>>> Secondly, we cannot even think about merging something that isn’t tested at all.
>>> 
>>> So I would urge you to look at any of the other providers, because what you are trying to implement has already been implemented here for example:
>>> 
>>>   https://git.ipfire.org/?p=ddns.git;a=blob;f=src/ddns/providers.py;h=56e6620c78ab3d9de1c945147f86e70a8d8614d7;hb=HEAD#l975
>>> 
>>> @Stefan: We might want to have a class variable in the DynDNS protocol 
> that adds the IPv6 address to a field that is named in that variable. That way, we avoid any code duplication.
>>> 
>>> On 18 May 2021, at 19:49, Martin Krieger <makrie(a)posteo.de> wrote:
>>> 
>>> From 37fb0237932fb12bdd635e9cb5e01b0bf9f03dda Mon Sep 17 00:00:00 2001
>>> From: Martin Krieger <makrie(a)posteo.de>
>>> Date: Tue, 18 May 2021 19:42:44 +0200
>>> Subject: [PATCH v6 1/1] ddns: Add provider Feste-IP.Net
>>> 
>>> Comments:
>>> Provider supports IPv4, IPv6 & DS (Dual-Stack)
>>> 
>>> Changelog:
>>> 
>>> 18.05.2021
>>> Improved failure handling.
>>> IPv6 & DS still not checked, because my network connection is pure IPv4.
>>> 
>>> 17.05.2021
>>> 5th attempt ([PATCH v5 1/1])
>>> 
>>> 16.05.2021
>>> 4th attempt ([PATCH v4 1/1])
>>> 
>>> 08.05.2021
>>> 3rd attempt ([PATCH v3 1/1])
>>> 
>>> 06.05.2021
>>> 2nd attempt ([PATCH v2 1/1])
>>> 
>>> 05.05.2021
>>> 1st attempt ([PATCH v1 1/1])
>>> 
>>> Signed-off-by: Martin Krieger <makrie(a)posteo.de>..
>>> README                |  1 +
>>> src/ddns/providers.py | 31 +++++++++++++++++++++++++++++++
>>> 2 files changed, 32 insertions(+)
>>> 
>>> diff --git a/README b/README
>>> index b6decb35c338..fa6ce5e598b8 100644
>>> --- a/README
>>> +++ b/README
>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>> 	easydns.com
>>> 	enom.com
>>> 	entrydns.net
>>> +	feste-ip.net
>>> 	freedns.afraid.org
>>> 	inwx.com|de|at|ch|es
>>> 	itsdns.de
>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>> index 56e6620c78ab..b7e14482a0b1 100644
>>> --- a/src/ddns/providers.py
>>> +++ b/src/ddns/providers.py
>>> @@ -1179,6 +1179,37 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>> 		raise DDNSUpdateError
>>> 
>>> 
>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>> +        handle    = "feste-ip.net"
>>> +        name      = "Feste-IP.Net"
>>> +        website   = "https://www.feste-ip.net/"
>>> +
>>> +        # Information about the format of the request is to be found
>>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>>> +
>>> +        myips = ("myip","myip2")
>>> +
>>> +        url = "https://members.feste-ip.net/nic/update/"
>>> +
>>> +        def update(self):
>>> +                data = {
>>> +                        "hostname" : self.hostname
>>> +                }
>>> +
>>> +                for proto in DDNSProvider.protocols:
>>> +                        idx = 0
>>> +                        tmpip = self.get_address(proto)
>>> +                        if tmpip:
>>> +                                data[self.myips[idx]] = tmpip
>>> +                                idx += 1
>>> 
>>> Setting idx is useless here. You will reset to zero in every iteration 
> of the loop. Since it is being incremented last and then reset to zero, idx will *always* be zero.
>>> 
>>> +
>>> +                if self.myips[0] in data:
>>> +                        self.send_request(data)
>>> +                        return#
>>> 
>>> You will only send one request here and only if it contains an IPv4 address. Why? What happens to systems that only have IPv6?
>>> 
>>> -Michael
>>> 
>>> +
>>> +                raise DDNSUpdateError
>>> +
>>> +
>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>> 	handle    = "freedns.afraid.org"
>>> 	name      = "freedns.afraid.org"
>>> -- 
>>> 2.31.0
>>> 
>>> 
>>> 
>>> 
>>> 
>>> -- 
>>> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
> 

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Martin Krieger]

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

Hello Michael,

I hope this will leave the path of ignoring *any* technical advice.

The (re-)definition of list providers in the derived class seems to be 
not necessary. Am I right?

Anything left or maybe simply misunderstood by me?

Regards,

Martin Krieger

 From 2e0f9067d04d6250b476a1cb6e5224ba8d91be70 Mon Sep 17 00:00:00 2001
From: Martin Krieger <makrie(a)posteo.de>
Date: Tue, 18 May 2021 19:42:44 +0200
Subject: [PATCH] ddns: Add provider Feste-IP.Net

Provider Feste-IP.Net addapted from class DDNSProviderDesecIO
---
  README                |  1 +
  src/ddns/providers.py | 29 +++++++++++++++++++++++++++++
  2 files changed, 30 insertions(+)

diff --git a/README b/README
index b6decb35c338..fa6ce5e598b8 100644
--- a/README
+++ b/README
@@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
      easydns.com
      enom.com
      entrydns.net
+    feste-ip.net
      freedns.afraid.org
      inwx.com|de|at|ch|es
      itsdns.de
diff --git a/src/ddns/providers.py b/src/ddns/providers.py
index 56e6620c78ab..e274fb4f7262 100644
--- a/src/ddns/providers.py
+++ b/src/ddns/providers.py
@@ -1179,6 +1179,35 @@ class DDNSProviderEntryDNS(DDNSProvider):
          raise DDNSUpdateError


+class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
+        handle    = "feste-ip.net"
+        name      = "Feste-IP.Net"
+        website   = "https://www.feste-ip.net/"
+        # protocols = ("ipv6", "ipv4",)
+
+        # Information about the format of the request is to be found
+        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
+        # ipv4 / ipv6 records are automatically removed when the update
+        # request originates from the respectively other protocol and no
+        # address is explicitly provided for the unused protocol.
+
+        url = "https://members.feste-ip.net/nic/update/"
+
+        # feste-ip.net sends the IPv6 and IPv4 address in one request
+
+        def update(self):
+                data = DDNSProtocolDynDNS2.prepare_request_data(self, 
"ipv4")
+
+                # This one supports IPv6
+                myip2 = self.get_address("ipv6")
+
+                # Add update information if we have an IPv6 address.
+                if myip2:
+                        data["myip2"] = myip2
+
+                self.send_request(data)
+
+
  class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
      handle    = "freedns.afraid.org"
      name      = "freedns.afraid.org"
-- 
2.31.0

Michael Tremer:
> Hello Martin,
> 
>> On 20 May 2021, at 17:51, Martin Krieger <makrie(a)posteo.de> wrote:
>>
>> Hello Michael,
>>
>> this is my latest improvement:
>>
>>  From c745be038550bf66f2c233ff9de1348bcfd99080 Mon Sep 17 00:00:00 2001
>> From: Martin Krieger <makrie(a)posteo.de>
>> Date: Tue, 18 May 2021 19:42:44 +0200
>> Subject: [PATCH] ddns: Add provider Feste-IP.Net
>>
>> Provider supports IPv4, IPv6 & DS (Dual-Stack).
>> Required url parameter myip for IPv4 or IPv6.
>> Optional url parameter myip2 for DS which holds IPv4 or IPv6.
>> Url parameters myip & myip2 will not hold the same record type.
>> ---
>> README                |  1 +
>> src/ddns/providers.py | 40 ++++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 41 insertions(+)
>>
>> diff --git a/README b/README
>> index b6decb35c338..fa6ce5e598b8 100644
>> --- a/README
>> +++ b/README
>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>> 	easydns.com
>> 	enom.com
>> 	entrydns.net
>> +	feste-ip.net
>> 	freedns.afraid.org
>> 	inwx.com|de|at|ch|es
>> 	itsdns.de
>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>> index 56e6620c78ab..8c6da69b6e50 100644
>> --- a/src/ddns/providers.py
>> +++ b/src/ddns/providers.py
>> @@ -1179,6 +1179,46 @@ class DDNSProviderEntryDNS(DDNSProvider):
>> 		raise DDNSUpdateError
>>
>>
>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>> +        handle    = "feste-ip.net"
>> +        name      = "Feste-IP.Net"
>> +        website   = "https://www.feste-ip.net/"
>> +
>> +        # Information about the format of the request is to be found
>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>> +
>> +	# myip is required and holds IPv4 or IPv6
>> +	# myip2 is optional for DS and holds IPv4 or Ipv6 but opposite record type of myip
>> +        myips = ("myip","myip2")
>> +
>> +        url = "https://members.feste-ip.net/nic/update/"
>> +
>> +        def update(self):
>> +                data = {
>> +                        "hostname" : self.hostname
>> +                }
>> +
>> +		# idx sets index of myips and checks for required url parameter myip
>> +		idx = 0
>> +		# Loop through DDNSProvider.protocols list
>> +                for proto in DDNSProvider.protocols:
>> +                        idx = 0
>> +                        tmpip = self.get_address(proto)
>> +			# Save only valid address response of protocol proto to the actual 
url parameter...
>> +			# and increase idx
>> +                        if tmpip:
>> +                                data[self.myips[idx]] = tmpip
>> +                                idx += 1
>> +
>> +		# Send DynDNS2 update request if the required url parameter myip is 
given
>> +                if idx:
>> +                        self.send_request(data)
>> +                        return
>> +
>> +		# Raise error if the required if the required url parameter myip gets no valid address
>> +                raise DDNSUpdateError
>> +
>> +
>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>> 	handle    = "freedns.afraid.org"
>> 	name      = "freedns.afraid.org"
>> -- 
>> 2.31.0
>>
>> Based on your correct comment idx = 0 was on the wrong line, it moved outside the loop over the protocols (IPv4 & IPv6).
> 
> You ignored all technical advice that I have given to you. There are solutions already in the same file that you simply need to copy and paste. They are tested, clean and just work.
> 
> Your implementation is not very Pythonic and raises a random error if no IP address could be determined.
> 
>> Pure IPv4 is tested & works as expected. As stated several times I have no option to test IPv6 and DS on my system.
>>
>> The provider serves a test possibility (https://www.feste-ip.net/ddns-service/faq-ddns/):
>>
>> "Kann Ich den Dienst testen ohne mich extra anzumelden?
>> Ja. Um zu testen ob Ihr Router bzw. Ihr Updateclient mit unserem Dienst funktioniert können Sie folgende Testdaten verwenden:
>>
>> DNS-Name: test.feste-ip.net
>> HOST-ID : 13135
>> Passwort: 7sN2KS6L8W
>>
>> Bitte beachten Sie dass diese Daten u.U. von mehreren Testern verwendet werden.
>> Ob Ihr Update erfolgreich war können Sie mit der zeitnahen Eingabe von "nslookup -query=txt test.feste-ip.net" prüfen."
>>
>> Could please anybody in the development-team who has the right technical conditions (IPv6 or DS) do test for these cases? That would be nice. Thanks.
>>
>> Should I mark my faulty patch on the ipfire git server as superseded?
> 
> Yes, please.
> 
>> Regards,
>>
>> Martin Krieger
>>
>> Michael Tremer:
>>> Hello,
>>>> On 20 May 2021, at 12:02, Martin Krieger <makrie(a)posteo.de> wrote:
>>>>
>>>> Hello Michael,
>>>>
>>>> got it. You are right, sorry for confusion.
>>>>
>>>> But I really read the process description and have read your comments as well as looking in providers.py for examples.
>>>>
>>>> What should I do? Remove anything, improve it,...?
>>> Yes, please improve the patch. If there is anything unclear, you should
>> ask those questions first before sending a fixed version of that patch.
>>>> Simply issue a request to you to implement that?
>>> No, this won’t work because we don’t have an account with this provider to test it.
>>>> One general question: Should a patch be send as uncommited diff from 
the staging area in before for revision instead of a commited one?
>>> No, you should have a branch in your local git repository and commit to
>> that. You can use “git commit —-amend” if you need to edit an existing commit.
>>> Best,
>>> -Michael
>>>>
>>>> Regards,
>>>>
>>>> Martin Krieger
>>>>
>>>>
>>>>
>>>> Am 20. Mai 2021 12:00:26 MESZ schrieb Michael Tremer <michael.tremer(a)ipfire.org>:
>>>> Hello Martin,
>>>>
>>>> Thanks for working on ddns, but *please* read Peter’s email.
>>>>
>>>> This is now the third patch (I don’t know why you are at number 6 already) and the idea of the review process is to submit something that at least works. You can ask all sorts of questions about anything you need to know, but you will have to listen to what people answer.
>>>>
>>>> This patch is fundamentally broken. I will comment on that further below.
>>>>
>>>> Secondly, we cannot even think about merging something that isn’t tested at all.
>>>>
>>>> So I would urge you to look at any of the other providers, because what you are trying to implement has already been implemented here for example:
>>>>
>>>>    https://git.ipfire.org/?p=ddns.git;a=blob;f=src/ddns/providers.py;h=56e6620c78ab3d9de1c945147f86e70a8d8614d7;hb=HEAD#l975
>>>>
>>>> @Stefan: We might want to have a class variable in the DynDNS protocol
>> that adds the IPv6 address to a field that is named in that variable. That way, we avoid any code duplication.
>>>>
>>>> On 18 May 2021, at 19:49, Martin Krieger <makrie(a)posteo.de> wrote:
>>>>
>>>>  From 37fb0237932fb12bdd635e9cb5e01b0bf9f03dda Mon Sep 17 00:00:00 2001
>>>> From: Martin Krieger <makrie(a)posteo.de>
>>>> Date: Tue, 18 May 2021 19:42:44 +0200
>>>> Subject: [PATCH v6 1/1] ddns: Add provider Feste-IP.Net
>>>>
>>>> Comments:
>>>> Provider supports IPv4, IPv6 & DS (Dual-Stack)
>>>>
>>>> Changelog:
>>>>
>>>> 18.05.2021
>>>> Improved failure handling.
>>>> IPv6 & DS still not checked, because my network connection is pure IPv4.
>>>>
>>>> 17.05.2021
>>>> 5th attempt ([PATCH v5 1/1])
>>>>
>>>> 16.05.2021
>>>> 4th attempt ([PATCH v4 1/1])
>>>>
>>>> 08.05.2021
>>>> 3rd attempt ([PATCH v3 1/1])
>>>>
>>>> 06.05.2021
>>>> 2nd attempt ([PATCH v2 1/1])
>>>>
>>>> 05.05.2021
>>>> 1st attempt ([PATCH v1 1/1])
>>>>
>>>> Signed-off-by: Martin Krieger <makrie(a)posteo.de>..
>>>> README                |  1 +
>>>> src/ddns/providers.py | 31 +++++++++++++++++++++++++++++++
>>>> 2 files changed, 32 insertions(+)
>>>>
>>>> diff --git a/README b/README
>>>> index b6decb35c338..fa6ce5e598b8 100644
>>>> --- a/README
>>>> +++ b/README
>>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>> 	easydns.com
>>>> 	enom.com
>>>> 	entrydns.net
>>>> +	feste-ip.net
>>>> 	freedns.afraid.org
>>>> 	inwx.com|de|at|ch|es
>>>> 	itsdns.de
>>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>>> index 56e6620c78ab..b7e14482a0b1 100644
>>>> --- a/src/ddns/providers.py
>>>> +++ b/src/ddns/providers.py
>>>> @@ -1179,6 +1179,37 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>> 		raise DDNSUpdateError
>>>>
>>>>
>>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>>> +        handle    = "feste-ip.net"
>>>> +        name      = "Feste-IP.Net"
>>>> +        website   = "https://www.feste-ip.net/"
>>>> +
>>>> +        # Information about the format of the request is to be found
>>>> +        # https://forum.feste-ip.net/viewtopic.php?f=13&t=469
>>>> +
>>>> +        myips = ("myip","myip2")
>>>> +
>>>> +        url = "https://members.feste-ip.net/nic/update/"
>>>> +
>>>> +        def update(self):
>>>> +                data = {
>>>> +                        "hostname" : self.hostname
>>>> +                }
>>>> +
>>>> +                for proto in DDNSProvider.protocols:
>>>> +                        idx = 0
>>>> +                        tmpip = self.get_address(proto)
>>>> +                        if tmpip:
>>>> +                                data[self.myips[idx]] = tmpip
>>>> +                                idx += 1
>>>>
>>>> Setting idx is useless here. You will reset to zero in every iteration
>> of the loop. Since it is being incremented last and then reset to zero, idx will *always* be zero.
>>>>
>>>> +
>>>> +                if self.myips[0] in data:
>>>> +                        self.send_request(data)
>>>> +                        return#
>>>>
>>>> You will only send one request here and only if it contains an IPv4 address. Why? What happens to systems that only have IPv6?
>>>>
>>>> -Michael
>>>>
>>>> +
>>>> +                raise DDNSUpdateError
>>>> +
>>>> +
>>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>> 	handle    = "freedns.afraid.org"
>>>> 	name      = "freedns.afraid.org"
>>>> -- 
>>>> 2.31.0
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> -- 
>>>> Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
>>
> 


[-- Attachment #2: OpenPGP_signature.sig --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: ddns add provider Feste-IP.NET

[Michael Tremer]

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

Hello Martin,

Thank you very much for your submission.

> On 5 May 2021, at 19:13, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> diff --git a/README b/README
> index b6decb3..fa6ce5e 100644
> --- a/README
> +++ b/README
> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>        easydns.com
>        enom.com
>        entrydns.net
> +       feste-ip.net
>        freedns.afraid.org
>        inwx.com|de|at|ch|es
>        itsdns.de
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index 56e6620..c70423b 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>                # If we got here, some other update error happened.
>                raise DDNSUpdateError
> 
> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    = "feste-ip.net"
> +        name      = "Feste-IP.Net"
> +        website   = "https//www.feste-ip.net/"

You are missing a “:” in the URL.

> +        protocols = ("ipv4",)

According to this documentation, the provider supports IPv6 as well:

  https://www.feste-ip.net/ddns-service/einrichtung/linux/

> +        # Information about the format of the request is to be found
> +        # https://www.feste-ip.net/ddns-service/allgemeine-informationen/
> +
> +        url = "https://members.feste-ip.net/nic/update"
> 
> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>        handle    = “freedns.afraid.org"
> 

Best,
-Michael

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Martin Krieger]

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

Hello Michael,

by eMail again, because I didn't figure out to use git (commit, 
format-patch & send-email) in the correct way for patch submission. Sorry.

Regards,

Martin Krieger

diff --git a/README b/README
index b6decb3..fa6ce5e 100644
--- a/README
+++ b/README
@@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
         easydns.com
         enom.com
         entrydns.net
+       feste-ip.net
         freedns.afraid.org
         inwx.com|de|at|ch|es
         itsdns.de
diff --git a/src/ddns/providers.py b/src/ddns/providers.py
index 56e6620..c53ff9a 100644
--- a/src/ddns/providers.py
+++ b/src/ddns/providers.py
@@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
                 # If we got here, some other update error happened.
                 raise DDNSUpdateError

+class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
+        handle    = "feste-ip.net"
+        name      = "Feste-IP.Net"
+        website   = "https://www.feste-ip.net/"
+        protocols = ("ipv4","ipv6")
+
+        # Information about the format of the request is to be found
+        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
+
+        url = "https://members.feste-ip.net/nic/update"

  class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
         handle    = "freedns.afraid.org"

Michael Tremer:
> Hello Martin,
> 
> Thank you very much for your submission.
> 
>> On 5 May 2021, at 19:13, Martin Krieger <makrie(a)posteo.de> wrote:
>>
>> diff --git a/README b/README
>> index b6decb3..fa6ce5e 100644
>> --- a/README
>> +++ b/README
>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>         easydns.com
>>         enom.com
>>         entrydns.net
>> +       feste-ip.net
>>         freedns.afraid.org
>>         inwx.com|de|at|ch|es
>>         itsdns.de
>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>> index 56e6620..c70423b 100644
>> --- a/src/ddns/providers.py
>> +++ b/src/ddns/providers.py
>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>                 # If we got here, some other update error happened.
>>                 raise DDNSUpdateError
>>
>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>> +        handle    = "feste-ip.net"
>> +        name      = "Feste-IP.Net"
>> +        website   = "https//www.feste-ip.net/"
> 
> You are missing a “:” in the URL.
> 
>> +        protocols = ("ipv4",)
> 
> According to this documentation, the provider supports IPv6 as well:
> 
>    https://www.feste-ip.net/ddns-service/einrichtung/linux/
> 
>> +        # Information about the format of the request is to be found
>> +        # https://www.feste-ip.net/ddns-service/allgemeine-informationen/
>> +
>> +        url = "https://members.feste-ip.net/nic/update"
>>
>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>         handle    = “freedns.afraid.org"
>>
> 
> Best,
> -Michael
> 


[-- Attachment #2: OpenPGP_signature.sig --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Michael Tremer]

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

Hello Martin,

> On 6 May 2021, at 11:21, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> Hello Michael,
> 
> by eMail again, because I didn't figure out to use git (commit, format-patch & send-email) in the correct way for patch submission. Sorry.
> 
> Regards,
> 
> Martin Krieger
> 
> diff --git a/README b/README
> index b6decb3..fa6ce5e 100644
> --- a/README
> +++ b/README
> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>        easydns.com
>        enom.com
>        entrydns.net
> +       feste-ip.net
>        freedns.afraid.org
>        inwx.com|de|at|ch|es
>        itsdns.de
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index 56e6620..c53ff9a 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>                # If we got here, some other update error happened.
>                raise DDNSUpdateError
> 
> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    = "feste-ip.net"
> +        name      = "Feste-IP.Net"
> +        website   = "https://www.feste-ip.net/"
> +        protocols = ("ipv4","ipv6")

According to the documentation, the request has to be sent to a different host for IPv6. Just adding it to the list probably won’t work because this provider has implemented a different way to use the DynDNS API.

> +        # Information about the format of the request is to be found
> +        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
> +
> +        url = "https://members.feste-ip.net/nic/update"
> 
> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>        handle    = "freedns.afraid.org"
> 
> Michael Tremer:
>> Hello Martin,
>> Thank you very much for your submission.
>>> On 5 May 2021, at 19:13, Martin Krieger <makrie(a)posteo.de> wrote:
>>> 
>>> diff --git a/README b/README
>>> index b6decb3..fa6ce5e 100644
>>> --- a/README
>>> +++ b/README
>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>        easydns.com
>>>        enom.com
>>>        entrydns.net
>>> +       feste-ip.net
>>>        freedns.afraid.org
>>>        inwx.com|de|at|ch|es
>>>        itsdns.de
>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>> index 56e6620..c70423b 100644
>>> --- a/src/ddns/providers.py
>>> +++ b/src/ddns/providers.py
>>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>                # If we got here, some other update error happened.
>>>                raise DDNSUpdateError
>>> 
>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>> +        handle    = "feste-ip.net"
>>> +        name      = "Feste-IP.Net"
>>> +        website   = "https//www.feste-ip.net/"
>> You are missing a “:” in the URL.
>>> +        protocols = ("ipv4",)
>> According to this documentation, the provider supports IPv6 as well:
>>   https://www.feste-ip.net/ddns-service/einrichtung/linux/
>>> +        # Information about the format of the request is to be found
>>> +        # https://www.feste-ip.net/ddns-service/allgemeine-informationen/
>>> +
>>> +        url = "https://members.feste-ip.net/nic/update"
>>> 
>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>        handle    = “freedns.afraid.org"
>>> 
>> Best,
>> -Michael
> 

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Martin Krieger]

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

Hello Michael,

unfortunately my actual Python knowledge is very basic. I found no 
solution to implement ipv6 in addition to ipv4. Also IPFire is based on 
ipv4 and will be there in future as far as I understood.

I adapted providers.py to force FesteIP.Net to use ipv4 according to the 
information on the homepage.

A test showed no issue:

ddns -d update --force ipfire.feste-ip.net

Running on distribution: ipfire-2
Loading configuration file /var/ipfire/ddns/ddns.conf
Updating ipfire.feste-ip.net forced
Sending request (GET): 
https://v4.members.feste-ip.net/nic/update/?hostname=ipfire.feste-ip.net&myip=5.63.50.246
Request header:
   Authorization: Basic MjA2MDA0Ol84UnN2bjZidDk=
   User-agent: IPFireDDNSUpdater/013
   Pragma: no-cache
Response header (Status Code 200):
   Date: Sat, 08 May 2021 12:01:04 GMT
   Content-Type: text/plain
   Connection: close
   Accept-Ranges: none
   Server: Feste-IP.net Server
   Strict-Transport-Security: max-age=15768000
Dynamic DNS update for ipfire.feste-ip.net (Feste-IP.Net) successful
Logging successful update for ipfire.feste-ip.net
Opening database /var/lib/ddns.db

So, from my point of view the patch should do the job.

Regards,

Martin Krieger

diff --git a/README b/README
index b6decb3..fa6ce5e 100644
--- a/README
+++ b/README
@@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
         easydns.com
         enom.com
         entrydns.net
+       feste-ip.net
         freedns.afraid.org
         inwx.com|de|at|ch|es
         itsdns.de
diff --git a/src/ddns/providers.py b/src/ddns/providers.py
index 56e6620..8348ed1 100644
--- a/src/ddns/providers.py
+++ b/src/ddns/providers.py
@@ -1178,6 +1178,17 @@ class DDNSProviderEntryDNS(DDNSProvider):
                 # If we got here, some other update error happened.
                 raise DDNSUpdateError

+class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
+        handle    = "feste-ip.net"
+        name      = "Feste-IP.Net"
+        website   = "https://www.feste-ip.net/"
+        protocols = ("ipv4",)
+
+        # Information about the format of the request is to be found
+        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
+        # https://www.feste-ip.net/ddns-service/einrichtung/ipv4-ipv6/
+
+        url = "https://v4.members.feste-ip.net/nic/update"

  class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
         handle    = "freedns.afraid.org"

Michael Tremer:
> Hello Martin,
> 
>> On 6 May 2021, at 11:21, Martin Krieger <makrie(a)posteo.de> wrote:
>>
>> Hello Michael,
>>
>> by eMail again, because I didn't figure out to use git (commit, format-patch & send-email) in the correct way for patch submission. Sorry.
>>
>> Regards,
>>
>> Martin Krieger
>>
>> diff --git a/README b/README
>> index b6decb3..fa6ce5e 100644
>> --- a/README
>> +++ b/README
>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>         easydns.com
>>         enom.com
>>         entrydns.net
>> +       feste-ip.net
>>         freedns.afraid.org
>>         inwx.com|de|at|ch|es
>>         itsdns.de
>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>> index 56e6620..c53ff9a 100644
>> --- a/src/ddns/providers.py
>> +++ b/src/ddns/providers.py
>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>                 # If we got here, some other update error happened.
>>                 raise DDNSUpdateError
>>
>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>> +        handle    = "feste-ip.net"
>> +        name      = "Feste-IP.Net"
>> +        website   = "https://www.feste-ip.net/"
>> +        protocols = ("ipv4","ipv6")
> 
> According to the documentation, the request has to be sent to a different host for IPv6. Just adding it to the list probably won’t work because this provider has implemented a different way to use the DynDNS API.
> 
>> +        # Information about the format of the request is to be found
>> +        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
>> +
>> +        url = "https://members.feste-ip.net/nic/update"
>>
>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>         handle    = "freedns.afraid.org"
>>
>> Michael Tremer:
>>> Hello Martin,
>>> Thank you very much for your submission.
>>>> On 5 May 2021, at 19:13, Martin Krieger <makrie(a)posteo.de> wrote:
>>>>
>>>> diff --git a/README b/README
>>>> index b6decb3..fa6ce5e 100644
>>>> --- a/README
>>>> +++ b/README
>>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>>         easydns.com
>>>>         enom.com
>>>>         entrydns.net
>>>> +       feste-ip.net
>>>>         freedns.afraid.org
>>>>         inwx.com|de|at|ch|es
>>>>         itsdns.de
>>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>>> index 56e6620..c70423b 100644
>>>> --- a/src/ddns/providers.py
>>>> +++ b/src/ddns/providers.py
>>>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>>                 # If we got here, some other update error happened.
>>>>                 raise DDNSUpdateError
>>>>
>>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>>> +        handle    = "feste-ip.net"
>>>> +        name      = "Feste-IP.Net"
>>>> +        website   = "https//www.feste-ip.net/"
>>> You are missing a “:” in the URL.
>>>> +        protocols = ("ipv4",)
>>> According to this documentation, the provider supports IPv6 as well:
>>>    https://www.feste-ip.net/ddns-service/einrichtung/linux/
>>>> +        # Information about the format of the request is to be found
>>>> +        # https://www.feste-ip.net/ddns-service/allgemeine-informationen/
>>>> +
>>>> +        url = "https://members.feste-ip.net/nic/update"
>>>>
>>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>>         handle    = “freedns.afraid.org"
>>>>
>>> Best,
>>> -Michael
>>
> 


[-- Attachment #2: OpenPGP_signature.sig --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] ddns: Add provider Feste-IP.Net

[Michael Tremer]

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

Hello Martin,

> On 8 May 2021, at 13:20, Martin Krieger <makrie(a)posteo.de> wrote:
> 
> Hello Michael,
> 
> unfortunately my actual Python knowledge is very basic. I found no solution to implement ipv6 in addition to ipv4. Also IPFire is based on ipv4 and will be there in future as far as I understood.

Currently IPFire 2 only supports IPv4, but that is not mean to stay for forever and ddns is used elsewhere, too.

> I adapted providers.py to force FesteIP.Net to use ipv4 according to the information on the homepage.

I do not think that it is a good idea to implement half a provider. Could you maybe contact them and ask them to integrate their custom API into ddns?

-Michael

> 
> A test showed no issue:
> 
> ddns -d update --force ipfire.feste-ip.net
> 
> Running on distribution: ipfire-2
> Loading configuration file /var/ipfire/ddns/ddns.conf
> Updating ipfire.feste-ip.net forced
> Sending request (GET): https://v4.members.feste-ip.net/nic/update/?hostname=ipfire.feste-ip.net&myip=5.63.50.246
> Request header:
>  Authorization: Basic MjA2MDA0Ol84UnN2bjZidDk=
>  User-agent: IPFireDDNSUpdater/013
>  Pragma: no-cache
> Response header (Status Code 200):
>  Date: Sat, 08 May 2021 12:01:04 GMT
>  Content-Type: text/plain
>  Connection: close
>  Accept-Ranges: none
>  Server: Feste-IP.net Server
>  Strict-Transport-Security: max-age=15768000
> Dynamic DNS update for ipfire.feste-ip.net (Feste-IP.Net) successful
> Logging successful update for ipfire.feste-ip.net
> Opening database /var/lib/ddns.db
> 
> So, from my point of view the patch should do the job.
> 
> Regards,
> 
> Martin Krieger
> 
> diff --git a/README b/README
> index b6decb3..fa6ce5e 100644
> --- a/README
> +++ b/README
> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>        easydns.com
>        enom.com
>        entrydns.net
> +       feste-ip.net
>        freedns.afraid.org
>        inwx.com|de|at|ch|es
>        itsdns.de
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index 56e6620..8348ed1 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1178,6 +1178,17 @@ class DDNSProviderEntryDNS(DDNSProvider):
>                # If we got here, some other update error happened.
>                raise DDNSUpdateError
> 
> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    = "feste-ip.net"
> +        name      = "Feste-IP.Net"
> +        website   = "https://www.feste-ip.net/"
> +        protocols = ("ipv4",)
> +
> +        # Information about the format of the request is to be found
> +        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
> +        # https://www.feste-ip.net/ddns-service/einrichtung/ipv4-ipv6/
> +
> +        url = "https://v4.members.feste-ip.net/nic/update"
> 
> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>        handle    = "freedns.afraid.org"
> 
> Michael Tremer:
>> Hello Martin,
>>> On 6 May 2021, at 11:21, Martin Krieger <makrie(a)posteo.de> wrote:
>>> 
>>> Hello Michael,
>>> 
>>> by eMail again, because I didn't figure out to use git (commit, format-patch & send-email) in the correct way for patch submission. Sorry.
>>> 
>>> Regards,
>>> 
>>> Martin Krieger
>>> 
>>> diff --git a/README b/README
>>> index b6decb3..fa6ce5e 100644
>>> --- a/README
>>> +++ b/README
>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>        easydns.com
>>>        enom.com
>>>        entrydns.net
>>> +       feste-ip.net
>>>        freedns.afraid.org
>>>        inwx.com|de|at|ch|es
>>>        itsdns.de
>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>> index 56e6620..c53ff9a 100644
>>> --- a/src/ddns/providers.py
>>> +++ b/src/ddns/providers.py
>>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>                # If we got here, some other update error happened.
>>>                raise DDNSUpdateError
>>> 
>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>> +        handle    = "feste-ip.net"
>>> +        name      = "Feste-IP.Net"
>>> +        website   = "https://www.feste-ip.net/"
>>> +        protocols = ("ipv4","ipv6")
>> According to the documentation, the request has to be sent to a different host for IPv6. Just adding it to the list probably won’t work because this provider has implemented a different way to use the DynDNS API.
>>> +        # Information about the format of the request is to be found
>>> +        # https://www.feste-ip.net/ddns-service/einrichtung/linux/
>>> +
>>> +        url = "https://members.feste-ip.net/nic/update"
>>> 
>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>        handle    = "freedns.afraid.org"
>>> 
>>> Michael Tremer:
>>>> Hello Martin,
>>>> Thank you very much for your submission.
>>>>> On 5 May 2021, at 19:13, Martin Krieger <makrie(a)posteo.de> wrote:
>>>>> 
>>>>> diff --git a/README b/README
>>>>> index b6decb3..fa6ce5e 100644
>>>>> --- a/README
>>>>> +++ b/README
>>>>> @@ -68,6 +68,7 @@ SUPPORTED PROVIDERS:
>>>>>        easydns.com
>>>>>        enom.com
>>>>>        entrydns.net
>>>>> +       feste-ip.net
>>>>>        freedns.afraid.org
>>>>>        inwx.com|de|at|ch|es
>>>>>        itsdns.de
>>>>> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
>>>>> index 56e6620..c70423b 100644
>>>>> --- a/src/ddns/providers.py
>>>>> +++ b/src/ddns/providers.py
>>>>> @@ -1178,6 +1178,16 @@ class DDNSProviderEntryDNS(DDNSProvider):
>>>>>                # If we got here, some other update error happened.
>>>>>                raise DDNSUpdateError
>>>>> 
>>>>> +class DDNSProviderFesteIPNet(DDNSProtocolDynDNS2, DDNSProvider):
>>>>> +        handle    = "feste-ip.net"
>>>>> +        name      = "Feste-IP.Net"
>>>>> +        website   = "https//www.feste-ip.net/"
>>>> You are missing a “:” in the URL.
>>>>> +        protocols = ("ipv4",)
>>>> According to this documentation, the provider supports IPv6 as well:
>>>>   https://www.feste-ip.net/ddns-service/einrichtung/linux/
>>>>> +        # Information about the format of the request is to be found
>>>>> +        # https://www.feste-ip.net/ddns-service/allgemeine-informationen/
>>>>> +
>>>>> +        url = "https://members.feste-ip.net/nic/update"
>>>>> 
>>>>> class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>>>>>        handle    = “freedns.afraid.org"
>>>>> 
>>>> Best,
>>>> -Michael
>>> 
>