[PR #623] [MERGED] GoIP.de support #768

New Issue

saavagebueno · 2025-11-20T04:25:59-05:00

saavagebueno commented

2025-11-20 04:25:59 -05:00

📋 Pull Request Information

Original PR: https://github.com/qdm12/ddns-updater/pull/623
Author: @CyberAustin
Created: 2/2/2024
Status: ✅ Merged
Merged: 2/3/2024
Merged by: @qdm12

Base: master ← Head: goip.de

📝 Commits (10+)

5d4f3b9 Have GoIP.de mostly working, just need to finish implementing error messages.
9ed3041 Ready to merge, or at least so I think
431c31e Fixed or adjusted most comments. Ready for another review.
99e6602 Fixed some linting errors and changed subdomain to FQDN to make more sense
8f3a58e Have GoIP.de mostly working, just need to finish implementing error messages.
eacc306 Ready to merge, or at least so I think
e627729 Fixed or adjusted most comments. Ready for another review.
2b32f01 Fixed some linting errors and changed subdomain to FQDN to make more sense
f693740 Update goip.md
6a33a96 Update goip.md

📊 Changes

6 files changed (+215 additions, -0 deletions)

View changed files

📝 README.md (+2 -0)
➕ docs/goip.md (+36 -0)
📝 internal/provider/constants/providers.go (+2 -0)
📝 internal/provider/errors/validation.go (+1 -0)
📝 internal/provider/provider.go (+3 -0)
➕ internal/provider/providers/goip/provider.go (+171 -0)

📄 Description

Resolves #581

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/qdm12/ddns-updater/pull/623 **Author:** [@CyberAustin](https://github.com/CyberAustin) **Created:** 2/2/2024 **Status:** ✅ Merged **Merged:** 2/3/2024 **Merged by:** [@qdm12](https://github.com/qdm12) **Base:** `master` ← **Head:** `goip.de` --- ### 📝 Commits (10+) - [`5d4f3b9`](https://github.com/qdm12/ddns-updater/commit/5d4f3b9af484c429dfee5de7ab6d69adacb8dfed) Have GoIP.de mostly working, just need to finish implementing error messages. - [`9ed3041`](https://github.com/qdm12/ddns-updater/commit/9ed3041e434f5111abe81cf2619cd9ae4479d511) Ready to merge, or at least so I think - [`431c31e`](https://github.com/qdm12/ddns-updater/commit/431c31ee4cd4462db34e4fc4f911a523f4135d61) Fixed or adjusted most comments. Ready for another review. - [`99e6602`](https://github.com/qdm12/ddns-updater/commit/99e660292effba9543a34c986e58bdd88f0a204b) Fixed some linting errors and changed subdomain to FQDN to make more sense - [`8f3a58e`](https://github.com/qdm12/ddns-updater/commit/8f3a58eaa77697067be756a09915bc1c533b475f) Have GoIP.de mostly working, just need to finish implementing error messages. - [`eacc306`](https://github.com/qdm12/ddns-updater/commit/eacc306e94d489abb7691d2e65c84d72cb40ae92) Ready to merge, or at least so I think - [`e627729`](https://github.com/qdm12/ddns-updater/commit/e627729092fde9fe77f8cee35b62332eda556960) Fixed or adjusted most comments. Ready for another review. - [`2b32f01`](https://github.com/qdm12/ddns-updater/commit/2b32f01f6204b5b802b8bb5e984784b6c6772476) Fixed some linting errors and changed subdomain to FQDN to make more sense - [`f693740`](https://github.com/qdm12/ddns-updater/commit/f6937403a94532612872cabb7640daaee0c0c550) Update goip.md - [`6a33a96`](https://github.com/qdm12/ddns-updater/commit/6a33a96f5627827786087f0d9bec2a61af52ab9f) Update goip.md ### 📊 Changes **6 files changed** (+215 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+2 -0) ➕ `docs/goip.md` (+36 -0) 📝 `internal/provider/constants/providers.go` (+2 -0) 📝 `internal/provider/errors/validation.go` (+1 -0) 📝 `internal/provider/provider.go` (+3 -0) ➕ `internal/provider/providers/goip/provider.go` (+171 -0) </details> ### 📄 Description Resolves #581 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

saavagebueno added the pull-request label 2025-11-20 04:25:59 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/ddns-updater#768