Skip to content

Add missing VinylDNS API endpoints#104

Merged
JoshSEdwards merged 3 commits into
vinyldns:masterfrom
Sokitha-21:implement_missing_enpoints
May 27, 2026
Merged

Add missing VinylDNS API endpoints#104
JoshSEdwards merged 3 commits into
vinyldns:masterfrom
Sokitha-21:implement_missing_enpoints

Conversation

@Sokitha-21
Copy link
Copy Markdown
Member

  • Added endpoint builders and query helpers for health/status, zones, record sets, groups and users.
  • Implemented client methods for monitoring, status management, backend IDs, deleted zones, ACL rule add/delete, record set metrics and history, group utilities, and user administration.
  • Introduced ownership transfer helpers for record sets (request/approve/reject/cancel) that wrap record set update payloads.
  • Added unit tests covering all new endpoints and client methods.
  • Fix SigV4 signing for DELETE endpoint.

@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Jan 21, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown
Contributor

@JoshSEdwards JoshSEdwards left a comment

Choose a reason for hiding this comment

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

Just a few minor Javadoc discrepancies and some suggestions to make error handling more consistent.

Comment thread src/main/java/io/vinyldns/java/VinylDNSClientImpl.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClientImpl.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
Comment thread src/main/java/io/vinyldns/java/VinylDNSClient.java Outdated
@JoshSEdwards JoshSEdwards merged commit b72c4e2 into vinyldns:master May 27, 2026
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