Disabled fax

app/controllers/epp/contacts_controller.rb

@@ -100,6 +100,7 @@ class Epp::ContactsController < EppController
       }
     end
     contact_org_disabled 
+    fax_disabled
     @prefix = nil
     requires 'extension > extdata > ident'
   end
@@ -113,6 +114,7 @@ class Epp::ContactsController < EppController
       }
     end
     contact_org_disabled
+    fax_disabled
     requires 'id', 'authInfo > pw'
     @prefix = nil
   end
@@ -131,4 +133,13 @@ class Epp::ContactsController < EppController
       msg: "#{I18n.t(:contact_org_error)}: postalInfo > org [org]"
     }
   end
+
+  def fax_disabled
+    return true if ENV['fax_enabled'] == 'true'
+    return true if params[:parsed_frame].css('fax').text.blank?
+    epp_errors << {
+      code: '2306',
+      msg: "#{I18n.t(:contact_fax_error)}: fax [fax]"
+    }
+  end
 end

config/locales/en.yml

@@ -491,3 +491,4 @@ en:
   revoke_this_certificate: 'Revoke this certificate'
   crt_revoked: 'CRT (revoked)'
   contact_org_error: 'Parameter value policy error. Org should be blank'
+  contact_fax_error: 'Parameter value policy error. Fax should be blank'

spec/epp/contact_spec.rb

@@ -145,7 +145,7 @@ describe 'EPP Contact', epp: true do
         Contact.last.code.should == 'registrar1:12345'
       end
 
-      it 'should return parameter value policy errror' do
+      it 'should return parameter value policy error for org' do
         response = create_request({ postalInfo: { org: { value: 'should not save' } } })
         response[:msg].should == 
           'Parameter value policy error. Org should be blank: postalInfo > org [org]'
@@ -153,6 +153,15 @@ describe 'EPP Contact', epp: true do
 
         Contact.last.org_name.should == nil
       end
+
+      it 'should return parameter value policy error for fax' do
+        response = create_request({ fax: { value: 'should not save' } })
+        response[:msg].should == 
+          'Parameter value policy error. Fax should be blank: fax [fax]'
+        response[:result_code].should == '2306'
+
+        Contact.last.fax.should == nil
+      end
     end
 
     context 'update command' do
@@ -289,6 +298,20 @@ describe 'EPP Contact', epp: true do
 
         Contact.find_by(code: 'sh8013').org_name.should == nil
       end
+
+      it 'should return parameter value policy errror for fax update' do
+        response = update_request({ 
+          id: { value: 'sh8013' }, 
+          chg: {
+            fax: { value: 'should not save' } 
+          }
+        })
+        response[:msg].should == 
+          'Parameter value policy error. Fax should be blank: fax [fax]'
+        response[:result_code].should == '2306'
+
+        Contact.find_by(code: 'sh8013').fax.should == nil
+      end
     end
 
     context 'delete command' do