[PR #921] Add TransIP provider #900

New Issue

saavagebueno · 2025-11-20T04:26:28-05:00

saavagebueno commented

2025-11-20 04:26:28 -05:00

📋 Pull Request Information

Original PR: https://github.com/qdm12/ddns-updater/pull/921
Author: @LucaScorpion
Created: 1/25/2025
Status: 🔄 Open

Base: master ← Head: transip

📝 Commits (8)

1dbbbb2 Start work on transip provider, wip update func
e0d7857 Fix lookupIPsResilient
59ee28c Read private key, create access token
52663a4 Set user agent
7f780f5 Fix duplicate token labels, working update call
8d2a970 Working create or update logic
66e4838 Add docs
cd592e4 Use encoding/pem package instead of custom string manipulation

📊 Changes

10 files changed (+331 additions, -1 deletions)

View changed files

📝 .gitignore (+2 -1)
📝 README.md (+2 -0)
➕ docs/transip.md (+42 -0)
📝 internal/provider/constants/providers.go (+2 -0)
📝 internal/provider/provider.go (+3 -0)
➕ internal/provider/providers/transip/provider.go (+279 -0)
📝 internal/update/service.go (+1 -0)
➕ readme/transip1.png (+0 -0)
➕ readme/transip2.png (+0 -0)
➕ readme/transip3.png (+0 -0)

📄 Description

Adds support for TransIP, resolves #254

Tested locally, and seemed to work like a charm! Ipv6 and wildcards are both supported, and it will either create a new entry, or update an existing one.

The relevant API documentation can be found here: https://api.transip.nl/rest/docs.html#domains-dns

Side note: this PR also includes a small fix from https://github.com/qdm12/ddns-updater/pull/918. It shouldn't conflict, but if that does happen I'll fix that after that one is merged :)

Let me know if any changes are needed, and thanks in advance!

_{🔄 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/921 **Author:** [@LucaScorpion](https://github.com/LucaScorpion) **Created:** 1/25/2025 **Status:** 🔄 Open **Base:** `master` ← **Head:** `transip` --- ### 📝 Commits (8) - [`1dbbbb2`](https://github.com/qdm12/ddns-updater/commit/1dbbbb231155466a97a39af9f22a6c0b250f03ee) Start work on transip provider, wip update func - [`e0d7857`](https://github.com/qdm12/ddns-updater/commit/e0d78578e932c76a2c7abb4af8aad3daab388352) Fix lookupIPsResilient - [`59ee28c`](https://github.com/qdm12/ddns-updater/commit/59ee28c6cee662d31f40c745366574e4633bbd29) Read private key, create access token - [`52663a4`](https://github.com/qdm12/ddns-updater/commit/52663a43ef4ad3f8516036c87839c4aadfc4a74c) Set user agent - [`7f780f5`](https://github.com/qdm12/ddns-updater/commit/7f780f5ab7d14e989354ba08679213aa6bfb3e65) Fix duplicate token labels, working update call - [`8d2a970`](https://github.com/qdm12/ddns-updater/commit/8d2a9709fc6d63f9364d58feb0535f7bc28fda47) Working create or update logic - [`66e4838`](https://github.com/qdm12/ddns-updater/commit/66e4838c45d729ffbab06f0bc323a46356fd9e49) Add docs - [`cd592e4`](https://github.com/qdm12/ddns-updater/commit/cd592e47d4a3b69700b3b0cc12783572fcb7017c) Use encoding/pem package instead of custom string manipulation ### 📊 Changes **10 files changed** (+331 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+2 -1) 📝 `README.md` (+2 -0) ➕ `docs/transip.md` (+42 -0) 📝 `internal/provider/constants/providers.go` (+2 -0) 📝 `internal/provider/provider.go` (+3 -0) ➕ `internal/provider/providers/transip/provider.go` (+279 -0) 📝 `internal/update/service.go` (+1 -0) ➕ `readme/transip1.png` (+0 -0) ➕ `readme/transip2.png` (+0 -0) ➕ `readme/transip3.png` (+0 -0) </details> ### 📄 Description Adds support for [TransIP](https://www.transip.nl/), resolves #254 Tested locally, and seemed to work like a charm! Ipv6 and wildcards are both supported, and it will either create a new entry, or update an existing one. The relevant API documentation can be found here: https://api.transip.nl/rest/docs.html#domains-dns Side note: this PR also includes a small fix from https://github.com/qdm12/ddns-updater/pull/918. It shouldn't conflict, but if that does happen I'll fix that after that one is merged :) Let me know if any changes are needed, and thanks in advance! --- <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:26:29 -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#900