Project

General

Profile

Actions

Bug #4447

closed

Unbound adds PTR records for host override aliases

Added by znerol znerol almost 10 years ago. Updated over 9 years ago.

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

100%

Estimated time:
Plus Target Version:
Release Notes:
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.

Actions #1

Updated by znerol znerol almost 10 years ago

Github Pull requests:
Actions #2

Updated by Phillip Davis almost 10 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

Actions #3

Updated by znerol znerol almost 10 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.

Actions #4

Updated by znerol znerol over 9 years ago

Opened PR 1509 (proper master).

Actions #5

Updated by Renato Botelho over 9 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

Actions #6

Updated by znerol znerol over 9 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

Actions #7

Updated by znerol znerol over 9 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.

Actions #8

Updated by Renato Botelho over 9 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF