Skip to content

Conversation

@jeffrego
Copy link
Contributor

@jeffrego jeffrego commented Aug 2, 2024

Add support for Directnic (https://directnic.com).

Documentation of the API is available at https://directnic.com/knowledge#/knowledge/article/3726

Configuration snippet for this provider:

protocol=directnic \
password=ad133743f001e318e455fdc04 \
my-domain.com

password is the secret part of the update URL, e.g. for the example from their docs https://directnic.com/dns/gateway/ad133743f001e318e455fdc04/?data= you would set password=ad133743f001e318e455fdc04

Directnic assigns a unique token to every dynamic DNS entry, so if you have multiple records (like an A record and a AAAA record) for the same domain, each will get its own token and will need to be managed separately.

@jeffrego jeffrego force-pushed the feature_provider_directnic branch from 701a9e5 to bf21d85 Compare August 2, 2024 03:54
Copy link
Member

@rhansen rhansen left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution!

@rhansen
Copy link
Member

rhansen commented Aug 3, 2024

If you want to add tests (not required, but appreciated), you can use t/protocol_dyndns2.pl for inspiration.

@jeffrego jeffrego force-pushed the feature_provider_directnic branch from bf21d85 to 0c3113c Compare August 5, 2024 00:31
Copy link
Member

@rhansen rhansen left a comment

Choose a reason for hiding this comment

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

Thanks for adding tests!

@jeffrego jeffrego force-pushed the feature_provider_directnic branch from 044b36d to bc2b477 Compare August 6, 2024 02:42
Copy link
Member

@rhansen rhansen left a comment

Choose a reason for hiding this comment

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

Looking good!

Would you mind rebasing the commits into two commits:

  • 1st commit: add T_URL type and tests
  • 2nd commit: new directnic protocol

@jeffrego jeffrego force-pushed the feature_provider_directnic branch 2 times, most recently from ff6321c to c858dd9 Compare August 7, 2024 02:36
Copy link
Member

@rhansen rhansen left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you!

@rhansen rhansen force-pushed the feature_provider_directnic branch from c858dd9 to 959b5dd Compare August 7, 2024 04:06
@rhansen rhansen merged commit 598dee5 into ddclient:master Aug 7, 2024
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.

2 participants