Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DietPi-DDNS | Update FreeDNS API URL #6375

Merged
merged 3 commits into from
May 18, 2023
Merged

DietPi-DDNS | Update FreeDNS API URL #6375

merged 3 commits into from
May 18, 2023

Conversation

ma651851384
Copy link
Contributor

Recently, I tried to set up DynDNS via the substantiated provider "FreeDNS". After acceptance of the setting using the token, the error 404 was displayed. When I used the custom URL, it worked. That URL was provided, if the user was logged in into the freedns user portal.

I believe that in the meantime the URL has changed from "https://sync.afraid.org/u/" to "http://freedns.afraid.org/dynamic/update.php?".

The effected line would have to be changed from

url="https://sync.afraid.org/u/$PASSWORD/"
to
url="https://freedns.afraid.org/dynamic/update.php?$PASSWORD"

It seems that the provider has changed the update URL, which has to be corrected for that predefined FreeDNS menu item.

With the previous implementation the dietpi system showed the error 404.
@MichaIng
Copy link
Owner

Strange, here it still shows the URL we use: https://freedns.afraid.org/

There is a link to "v2 interface", but it is hidden behind login. Is it probably that one which matches your new URL? Because the old one is still shown on this public place, I wonder whether it depends on some registration plan or option?

@MichaIng MichaIng added this to the v8.18 milestone May 13, 2023
@MichaIng MichaIng self-requested a review May 13, 2023 13:38
@ma651851384
Copy link
Contributor Author

Hello @MichaIng
I've registered myself for testing purpose today and set up a subdomain. After that, I could retrieve the direct URL. It corresponds to the scheme as I have reported on this issue:

https://freedns.afraid.org/dynamic/update.php?<token>

Find attached some relevant screenshot from the provider user account.

Your mentioned "v2 interface" seems to be only a web interface update.

If there are still questions, I can view the necessary data by my test access by 07.06.2023 at the latest.

2023-05-14_01
2023-05-14_02
2023-05-14_03
2023-05-14_04
2023-05-14_05

@MichaIng
Copy link
Owner

Okay, seems to be right, let's apply this change then. Aside of changelog I'll try to get some other FreeDNS users testing this during beta phase.

If there is a good support channel, probably you could ask why the API URL from their homepage differs from what one gets from the user control panel?

@MichaIng MichaIng changed the title DietPi-Software | DDNS: Update dietpi-ddns regarding provider FreeDNS DietPi-DDNS | Update FreeDNS API URL May 18, 2023
- CHANGELOG | DietPi-DDNS: Resolved an issue where the IP sync failed because the API URL changed recently. Many thanks to @ma651851384 for implementing the update: #6375
@MichaIng MichaIng merged commit 3398a0e into MichaIng:dev May 18, 2023
@ma651851384
Copy link
Contributor Author

Thank you. Also from my point of view, it would be good if other users would test my reported URL structure.

I wrote an email to the FreeDNS support a few minutes ago, and I'll get back to you when I got the answer.

@ma651851384 ma651851384 deleted the patch-2 branch May 18, 2023 17:55
disconn3ct added a commit to disconn3ct/DietPi that referenced this pull request May 26, 2023
* dev:
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18
  v8.18 (MichaIng#6375)
  v8.18 (MichaIng#6380)
  v8.18
  v8.18
  v8.18 (MichaIng#6378)
@ma651851384
Copy link
Contributor Author

Hi.
I tested the new implementation (inserting only the token in the input field) with dietpi V8.18.2. It works fine, many thanks!

In one of the previouse posts I told that I contacted the FreeDNS support. Unfortunately, no one ever answered.

This topic is closed/solved from my side.

@MichaIng MichaIng added the Solution available 🥂 Definite solution has been done label Jun 18, 2023
@MichaIng
Copy link
Owner

Many thanks for testing and feedback 🙂.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Solution available 🥂 Definite solution has been done Upstream change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants