Skip to content

Comments

ddns-scripts: change servercow.de to new dns api#28549

Open
jummo wants to merge 1 commit intoopenwrt:masterfrom
jummo:ddns_servercow
Open

ddns-scripts: change servercow.de to new dns api#28549
jummo wants to merge 1 commit intoopenwrt:masterfrom
jummo:ddns_servercow

Conversation

@jummo
Copy link
Contributor

@jummo jummo commented Feb 10, 2026

📦 Package Details

Maintainer: @jummo

Description:
Change servercow.de to use new dns api, see https://wiki.servercow.de/de/domains/dns_api/api-syntax/.
Sorry for creating another Pull Request, but I still couldn't figure out how to do it properly.


🧪 Run Testing Details

  • OpenWrt Version: r29087-d9c5716d1d
  • OpenWrt Target/Subtarget: x86_64
  • OpenWrt Device: KVM

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

@feckert
Copy link
Member

feckert commented Feb 11, 2026

If this can no longer be handled via the generic ddns-scripts packages, but requires a separate script, then a separate package has been created now. Please take another look at this.

$(eval $(call BuildPackage,ddns-scripts-luadns))
$(eval $(call BuildPackage,ddns-scripts-cloudflare))
$(eval $(call BuildPackage,ddns-scripts-gcp))
$(eval $(call BuildPackage,ddns-scripts-freedns))
$(eval $(call BuildPackage,ddns-scripts-godaddy))
$(eval $(call BuildPackage,ddns-scripts-hetzner-cloud))
$(eval $(call BuildPackage,ddns-scripts-namesilo))
$(eval $(call BuildPackage,ddns-scripts-digitalocean))
$(eval $(call BuildPackage,ddns-scripts-dnspod))
$(eval $(call BuildPackage,ddns-scripts-dnspod-v3))
$(eval $(call BuildPackage,ddns-scripts-noip))
$(eval $(call BuildPackage,ddns-scripts-nsupdate))
$(eval $(call BuildPackage,ddns-scripts-route53))
$(eval $(call BuildPackage,ddns-scripts-cnkuai))
$(eval $(call BuildPackage,ddns-scripts-gandi))
$(eval $(call BuildPackage,ddns-scripts-beget))
$(eval $(call BuildPackage,ddns-scripts-pdns))
$(eval $(call BuildPackage,ddns-scripts-scaleway))
$(eval $(call BuildPackage,ddns-scripts-transip))
$(eval $(call BuildPackage,ddns-scripts-ns1))
$(eval $(call BuildPackage,ddns-scripts-one))
$(eval $(call BuildPackage,ddns-scripts-porkbun))
$(eval $(call BuildPackage,ddns-scripts-huaweicloud))
$(eval $(call BuildPackage,ddns-scripts-aliyun))

@jummo
Copy link
Contributor Author

jummo commented Feb 13, 2026

If this can no longer be handled via the generic ddns-scripts packages, but requires a separate script, then a separate package has been created now. Please take another look at this.

$(eval $(call BuildPackage,ddns-scripts-luadns))
$(eval $(call BuildPackage,ddns-scripts-cloudflare))
$(eval $(call BuildPackage,ddns-scripts-gcp))
$(eval $(call BuildPackage,ddns-scripts-freedns))
$(eval $(call BuildPackage,ddns-scripts-godaddy))
$(eval $(call BuildPackage,ddns-scripts-hetzner-cloud))
$(eval $(call BuildPackage,ddns-scripts-namesilo))
$(eval $(call BuildPackage,ddns-scripts-digitalocean))
$(eval $(call BuildPackage,ddns-scripts-dnspod))
$(eval $(call BuildPackage,ddns-scripts-dnspod-v3))
$(eval $(call BuildPackage,ddns-scripts-noip))
$(eval $(call BuildPackage,ddns-scripts-nsupdate))
$(eval $(call BuildPackage,ddns-scripts-route53))
$(eval $(call BuildPackage,ddns-scripts-cnkuai))
$(eval $(call BuildPackage,ddns-scripts-gandi))
$(eval $(call BuildPackage,ddns-scripts-beget))
$(eval $(call BuildPackage,ddns-scripts-pdns))
$(eval $(call BuildPackage,ddns-scripts-scaleway))
$(eval $(call BuildPackage,ddns-scripts-transip))
$(eval $(call BuildPackage,ddns-scripts-ns1))
$(eval $(call BuildPackage,ddns-scripts-one))
$(eval $(call BuildPackage,ddns-scripts-porkbun))
$(eval $(call BuildPackage,ddns-scripts-huaweicloud))
$(eval $(call BuildPackage,ddns-scripts-aliyun))

Thanks Florian, I have adjusted Makefile to include Servercow script.

@jummo jummo requested a review from feckert February 16, 2026 19:36
@feckert
Copy link
Member

feckert commented Feb 17, 2026

Please also bump again the PKG_RELEASE

@jummo
Copy link
Contributor Author

jummo commented Feb 17, 2026

Please also bump again the PKG_RELEASE

Increase by 1

@feckert
Copy link
Member

feckert commented Feb 18, 2026

Unfortunately, the commit description in the body of the commit is not correct.
Please take a look at other commits in this repository.

  • The Sigend-off-by line is always the last line
  • Add why this is being done and why is the generic not correct anymore
  • Add more information about the API documentation from this provider (Web Link)

The previous way to update dns records at servercow.de was replaced
with a https based dns api. The new dns api is documented at
https://wiki.servercow.de/de/domains/dns_api/api-syntax/#einfuhrung

Signed-off-by: Patrick Hemmen <jummo4@yahoo.de>
@jummo
Copy link
Contributor Author

jummo commented Feb 20, 2026

Unfortunately, the commit description in the body of the commit is not correct. Please take a look at other commits in this repository.

* The Sigend-off-by line is always the last line

* Add why this is being done and why is the generic not correct anymore

* Add more information about the API documentation from this provider (Web Link)

Adjusted the commit message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants