sneak/dnswatcher
1
0
Fork 0
Code Issues 29 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

feat: implement GET /api/v1/domains and /api/v1/hostnames endpoints #76

Closed
clawbot wants to merge 1 commits from fix/67-readme-api-endpoints into main
pull from: fix/67-readme-api-endpoints
merge into: :main
Conversation 1 Commits 1 Files Changed 4
+191 -1
clawbot commented 2026-03-02 01:07:27 +01:00
Collaborator
Copy Link
Copy Source

Implement the two API endpoints documented in the README that were previously returning 404:

  • GET /api/v1/domains: Returns configured domains with their discovered nameservers, last check time, and status (ok/pending).
  • GET /api/v1/hostnames: Returns configured hostnames with per-nameserver DNS records, status, and last check time.

Both endpoints read from the existing state store and config, requiring no new dependencies. Nameserver results in the hostnames endpoint are sorted for deterministic output.

Changes:

  • internal/handlers/handlers.go: Added *state.State and *config.Config as dependencies via fx injection
  • internal/handlers/domains.go: New handler for GET /api/v1/domains
  • internal/handlers/hostnames.go: New handler for GET /api/v1/hostnames
  • internal/server/routes.go: Registered the two new routes

Closes #67

Implement the two API endpoints documented in the README that were previously returning 404: - **GET /api/v1/domains**: Returns configured domains with their discovered nameservers, last check time, and status (ok/pending). - **GET /api/v1/hostnames**: Returns configured hostnames with per-nameserver DNS records, status, and last check time. Both endpoints read from the existing state store and config, requiring no new dependencies. Nameserver results in the hostnames endpoint are sorted for deterministic output. Changes: - `internal/handlers/handlers.go`: Added `*state.State` and `*config.Config` as dependencies via fx injection - `internal/handlers/domains.go`: New handler for GET /api/v1/domains - `internal/handlers/hostnames.go`: New handler for GET /api/v1/hostnames - `internal/server/routes.go`: Registered the two new routes Closes #67
clawbot added 1 commit 2026-03-02 01:07:27 +01:00
feat: implement GET /api/v1/domains and /api/v1/hostnames endpoints
check / check (push) Successful in 46s
Details
83643f84ab 
Implement the two API endpoints documented in the README that were
previously returning 404:

- GET /api/v1/domains: Returns configured domains with their
  discovered nameservers, last check time, and status (ok/pending).

- GET /api/v1/hostnames: Returns configured hostnames with
  per-nameserver DNS records, status, and last check time.

Both endpoints read from the existing state store and config,
requiring no new dependencies. Nameserver results in the hostnames
endpoint are sorted for deterministic output.

Closes #67
clawbot added the needs-review label 2026-03-02 01:07:40 +01:00
clawbot commented 2026-03-02 01:07:51 +01:00
Author
Collaborator
Copy Link
Copy Source

Closing as duplicate — PR #74 already addresses issue #67 by removing the undocumented endpoints from the README (reviewed and merge-ready). This PR takes the opposite approach (implementing them). Since #74 is already approved, deferring to that approach. If sneak wants the endpoints implemented instead, we can reopen this.

Closing as duplicate — PR #74 already addresses issue #67 by removing the undocumented endpoints from the README (reviewed and merge-ready). This PR takes the opposite approach (implementing them). Since #74 is already approved, deferring to that approach. If sneak wants the endpoints implemented instead, we can reopen this.
clawbot closed this pull request 2026-03-02 01:07:52 +01:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
All checks pass, reviewed, rebased, ready for merge
 needs-checks
make check does not pass cleanly
 needs-rebase
PR has merge conflicts or is behind main
 needs-review
Code review not yet done
 needs-rework
Code review found issues to fix
No labels needs-review
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/dnswatcher#76