Project

General

Profile

Actions

Bug #16952

closed

Removing ipv6 entries does not pass record type (AAAA) to smart proxy

Added by Robert Frank about 8 years ago. Updated over 6 years ago.

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

Description

When I try to remove the IPv6 address from a host, the smart proxy fails to update the DNS server.

  1. The foreman server doesn't set the resource type when sending the delete request for an fqdn to the proxy. It calls <url>/dns/<fqdn> twice instead of <url>/dns/<fqdn>/A and <url>/dns/<fqdn>/AAAA. This causes the proxy to default to A and deleting the A record twice (rather than the A and the AAAA record). I believe this can be fixed by changing the destroy method in ./lib/net/dns/forward_record.rb to
    def destroy
      super
      proxy.delete("#{hostname}/#{type}")
    

Related issues 1 (0 open1 closed)

Copied to Smart Proxy - Bug #16964: Removing ipv6 entries fails with dns_nsupdate due to name lookupClosed10/14/2016Actions
Actions #1

Updated by Dominic Cleal about 8 years ago

  • Copied to Bug #16964: Removing ipv6 entries fails with dns_nsupdate due to name lookup added
Actions #2

Updated by Dominic Cleal about 8 years ago

  • Subject changed from Removing ipv6 entries does not work with dns_nsupdate proxy (foreman 1.13.0) to Removing ipv6 entries does not pass record type (AAAA) to smart proxy
  • Description updated (diff)
  • Category changed from Smart Proxy to DNS
  • Translation missing: en.field_release set to 190
Actions #3

Updated by Timo Goebel about 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Timo Goebel
Actions #4

Updated by The Foreman Bot about 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3948 added
Actions #5

Updated by Timo Goebel about 8 years ago

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

Also available in: Atom PDF