Skip to content

feat(api): gate NS changes on local suffix#1188

Open
nils-wisiol wants to merge 1 commit intomainfrom
20260210_dedyn_delegations
Open

feat(api): gate NS changes on local suffix#1188
nils-wisiol wants to merge 1 commit intomainfrom
20260210_dedyn_delegations

Conversation

@nils-wisiol
Copy link
Copy Markdown
Contributor

  • Add allow_local_ns_changes to Domain with defaults and migration
  • Use can_modify_ns_records to permit NS edits for local suffixes
  • Update RRset validation and deletion paths
  • Add RRset tests for allowed local-suffix NS changes

@nils-wisiol nils-wisiol force-pushed the 20260210_dedyn_delegations branch from 0369759 to 2cc29f5 Compare February 10, 2026 18:49
Add allow_local_ns_changes to Domain with defaults and migration
Use can_modify_ns_records to permit NS edits for local suffixes
Update RRset validation and deletion paths
Add RRset tests for allowed local-suffix NS changes

Tests: ./run-api-tests-stack.sh (fails: desecapi.tests.test_dyndns12update.MultipleDomainDynDNS12UpdateTest.test_ignore_minimum_ttl)
@nils-wisiol nils-wisiol force-pushed the 20260210_dedyn_delegations branch from 2cc29f5 to 23f70de Compare February 10, 2026 18:56
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.

1 participant