Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dns/pfSense-pkg-bind: fixes and improvements. Implement #14314, Fix #11563, Fix #10693 #1256

Open
wants to merge 16 commits into
base: devel
Choose a base branch
from

Conversation

styletronix
Copy link

@styletronix styletronix commented May 2, 2023

Redmine issue #14314
Redmine issue #11563
Redmine issue #10693

new GUI options

  • Merge DDNS entries during zone update.
  • Using named-checkzone prior to replacing zone.db on config change.
  • Increment serial if zone changed. Based on actual zone serial.

Changes to bind.inc:
Added logic to detect zone config changes and update zone.db only if required.

Added ability to merge all zone records from existing zone.db bevor overwriting with new zone config.
Includes detection of deleted and added records of custom zone data and manual entries.
Current zone data is parsed with named-compilezone. Fallback to overwrite zone.db if current zone data can not be validated.

Added validation of Zone.db bevor overwriting the existing zone with named-checkzone.
If validation fails, a notice is generated and bind uses the old zone.db until the new zone is valid.

Changed service restart behaviour to use "reconfigure" instead. Includes fallback to restart service if reconfigure fails.

Escaping TXT and SPF zone records

@marcos-ng
Copy link
Collaborator

Please rebase and squash commits to one per feature or redmine issue.

@marcos-ng marcos-ng changed the title pfSense-pkg-bind Add option to keep DDNS Changes of master zones dns/pfSense-pkg-bind: fixes and improvements. Implement #14314, Fix #11563, Fix #10693 Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants