Add update record endpoint #7

antman · 2026-09-03T04:30:52Z

antman commented

2026-09-03 04:30:52 +00:00

Motivation

The server manager will need an endpoint where it can update existing DNS records, for example if the IP address of the server changes.

Solution

Implemented and endpoint to update an existing DNS record. Found that supplying an invalid record id (not a valid uuid) could cause the server to throw and return 500. When the client supplies incorrect data, the server must return a 400, so implemented a middleware that validates the record id before the controller operates on it.

### Motivation The server manager will need an endpoint where it can update existing DNS records, for example if the IP address of the server changes. ### Solution Implemented and endpoint to update an existing DNS record. Found that supplying an invalid record id (not a valid uuid) could cause the server to throw and return 500. When the client supplies incorrect data, the server must return a 400, so implemented a middleware that validates the record id before the controller operates on it.

antman added 1 commit

2026-09-03 04:30:52 +00:00

Add tests for update record endpoint

/ test-and-lint (push) Failing after 10s

Details

2e1663e4d3

antman added 1 commit

2026-09-03 04:40:25 +00:00

Adds schema deno task

/ test-and-lint (push) Failing after 10s

Details

eb8564c9de

antman added 1 commit

2026-09-03 05:51:48 +00:00

adds update record endpoint

/ test-and-lint (push) Failing after 10s

Details

7119d55d5b

antman added 1 commit

2026-09-03 05:53:01 +00:00

update test

/ test-and-lint (push) Failing after 10s

Details

3ecd3b29fa

antman added 1 commit

2026-09-03 05:57:02 +00:00

update porkbun mock

/ test-and-lint (push) Failing after 9s

Details

fda58ed455

antman added 1 commit

2026-09-03 06:01:21 +00:00

return updated record

/ test-and-lint (push) Failing after 12s

Details

d1aea80f48

antman added 1 commit

2026-09-03 06:15:33 +00:00

Validate record id

/ test-and-lint (push) Successful in 10s

Details

6ddac24e4c

antman added 1 commit

2026-09-03 06:16:30 +00:00

cleanup mock logs

/ test-and-lint (push) Successful in 10s

Details

522de5d766

antman added 1 commit

2026-09-03 06:49:36 +00:00

cleanup auth log

/ test-and-lint (push) Successful in 11s

Details

777c1c334c

antman added 1 commit

2026-09-03 06:51:41 +00:00

hmm

/ test-and-lint (push) Successful in 10s

Details

b3c04e3e93

antman changed title from ~~WIP: Add tests for update record endpoint~~ to WIP: Add update record endpoint

2026-09-03 06:53:20 +00:00

antman added 1 commit

2026-09-03 06:55:22 +00:00

lint auth

/ test-and-lint (push) Successful in 9s

Details

385959e1dc

antman reviewed

2026-09-03 07:02:55 +00:00

src/logic/dns/updateRecord.controller.ts

					
				@ -0,0 +30,4 @@

				    content: updated.content,

				    ttl: updated.ttl,

				    priority: updated.priority,

				    notes: updated.notes,

I explicitly set the properties here so that extra properties in the update object don't get saved to the database. Otherwise someone could supply a property like domainId in their request and the server would change which domain a record was for 😬

I explicitly set the properties here so that extra properties in the update object don't get saved to the database. Otherwise someone could supply a property like `domainId` in their request and the server would change which domain a record was for 😬