diff --git a/app/controllers/epp/domains_controller.rb b/app/controllers/epp/domains_controller.rb index 65ee56306..d9504a71d 100644 --- a/app/controllers/epp/domains_controller.rb +++ b/app/controllers/epp/domains_controller.rb @@ -4,43 +4,53 @@ class Epp::DomainsController < EppController skip_authorization_check only: :info def info - if Domain.release_to_auction - domain_name = DNS::DomainName.new(params[:parsed_frame].at_css('name').text.strip.downcase) + domain_name = DNS::DomainName.new(params[:parsed_frame].at_css('name').text.strip.downcase) + if Domain.release_to_auction if domain_name.at_auction? @name = domain_name @status = 'At auction' - render_epp_response '/epp/domains/info_auction' + render_epp_response '/epp/domains/info/unregistered_domain' return elsif domain_name.awaiting_payment? @name = domain_name @status = 'Awaiting payment' - render_epp_response '/epp/domains/info_auction' + render_epp_response '/epp/domains/info/unregistered_domain' return elsif domain_name.pending_registration? @name = domain_name @status = 'Reserved' - render_epp_response '/epp/domains/info_auction' + render_epp_response '/epp/domains/info/unregistered_domain' return end end - find_domain - find_password - authorize! :info, @domain, @password + if domain_name.registered? + find_domain + find_password + authorize! :info, @domain, @password - @hosts = params[:parsed_frame].css('name').first['hosts'] || 'all' + @hosts = params[:parsed_frame].css('name').first['hosts'] || 'all' - case @hosts - when 'del' - @nameservers = @domain.delegated_nameservers.sort - when 'sub' - @nameservers = @domain.subordinate_nameservers.sort - when 'all' - @nameservers = @domain.nameservers.sort + case @hosts + when 'del' + @nameservers = @domain.delegated_nameservers.sort + when 'sub' + @nameservers = @domain.subordinate_nameservers.sort + when 'all' + @nameservers = @domain.nameservers.sort + end + + render_epp_response '/epp/domains/info/registered_domain' + elsif domain_name.blocked? + @name = domain_name + @status = 'Blocked' + render_epp_response '/epp/domains/info/unregistered_domain' + elsif domain_name.reserved? + @name = domain_name + @status = 'Reserved' + render_epp_response '/epp/domains/info/unregistered_domain' end - - render_epp_response '/epp/domains/info' end def create diff --git a/app/models/dns/domain_name.rb b/app/models/dns/domain_name.rb index be7ece5c6..05eea2086 100644 --- a/app/models/dns/domain_name.rb +++ b/app/models/dns/domain_name.rb @@ -53,14 +53,6 @@ module DNS Whois::Record.refresh(self) end - def to_s - name - end - - private - - attr_reader :name - def registered? Domain.find_by_idn(name) end @@ -69,6 +61,18 @@ module DNS BlockedDomain.where(name: name).any? end + def reserved? + ReservedDomain.where(name: name).any? + end + + def to_s + name + end + + private + + attr_reader :name + def zone_with_same_origin? DNS::Zone.where(origin: name).any? end diff --git a/app/views/epp/domains/info.xml.builder b/app/views/epp/domains/info/registered_domain.xml.builder similarity index 72% rename from app/views/epp/domains/info.xml.builder rename to app/views/epp/domains/info/registered_domain.xml.builder index 850334955..5424b73d4 100644 --- a/app/views/epp/domains/info.xml.builder +++ b/app/views/epp/domains/info/registered_domain.xml.builder @@ -5,7 +5,7 @@ xml.epp_head do end xml.resData do - xml.tag!('domain:infData', 'xmlns:domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd') do + xml.tag! 'domain:infData', 'xmlns:domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd' do xml.tag!('domain:name', @domain.name) xml.tag!('domain:roid', @domain.roid) @domain.statuses.each do |s| @@ -27,32 +27,32 @@ xml.epp_head do @nameservers.each do |x| xml.tag!('domain:hostAttr') do xml.tag!('domain:hostName', x.hostname) - x.ipv4.each{|ip| xml.tag!('domain:hostAddr', ip, 'ip' => 'v4') } if x.ipv4.present? - x.ipv6.each{|ip| xml.tag!('domain:hostAddr', ip, 'ip' => 'v6') } if x.ipv6.present? + if x.ipv4.present? + x.ipv4.each { |ip| xml.tag!('domain:hostAddr', ip, 'ip' => 'v4') } + end + + if x.ipv6.present? + x.ipv6.each { |ip| xml.tag!('domain:hostAddr', ip, 'ip' => 'v6') } + end end end end end - ## TODO Find out what this domain:host is all about - xml.tag!('domain:clID', @domain.registrar.code) xml.tag!('domain:crID', @domain.cr_id) xml.tag!('domain:crDate', @domain.created_at.try(:iso8601)) if @domain.updated_at > @domain.created_at - upID = @domain.updator.try(:registrar) - upID = upID.code if upID.present? # Did updator return a kind of User that has a registrar? - xml.tag!('domain:upID', upID) if upID.present? # optional upID + updator = @domain.updator.try(:registrar) + updator = updator.code if updator.present? + xml.tag!('domain:upID', updator) if updator.present? xml.tag!('domain:upDate', @domain.updated_at.try(:iso8601)) end xml.tag!('domain:exDate', @domain.valid_to.iso8601) - # TODO Make domain transferrable - #xml.tag!('domain:trDate', @domain.transferred_at) if @domain.transferred_at - if can? :view_password, @domain, @password xml.tag!('domain:authInfo') do xml.tag!('domain:pw', @domain.transfer_code) @@ -62,10 +62,11 @@ xml.epp_head do end if @domain.dnskeys.any? - ds_data = Setting.ds_data_allowed ? @domain.dnskeys.find_all { |key| key.ds_digest.present? } : [] - key_data = Setting.key_data_allowed ? @domain.dnskeys.find_all { |key| key.public_key.present? } : [] + ds_data = Setting.ds_data_allowed ? + @domain.dnskeys.find_all { |key| key.ds_digest.present? } : [] + key_data = Setting.key_data_allowed ? + @domain.dnskeys.find_all { |key| key.public_key.present? } : [] - # is there any reason to include without xml.extension do def tag_key_data(xml, key) xml.tag!('secDNS:keyData') do diff --git a/app/views/epp/domains/info_auction.xml.builder b/app/views/epp/domains/info/unregistered_domain.xml.builder similarity index 57% rename from app/views/epp/domains/info_auction.xml.builder rename to app/views/epp/domains/info/unregistered_domain.xml.builder index b95553935..a34dfa51a 100644 --- a/app/views/epp/domains/info_auction.xml.builder +++ b/app/views/epp/domains/info/unregistered_domain.xml.builder @@ -1,16 +1,16 @@ xml.epp_head do xml.response do - xml.result('code' => '1000') do + xml.result code: '1000' do xml.msg 'Command completed successfully' end xml.resData do xml.tag! 'domain:infData', 'xmlns:domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd' do - xml.tag!('domain:name', @name) - xml.tag!('domain:status', 's' => @status) + xml.tag! 'domain:name', @name + xml.tag! 'domain:status', 's' => @status end end - render('epp/shared/trID', builder: xml) + render 'epp/shared/trID', builder: xml end -end +end \ No newline at end of file diff --git a/test/integration/epp/domain/info/base_test.rb b/test/integration/epp/domain/info/base_test.rb index a112ed840..93f04a476 100644 --- a/test/integration/epp/domain/info/base_test.rb +++ b/test/integration/epp/domain/info/base_test.rb @@ -106,4 +106,54 @@ class EppDomainInfoBaseTest < ApplicationIntegrationTest assert_nil response_xml.at_xpath('//domain:authInfo/domain:pw', 'domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd') end + + def test_blocked_domain + assert_equal 'blocked.test', blocked_domains(:one).name + + request_xml = <<-XML + + + + + + blocked.test + + + + + XML + + post '/epp/command/info', { frame: request_xml }, 'HTTP_COOKIE' => 'session=api_bestnames' + + response_xml = Nokogiri::XML(response.body) + assert_equal '1000', response_xml.at_css('result')[:code] + assert_equal 1, response_xml.css('result').size + assert_equal 'blocked.test', response_xml.at_xpath('//domain:name', 'domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd').text + assert_equal 'Blocked', response_xml.at_xpath('//domain:status', 'domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd')['s'] + end + + def test_reserved_domain + assert_equal 'reserved.test', reserved_domains(:one).name + + request_xml = <<-XML + + + + + + reserved.test + + + + + XML + + post '/epp/command/info', { frame: request_xml }, 'HTTP_COOKIE' => 'session=api_bestnames' + + response_xml = Nokogiri::XML(response.body) + assert_equal '1000', response_xml.at_css('result')[:code] + assert_equal 1, response_xml.css('result').size + assert_equal 'reserved.test', response_xml.at_xpath('//domain:name', 'domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd').text + assert_equal 'Reserved', response_xml.at_xpath('//domain:status', 'domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd')['s'] + end end diff --git a/test/models/dns/domain_name_test.rb b/test/models/dns/domain_name_test.rb index 0fdd81d34..e1e9df79e 100644 --- a/test/models/dns/domain_name_test.rb +++ b/test/models/dns/domain_name_test.rb @@ -111,4 +111,16 @@ class DNS::DomainNameTest < ActiveSupport::TestCase domain_name = DNS::DomainName.new('shop.test') assert_equal 'shop.test', domain_name.to_s end + + def test_blocked + assert_equal 'blocked.test', blocked_domains(:one).name + assert DNS::DomainName.new('blocked.test').blocked? + assert_not DNS::DomainName.new('nonblocked .test').blocked? + end + + def test_reserved + assert_equal 'reserved.test', reserved_domains(:one).name + assert DNS::DomainName.new('reserved.test').reserved? + assert_not DNS::DomainName.new('unreserved.test').reserved? + end end