More than one "Conditional Forwarding" entry in the GUI

DL6ER · December 1, 2023, 11:01am

Add support for multiple reverse servers

development-v6 ← new/multi_revServer

opened 11:00AM - 01 Dec 23 UTC

# What does this implement/fix? Add `dns.revServers` and migrate existing `dn…s.revServer` if present. This allows multiple reverse servers to be added. --- **Related issue or feature (if applicable):** N/A **Pull request in [docs](https://github.com/pi-hole/docs) with documentation (if applicable):** N/A --- **By submitting this pull request, I confirm the following:** 1. I have read and understood the [contributors guide](https://docs.pi-hole.net/guides/github/contributing/), as well as this entire template. I understand which branch to base my commits and Pull Requests against. 2. I have commented my proposed changes within the code. 3. I am willing to help maintain this change if there are issues with it later. 4. It is compatible with the [EUPL 1.2 license](https://opensource.org/licenses/EUPL-1.1) 5. I have squashed any insignificant commits. ([`git rebase`](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)) ## Checklist: - [x] The code change is tested and works locally. - [x] I based my code and PRs against the repositories `developmental` branch. - [x] I [signed off](https://docs.pi-hole.net/guides/github/how-to-signoff/) all commits. Pi-hole enforces the [DCO](https://docs.pi-hole.net/guides/github/dco/) for all contributions - [x] I [signed](https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits) all my commits. Pi-hole requires signatures to verify authorship - [x] I have read the above and my PR is ready for review.

github.com/pi-hole/web

Add support for multiple reverse server

development-v6 ← new/multi_revServer

opened 12:28PM - 01 Dec 23 UTC

DL6ER

+65 -91

# What does this implement/fix? This PR implements the necessary UI-changes f…or a long-wanted feature request. This PR depends on https://github.com/pi-hole/FTL/pull/1797 ![Screenshot from 2023-12-01 12-48-52](https://github.com/pi-hole/web/assets/16748619/99820d58-1563-4961-b467-9a7f021f9d4a) --- **Related issue or feature (if applicable):** N/A **Pull request in [docs](https://github.com/pi-hole/docs) with documentation (if applicable):** N/A --- **By submitting this pull request, I confirm the following:** 1. I have read and understood the [contributors guide](https://docs.pi-hole.net/guides/github/contributing/), as well as this entire template. I understand which branch to base my commits and Pull Requests against. 2. I have commented my proposed changes within the code. 3. I am willing to help maintain this change if there are issues with it later. 4. It is compatible with the [EUPL 1.2 license](https://opensource.org/licenses/EUPL-1.1) 5. I have squashed any insignificant commits. ([`git rebase`](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)) ## Checklist: - [x] The code change is tested and works locally. - [x] I based my code and PRs against the repositories `developmental` branch. - [x] I [signed off](https://docs.pi-hole.net/guides/github/how-to-signoff/) all commits. Pi-hole enforces the [DCO](https://docs.pi-hole.net/guides/github/dco/) for all contributions - [x] I [signed](https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits) all my commits. Pi-hole requires signatures to verify authorship - [x] I have read the above and my PR is ready for review.

DL6ER · February 13, 2024, 11:58am

Follow-up: The feature for defining multiple server has landed in development-v6 and will definitely be released with Pi-hole v6.0. We are currently also working on a nicer user interface:

github.com/pi-hole/web

Better user interface to edit reverse DNS servers (`dns.revServers`)

development ← tweak/multi_revServers

opened 07:01PM - 27 Jan 24 UTC

rdwebdesign

+332 -6

### What does this PR aim to accomplish? Replace the current single `<textare…a>` with a table. Each reverse server will be in a separate row and each field will be in a separate table cell. A small legend will improve usability (maybe we will need to change the help text). **NOTE:** ~~Depends on https://github.com/pi-hole/web/pull/2885~~ _(already merged)_ ### How does this PR accomplish the above? Using _datatables_ plugin and a few functions to deal with _dns.revServers_ entries. ![image](https://github.com/pi-hole/web/assets/1385443/9c8613ae-bb71-4dbe-89b0-9576942de1a2) --- **By submitting this pull request, I confirm the following:** 1. I have read and understood the [contributors guide](https://docs.pi-hole.net/guides/github/contributing/), as well as this entire template. I understand which branch to base my commits and Pull Requests against. 2. I have commented my proposed changes within the code and I have tested my changes. 3. I am willing to help maintain this change if there are issues with it later. 4. It is compatible with the [EUPL 1.2 license](https://opensource.org/licenses/EUPL-1.1) 5. I have squashed any insignificant commits. ([`git rebase`](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)) 6. I have checked that another pull request for this purpose does not exist. 7. I have considered, and confirmed that this submission will be valuable to others. 8. I accept that this submission may not be used, and the pull request closed at the will of the maintainer. 9. I give this submission freely, and claim no ownership to its content. --- - [x] I have read the above and my PR is ready for review. *Check this box to confirm*

rastlov · November 8, 2025, 4:33am

Multiple conditional forwarders works for me now. One per line on the GUI. I have multiple subnets and one domain. One pi-hole per subnet. DHCP on the routers. Port forwarding for DNS on the internal routers.

true,10.69.5.0/24,10.69.5.1,example.com
true,10.69.2.0/24,10.69.2.1,example.com
true,10.69.9.0/24,10.69.9.1,example.com