rollback

app/models/reserved_domain.rb

@@ -6,7 +6,6 @@ class ReservedDomain < ApplicationRecord
   before_save :sync_dispute_password
   after_destroy :remove_data
 
-  before_validation :normalize_name, on: %w[create, update]
   validates :name, domain_name: true, uniqueness: true
 
   alias_attribute :registration_code, :password
@@ -41,7 +40,7 @@ def new_password_for(name)
   end
 
   def name=(val)
-    super SimpleIDN.to_unicode(val).mb_chars.downcase.strip
+    super SimpleIDN.to_unicode(val)
   end
 
   def fill_empty_passwords
@@ -70,10 +69,4 @@ def generate_data
   def remove_data
     UpdateWhoisRecordJob.perform_later name, 'reserved'
   end
-
-  private
-
-  def normalize_name
-    self.name = SimpleIDN.to_unicode(name).mb_chars.downcase.strip
-  end
 end

app/validators/domain_name_validator.rb

@@ -2,11 +2,7 @@ class DomainNameValidator < ActiveModel::EachValidator
   # rubocop:disable Metrics/CyclomaticComplexity
   # rubocop:disable Metrics/LineLength
   def validate_each(record, attribute, value)
-    origins = DNS::Zone.origins
-
-    if origins.include?(value)
-      record.errors.add(:base, :domain_name_blocked)
-    elsif !self.class.validate_format(value)
+    if !self.class.validate_format(value)
       record.errors.add(attribute, options[:message] || record.errors.generate_message(attribute, :invalid))
     elsif !self.class.validate_blocked(value)
       record.errors.add(:base, :domain_name_blocked)
@@ -22,7 +18,7 @@ def validate_format(value)
       origins = DNS::Zone.origins
       # if someone tries to register an origin domain, let this validation pass
       # the error will be caught in blocked domains validator
-      return false unless origins.include?(value.split('.').last)
+      return true if origins.include?(value)
 
       general_domains = /(#{origins.join('|')})/
 
@@ -32,7 +28,6 @@ def validate_format(value)
         return false unless value.match?(regexp)
 
         value = SimpleIDN.to_unicode(value).mb_chars.downcase.strip
-
       end
 
       unicode_chars = /\u00E4\u00F5\u00F6\u00FC\u0161\u017E/ # äõöüšž

test/models/reserved_domain_test.rb

@@ -13,23 +13,4 @@ def test_aliases_registration_code_to_password
     reserved_domain = ReservedDomain.new(password: 'reserved-001')
     assert_equal 'reserved-001', reserved_domain.registration_code
   end
-
-  def test_create_reserved_domain_with_punycode_name
-    reserved_domain = ReservedDomain.create(name: 'xn--4ca7aey.test')
-    assert reserved_domain.valid?
-  end
-
-  def test_create_reserved_domain_with_unicode_name
-    reserved_domain = ReservedDomain.create(name: 'õäöü.test')
-    assert reserved_domain.valid?
-  end
-
-  def test_cannot_create_the_same_domain_twicde_with_punycode_and_unicode
-    punycode_reserved_domain = ReservedDomain.new(name: 'xn--4ca7aey.test')
-    assert punycode_reserved_domain.valid?
-    punycode_reserved_domain.save && punycode_reserved_domain.reload
-
-    unicode_reserved_domain = ReservedDomain.new(name: 'õäöü.test')
-    assert_not unicode_reserved_domain.valid?
-  end
 end