Refactor

app/controllers/concerns/epp/common.rb

@@ -0,0 +1,15 @@
+module Epp::Common
+  extend ActiveSupport::Concern
+
+  included do
+    protect_from_forgery with: :null_session
+  end
+
+  def proxy
+    send(params[:command])
+  end
+
+  def parsed_frame
+    Nokogiri::XML(params[:frame]).remove_namespaces!
+  end
+end

app/controllers/epp/sessions_controller.rb

@@ -1,9 +1,6 @@
 class Epp::SessionsController < ApplicationController
-  protect_from_forgery with: :null_session
+  include Epp::Common
-
+  include Epp::SessionsHelper
-  def proxy
-    send(params[:command])
-  end
 
   private
   def hello
@@ -11,11 +8,7 @@ class Epp::SessionsController < ApplicationController
   end
 
   def login
-    login_params = parsed_frame.css('epp command login')
+    @epp_user = EppUser.find_by(login_params)
-    username = login_params.css('clID').text
-    password = login_params.css('pw').text
-
-    @epp_user = EppUser.find_by(username: username, password: password)
 
     if @epp_user.try(:active)
       render 'login_success'
@@ -24,8 +17,4 @@ class Epp::SessionsController < ApplicationController
       render 'login_fail'
     end
   end
-
-  def parsed_frame
-    Nokogiri::XML(params[:frame]).remove_namespaces!
-  end
 end

app/helpers/epp/sessions_helper.rb

@@ -0,0 +1,6 @@
+module Epp::SessionsHelper
+  def login_params
+    login_params = parsed_frame.css('epp command login')
+    { username: login_params.css('clID').text, password: login_params.css('pw').text }
+  end
+end