Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Overload local_data_remove to support removing specific records #1087

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rcmcdonald91
Copy link
Contributor

Here we overload the local_data_remove control command to support deleting specific records. Curently, this command deletes all records for a given zone. The modification works by attempting to parse the command argument first as a complete record and then as just a domain name, if the first attempt failed.

This preserves the command's behavior, while also supporting removing specific records from the zone tree.

Here we overload the `local_data_remove` control command to support
deleting specific records. Curently, this command deletes all records
for a given zone. The modification works by attempting to parse the
command argument first as a complete record and then as just a domain
name, if the first attempt failed.

This preserves the command's behavior, while also supporting removing
specific records from the zone tree.

Signed-off-by: R. Christian McDonald <rcm@rcm.sh>
@rcmcdonald91
Copy link
Contributor Author

@wcawijngaards can I get some eyes on this? Thanks!

@rcmcdonald91
Copy link
Contributor Author

rcmcdonald91 commented Jul 25, 2024

@gthess

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

Successfully merging this pull request may close these issues.

1 participant