Skip to content

Conversation

@obbardc
Copy link

@obbardc obbardc commented Jan 31, 2026

📦 Package Details

Maintainer: @feckert
(You can find this by checking the history of the package Makefile.)

Description:
Add a new Hetzner provider using the Hetzner Cloud API (api.hetzner.cloud) with Bearer token authentication.


🧪 Run Testing Details

  • OpenWrt Version: OpenWrt 24.10.5 r29087-d9c5716d1d / LuCI openwrt-24.10 branch 25.340.26705~d88390b
  • OpenWrt Target/Subtarget: mediatek/filogic
  • OpenWrt Device: GL.iNet GL-MT6000

Seems to work; I copied it to the device like this:

scp -O net/ddns-scripts/files/usr/lib/ddns/update_hetzner_cloud.sh     [email protected]:/usr/lib/ddns/
scp -O net/ddns-scripts/files/usr/share/ddns/default/hetzner.com.json   [email protected]:/usr/share/ddns/default/

✅ Formalities

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

@obbardc obbardc force-pushed the wip/obbardc/net-ddns-hetzner branch 2 times, most recently from c8ef110 to 048cbc0 Compare January 31, 2026 04:43
@obbardc obbardc marked this pull request as ready for review January 31, 2026 04:51
@feckert
Copy link
Member

feckert commented Jan 31, 2026

LGTM please update the PKG_RELEASE in the ddns-scripts makefile by one and then we are ready to merge 👍

@obbardc obbardc force-pushed the wip/obbardc/net-ddns-hetzner branch from 048cbc0 to 91f47ab Compare February 1, 2026 01:04
@obbardc
Copy link
Author

obbardc commented Feb 1, 2026

LGTM please update the PKG_RELEASE in the ddns-scripts makefile by one and then we are ready to merge 👍

Done; I did that in a separate commit. Let me know if you want me to squash it or something else

@GeorgeSapkin
Copy link
Member

@obbardc you need to bump PKG_RELEASE, not PKG_VERSION. And please squash your commits.

@obbardc obbardc force-pushed the wip/obbardc/net-ddns-hetzner branch from 91f47ab to 9dcaf49 Compare February 1, 2026 01:36
@obbardc
Copy link
Author

obbardc commented Feb 1, 2026

@obbardc you need to bump PKG_RELEASE, not PKG_VERSION. And please squash your commits.

done!

@obbardc obbardc force-pushed the wip/obbardc/net-ddns-hetzner branch from 9dcaf49 to 7773933 Compare February 1, 2026 01:38
@GeorgeSapkin
Copy link
Member

And please start the first word after prefix with lower case.

Add a new Hetzner DDNS provider using the Hetzner Cloud API
(api.hetzner.cloud) with Bearer token authentication.

Configuration guide:
* set [domain] to domain
* set [username] to subdomain (without domain)
* set [password] to Bearer API key

Signed-off-by: Christopher Obbard <[email protected]>
@obbardc obbardc force-pushed the wip/obbardc/net-ddns-hetzner branch from 7773933 to 6161b72 Compare February 1, 2026 02:22
@obbardc
Copy link
Author

obbardc commented Feb 1, 2026

And please start the first word after prefix with lower case.

think that's a very petty nitpick; but all done now :-)

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.

3 participants