Project

General

Profile

Actions

Bug #11850

closed

DNS error message assumes two nameservers

Added by Dominic Cleal over 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Low
Category:
DNS
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

The error message used by the DNS conflict check to handle DNS resolution failures assumes the domain always has zero or two nameservers. Given the implementation of Domain#nameservers, which can return 0-many, it shouldn't be hardcoded to two.

https://github.com/theforeman/foreman/pull/2680/files#diff-2e08cc169e9a6f52d8ede37144c208acR179


Related issues 1 (0 open1 closed)

Related to Foreman - Refactor #11709: Improve DNS conflict timeout handlingClosedLukas Zapletal09/07/2015Actions
Actions #1

Updated by Dominic Cleal over 8 years ago

Actions #2

Updated by Lukas Zapletal over 8 years ago

Doesnt our UI force you to enter 0-2?

In the worst case there will be "nil" in the error message, the code works just fine.

Actions #3

Updated by Dominic Cleal over 8 years ago

Lukas Zapletal wrote:

Doesnt our UI force you to enter 0-2?

Domain#nameservers uses SOA and NS records, I think you're thinking of the subnet dns_primary/secondary fields which are different and not used here.

In the worst case there will be "nil" in the error message, the code works just fine.

Yeah, or missing nameservers in the error message. This is why I've filed it as low priority.

Actions #4

Updated by The Foreman Bot over 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2733 added
  • Pull request deleted ()
Actions #5

Updated by Dominic Cleal over 8 years ago

  • Assignee set to Lukas Zapletal
  • translation missing: en.field_release set to 63
Actions #6

Updated by Lukas Zapletal over 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF