Project

General

Profile

Bug #16952

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

Added by Robert Frank over 4 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
DNS
Target version:
Difficulty:
Triaged:
Bugzilla link:
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

Copied to Smart Proxy - Bug #16964: Removing ipv6 entries fails with dns_nsupdate due to name lookupClosed2016-10-14

Associated revisions

Revision 39ae1782 (diff)
Added by Timo Goebel over 4 years ago

fixes #16952 - pass type to proxy when destroying dns records

Revision d1fca2aa (diff)
Added by Timo Goebel over 4 years ago

fixes #16952 - pass type to proxy when destroying dns records

(cherry picked from commit 39ae1782ffde34a71f6ca2092acc8efd56522039)

History

#1 Updated by Dominic Cleal over 4 years ago

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

#2 Updated by Dominic Cleal over 4 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
  • Legacy Backlogs Release (now unused) set to 190

#3 Updated by Timo Goebel over 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Timo Goebel

#4 Updated by The Foreman Bot over 4 years ago

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

#5 Updated by Timo Goebel over 4 years ago

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

Also available in: Atom PDF