[PR #837] [MERGED] Update porkbun api to new url #865

New Issue

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

saavagebueno commented

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

📋 Pull Request Information

Original PR: https://github.com/qdm12/ddns-updater/pull/837
Author: @likeaninja5
Created: 10/11/2024
Status: ✅ Merged
Merged: 10/12/2024
Merged by: @qdm12

Base: master ← Head: master

📝 Commits (1)

d5e4a87 Update porkbun api to new url

📊 Changes

1 file changed (+4 additions, -4 deletions)

View changed files

📝 internal/provider/providers/porkbun/api.go (+4 -4)

📄 Description

Porkbun sent out an email notifying users that the api url will change starting 2024-12-01.

https://porkbun.com/api/json/v3/documentation:

API Hostname

api.porkbun.com

Please note that porkbun.com currently works and has historically been the correct hostname for our API. However, we will be migrating away from that hostname and the API will no function using it at some point in the future.

Doesn't look like you have any provider tests, but I tested locally with docker and it worked.

_{🔄 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/837 **Author:** [@likeaninja5](https://github.com/likeaninja5) **Created:** 10/11/2024 **Status:** ✅ Merged **Merged:** 10/12/2024 **Merged by:** [@qdm12](https://github.com/qdm12) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`d5e4a87`](https://github.com/qdm12/ddns-updater/commit/d5e4a87b55aaeb476dc3c96814293a1abe6cf3f6) Update porkbun api to new url ### 📊 Changes **1 file changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `internal/provider/providers/porkbun/api.go` (+4 -4) </details> ### 📄 Description Porkbun sent out an email notifying users that the api url will change starting 2024-12-01. https://porkbun.com/api/json/v3/documentation: ``` API Hostname api.porkbun.com Please note that porkbun.com currently works and has historically been the correct hostname for our API. However, we will be migrating away from that hostname and the API will no function using it at some point in the future. ``` Doesn't look like you have any provider tests, but I tested locally with docker and it worked. --- <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:21 -05:00

saavagebueno closed this issue

2025-11-20 04:26:21 -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#865