Skip to content

[CDTOOL-1321] Add DNS Zones and TSIG Keys Support#805

Merged
rcaril merged 7 commits intomainfrom
rcaril/CDTOOL-1321-dns-support
May 6, 2026
Merged

[CDTOOL-1321] Add DNS Zones and TSIG Keys Support#805
rcaril merged 7 commits intomainfrom
rcaril/CDTOOL-1321-dns-support

Conversation

@rcaril
Copy link
Copy Markdown
Member

@rcaril rcaril commented May 1, 2026

Change summary

This PR adds support for Fastly DNS Zones and TSIG Keys.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  • Does your submission pass tests?

Changes to Core Features:

  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

Notes:

Added a generic Nullable[T] helper to fastly/helpers.go to support fields that need to distinguish between "omit from request" and "explicitly set to null". This was specifically needed to support the inbound_tsig_key_id field.
Internal conversation on this can be found here.

rcaril added 2 commits May 1, 2026 10:19
- Add CRUD and paginated list operations for /dns/v1/zones
  - Add CRUD and paginated list operations for /dns/v1/tsig-keys
  - Add ErrMissingAlgorithm to fastly/errors.go
@rcaril rcaril marked this pull request as ready for review May 1, 2026 14:41
@rcaril rcaril requested a review from a team as a code owner May 1, 2026 14:41
@rcaril rcaril requested a review from jedisct1 May 1, 2026 14:41
@rcaril rcaril force-pushed the rcaril/CDTOOL-1321-dns-support branch from 78b28ad to 3fc4199 Compare May 1, 2026 14:48
Comment thread fastly/errors.go Outdated
Comment thread fastly/dns/v1/tsigkeys/api_update.go
Comment thread fastly/dns/v1/dnszones/api_response.go
Comment thread fastly/dns/v1/dnszones/api_response.go
Comment thread fastly/dns/v1/dnszones/api_response.go Outdated
@rcaril rcaril requested a review from jedisct1 May 5, 2026 19:49
@rcaril
Copy link
Copy Markdown
Member Author

rcaril commented May 5, 2026

Since I added a new helper function i'm looping in @kpfleming to eyeball this PR.

@rcaril rcaril requested a review from kpfleming May 5, 2026 20:34
Comment thread fastly/dns/v1/dnszones/api_list.go Outdated
@rcaril rcaril requested a review from kpfleming May 6, 2026 16:31
@rcaril rcaril merged commit fab9fb2 into main May 6, 2026
8 checks passed
@rcaril rcaril deleted the rcaril/CDTOOL-1321-dns-support branch May 6, 2026 16:59
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.

3 participants