Unbound adds PTR records for host override aliases
It is not recommended to add more than one
PTR per IP. Especially for systems relying on reverse IP lookups like Kerberos, it is crucial that the DNS server behaves properly.
I expect the devs will want you to make just a pull request against master. Then the devs will sort out committing to RELENG_2_2 branch also.
And I assume this is your first contribution for a while? If so, then you will also need to submit a Contributor License Agreement (CLA) https://portal.pfsense.org/members/signup/ICLA
This is not my first patch, CLA is already done.
1500 was bogus, because it was against an old master,
RELENG_2_2 is identical to
master in this case.
Opened PR 1509 (proper
- Status changed from New to Feedback
- Target version set to 2.2.1
- % Done changed from 0 to 100
Pull Request has been merged and commit cherry-picked to RELENG_2_2
Oops, that completely broke revers DNS lookups. Regrettably I confused the config keys, the code looks for
ip instead of
ipaddr. As a result only one
PTR is ever added.
Opened PR 1513
Thanks for merging the followup, and sorry for the noise. It works now as expected with the newest snapshot.
PTR queries do not return alias names anymore.
- Status changed from Feedback to Resolved
Also available in: Atom