Project

General

Profile

Bug #4447

Unbound adds PTR records for host override aliases

Added by Lorenz Schori over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
02/19/2015
Due date:
% Done:

100%

Estimated time:
Affected Version:
Affected Architecture:

Description

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.

History

#1 Updated by Lorenz Schori over 5 years ago

Github Pull requests:

#2 Updated by Phillip Davis over 5 years ago

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

#3 Updated by Lorenz Schori over 5 years ago

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.

#4 Updated by Lorenz Schori over 5 years ago

Opened PR 1509 (proper master).

#5 Updated by Renato Botelho over 5 years ago

  • 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

#6 Updated by Lorenz Schori over 5 years ago

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

#7 Updated by Lorenz Schori over 5 years ago

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.

#8 Updated by Renato Botelho over 5 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF