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

dns: fix dnsrecord_add interactive mode #505

Closed
wants to merge 1 commit into from
Closed

dns: fix dnsrecord_add interactive mode #505

wants to merge 1 commit into from

Conversation

HonzaCholasta
Copy link
Contributor

@HonzaCholasta HonzaCholasta commented Feb 24, 2017

dnsrecord_add interactive mode might prompt for value of non-existent
arguments a_part_create_reverse and aaaa_part_create_reverse. This
happens because dnsrecord_add extra flags are incorrectly defined as
parts of the respective DNS records.

Remove extra flags from DNS record parts to fix the interactive mode on old
clients talking to new servers. Skip non-existent arguments in the
interactive mode to fix new clients talking to old servers.

https://fedorahosted.org/freeipa/ticket/6457

`dnsrecord_add` interactive mode might prompt for value of non-existent
arguments `a_part_create_reverse` and `aaaa_part_create_reverse`. This
happens because `dnsrecord_add` extra flags are incorrectly defined as
parts of the respective DNS records.

Remove extra flags from DNS record parts to fix the interactive mode on old
clients talking to new servers. Skip non-existent arguments in the
interactive mode to fix new clients talking to old servers.

https://fedorahosted.org/freeipa/ticket/6457
@MartinBasti MartinBasti self-assigned this Feb 24, 2017
@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Mar 8, 2017
@ghost
Copy link

ghost commented Mar 8, 2017

master:

  • 1e912f5 dns: fix dnsrecord_add interactive mode

@ghost ghost added the pushed Pull Request has already been pushed label Mar 8, 2017
@ghost ghost closed this Mar 8, 2017
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants