Compare domains instead of strings

Fixes #1291
This commit is contained in:
Artur Beljajev 2019-09-09 14:32:43 +03:00
parent a0212490d4
commit 6aa25456f5
2 changed files with 9 additions and 1 deletions

View file

@ -81,7 +81,7 @@ class Nameserver < ActiveRecord::Base
def glue_record_required?
return unless hostname? && domain
hostname.end_with?(domain.name)
DomainName(hostname).domain == domain.name
end
def normalize_attributes

View file

@ -24,4 +24,12 @@ class NameserverGlueRecordTest < ActiveSupport::TestCase
@nameserver.ipv4 = @nameserver.ipv6 = ''
assert @nameserver.valid?
end
def test_valid_without_ip_when_glue_record_is_not_required
domain = Domain.new(name: 'le.test')
nameserver = Nameserver.new(domain: domain, hostname: 'ns1.example.test')
nameserver.ipv4 = nameserver.ipv6 = nil
assert nameserver.valid?
end
end