Skip to content

HETZNER_V2: support init command#4290

Merged
TomOnTime merged 3 commits into
mainfrom
hetznerv2-register-creds-metadata
May 19, 2026
Merged

HETZNER_V2: support init command#4290
TomOnTime merged 3 commits into
mainfrom
hetznerv2-register-creds-metadata

Conversation

@TomOnTime
Copy link
Copy Markdown
Collaborator

Summary

Register CredsMetadata for HETZNER_V2 so the provider is offered by the dnscontrol init wizard.

Fields mirror the entries in integrationTest/profiles.json.

CC: @das7pad

Test plan

  • go build ./... passes
  • dnscontrol init lists HETZNER_V2 as an option
  • Verify any // TODO: Verify annotations in the diff (e.g. PortalURL)

🤖 Generated with Claude Code

Register CredsMetadata so the HETZNER_V2 provider appears in the `dnscontrol init` wizard.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@das7pad das7pad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

The below is an attempt at adopting the naming from their docs. cc @jooola

Comment thread providers/hetznerv2/hetznerv2Provider.go Outdated
DisplayName: "Hetzner DNS Console (v2)",
Kind: providers.KindDNS,
DocsURL: "https://docs.dnscontrol.org/provider/hetzner_v2",
PortalURL: "https://dns.hetzner.com/settings/api-token", // TODO: Verify
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
PortalURL: "https://dns.hetzner.com/settings/api-token", // TODO: Verify
PortalURL: "https://console.hetzner.com/projects",

The v2 dns offering does not have a deep link for configuring an API token. It is a tab of the security view of each project inside the "Hetzner console".

Docs for auth: https://docs.hetzner.cloud/reference/cloud#description/authentication

Comment thread providers/hetznerv2/hetznerv2Provider.go Outdated
Copy link
Copy Markdown
Contributor

@jooola jooola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from my end.

TomOnTime and others added 2 commits May 19, 2026 10:02
Co-authored-by: Jakob Ackermann <das7pad@outlook.com>
Co-authored-by: Jakob Ackermann <das7pad@outlook.com>
@TomOnTime TomOnTime merged commit c8b185d into main May 19, 2026
@TomOnTime TomOnTime deleted the hetznerv2-register-creds-metadata branch May 19, 2026 14:03
@TomOnTime
Copy link
Copy Markdown
Collaborator Author

Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants