Actions
Refactor #17880
closedDNS CNAME parameters methods are swapped
Difficulty:
trivial
Triaged:
Pull request:
Description
All DNS methods uses fqdn parameters as the target of the action and the second argument as the attribute that will be set to that target, e.g.
modules/dns_common/dns_common.rb
def a_record_conflicts(fqdn, ip)
...
end
def create_a_record(fqdn, ip)
...
end
def ptr_record_conflicts(fqdn, ip)
...
end
CNAME methods does the opposite, using attribute(fqdn alias) as the first argument and target(fqdn) as a second argument
modules/dns_common/dns_common.rb
def create_cname_record(fqdn, value)
...
end
def cname_record_conflicts(fqdn, target)
...
end
fqdn here actually means the alias we want to attribute to target
CNAME implementation is only used in dnscmd provider and I imagine that just a few make use of it.
Those which use this API would not have their aliases been set to their hosts.
Probably the provider would raise an error saying the fqdn is been use by an existing entry or the CNAME doesn't exist.
Updated by Guilherme Maluf Balzana about 8 years ago
- Related to Feature #17879: Add DNS CNAME entry for nsupdate added
Updated by Guilherme Maluf Balzana about 8 years ago
- Related to deleted (Feature #17879: Add DNS CNAME entry for nsupdate)
Updated by Guilherme Maluf Balzana about 8 years ago
- Blocked by Feature #17879: Add DNS CNAME entry for nsupdate added
Updated by The Foreman Bot about 8 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/smart-proxy/pull/490 added
Updated by Anonymous almost 8 years ago
- Related to Refactor #18398: Use consistent naming of parameters in the DNS modules added
Updated by Anonymous almost 8 years ago
- Status changed from Ready For Testing to Rejected
That PR got cosed, but #18398 opened.
Actions