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

Feature: Add function to zones API to add multiple ResourceRecordSets at once #23

Merged

Conversation

Veratil
Copy link
Contributor

@Veratil Veratil commented Aug 26, 2024

PowerDNS' API allows adding multiple records at the same time, but the current go-powerdns API limits this to a single ResourceRecordSet. This new function, AddRecordSetsToZone (added the s), allows an array to be passed allowing batch updates of multiple DNS records.

@Veratil Veratil force-pushed the feature/zones-add-multiple-recordsets branch from 7d1b0b9 to ae05536 Compare August 26, 2024 01:53
apis/zones/zones_addrecordset.go Outdated Show resolved Hide resolved
Copy link
Member

@martin-helmich martin-helmich left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@martin-helmich martin-helmich merged commit d57738d into mittwald:master Aug 29, 2024
3 checks passed
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