From eff8dd5d7a625b1e3bb7ab4c0a282980861f3a58 Mon Sep 17 00:00:00 2001
From: Martin Lensment <mlensment@gmail.com>
Date: Thu, 13 Aug 2015 11:42:54 +0300
Subject: [PATCH] Do not overwrite auth info on domain create #2833

---
 app/models/domain.rb     | 7 ++++++-
 app/models/epp/domain.rb | 2 +-
 spec/support/epp.rb      | 4 +---
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/app/models/domain.rb b/app/models/domain.rb
index d5db03798..2a03277de 100644
--- a/app/models/domain.rb
+++ b/app/models/domain.rb
@@ -528,8 +528,13 @@ class Domain < ActiveRecord::Base
     Registrant.find_by(id: pending_json['domain']['registrant_id'].last)
   end
 
-  # rubocop:disable Lint/Loop
   def generate_auth_info
+    return if auth_info.present?
+    generate_auth_info!
+  end
+
+  # rubocop:disable Lint/Loop
+  def generate_auth_info!
     begin
       self.auth_info = SecureRandom.hex
     end while self.class.exists?(auth_info: auth_info)
diff --git a/app/models/epp/domain.rb b/app/models/epp/domain.rb
index f34312bc6..c1852fef5 100644
--- a/app/models/epp/domain.rb
+++ b/app/models/epp/domain.rb
@@ -616,7 +616,7 @@ class Epp::Domain < Domain
       if dt.approved?
         transfer_contacts(current_user.registrar_id)
         dt.notify_losing_registrar(old_contact_codes, old_registrant_code)
-        generate_auth_info
+        generate_auth_info!
         self.registrar = current_user.registrar
       end
 
diff --git a/spec/support/epp.rb b/spec/support/epp.rb
index db6b522c5..b14eb79a7 100644
--- a/spec/support/epp.rb
+++ b/spec/support/epp.rb
@@ -353,9 +353,7 @@ module Epp
     defaults = {
       name: { value: next_domain_name },
       period: nil,
-      authInfo: {
-        pw: { value: '98oiewslkfkd', attrs: { roid: 'citizen_1234-REP' } }
-      }
+      authInfo: nil
     }
 
     xml_params = defaults.deep_merge(xml_params)