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

pdnsutil replace-rrset should be improved #11128

Open
tuxis-ie opened this issue Dec 23, 2021 · 3 comments
Open

pdnsutil replace-rrset should be improved #11128

tuxis-ie opened this issue Dec 23, 2021 · 3 comments
Labels

Comments

@tuxis-ie
Copy link

While trying to update the NS rrset for a zone, when using logical parameters on the commandline, you get unexpected results.

e.g.:

# pdnsutil list-zone prevented.nl | grep NS
prevented.nl	3600	IN	NS	nssecauth1.tuxis.net.
prevented.nl	3600	IN	NS	nssecauth2.tuxis.net.

I expect the following to work:

pdnsutil replace-rrset prevented.nl prevented.nl NS ns1.allcloud.nl ns2.allcloud.nl

But that results in:

# pdnsutil list-zone prevented.nl | grep NS
prevented.nl	3600	IN	NS	nssecauth1.tuxis.net.
prevented.nl	3600	IN	NS	nssecauth2.tuxis.net.
prevented.nl.prevented.nl	3600	IN	NS	ns1.allcloud.nl.
prevented.nl.prevented.nl	3600	IN	NS	ns2.allcloud.nl.

Adding a . to the NAME does not help, that creates NS records or prevented.nl.prevented.nl. . Using . as NAME does work, as does @.

While how this works isn't actually incorrect, it is kind of confusing and could do with some improvement.

@Habbie Habbie added the auth label Dec 23, 2021
@Habbie Habbie added this to the auth-4.7.0 milestone Dec 23, 2021
@Habbie Habbie modified the milestones: auth-4.7.0, auth-4.8.0 Sep 8, 2022
@Habbie Habbie modified the milestones: auth-4.8.0, auth-4.9.0 Apr 14, 2023
@jpmens
Copy link
Contributor

jpmens commented Jul 21, 2023

IMO this isn't confusing but wrong, particularly as the same utility with the edit-zone subcommand clearly indicates that all names therein are ABSOLUTE! (emphasis not mine).

related: #8595

@Habbie
Copy link
Member

Habbie commented Jan 23, 2024

related, perhaps even duplicate: #13420

@Habbie Habbie modified the milestones: auth-4.9.0, auth-5, auth-helpneeded Jan 23, 2024
@jpmens
Copy link
Contributor

jpmens commented Jan 23, 2024

Looks like a duplicate to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants