Skip to content

fix(ndcli): accept layer3domain for show rr / modify rr - #328

Open
miesi wants to merge 1 commit into
IONOS-Core:masterfrom
miesi:fix/modify-rr-layer3domain
Open

miesi wants to merge 1 commit into
IONOS-Core:masterfrom
miesi:fix/modify-rr-layer3domain

Conversation

@miesi

@miesi miesi commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

"modify rr NAME a IP" and "show rr NAME a IP" had no layer3domain argument and ignored NDCLI_LAYER3DOMAIN / layer3domain= in ~/.ndclirc. When IP exists in more than one layer3domain the server cannot pick one and answered "A layer3domain is needed" with no way for the user to supply it.

They now take "layer3domain NAME" and the configured default, exactly like "create rr" and "delete rr". The server drops layer3domain after using it to resolve the ipblock, as rr_delete() already does, since it is not part of the record value.

ITOUDP-8041

"modify rr NAME a IP" and "show rr NAME a IP" had no layer3domain
argument and ignored NDCLI_LAYER3DOMAIN / layer3domain= in ~/.ndclirc.
When IP exists in more than one layer3domain the server cannot pick
one and answered "A layer3domain is needed" with no way for the user
to supply it.

They now take "layer3domain NAME" and the configured default, exactly
like "create rr" and "delete rr". The server drops layer3domain after
using it to resolve the ipblock, as rr_delete() already does, since it
is not part of the record value.

ITOUDP-8041

Co-Authored-By: Claude Opus 5.5 <[email protected]>
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