diff --git a/.reek b/.reek index e51946a9a..d9ea95353 100644 --- a/.reek +++ b/.reek @@ -34,7 +34,6 @@ UncommunicativeVariableName: - Admin::SettingsController#create - Epp::DomainsController#renew - Epp::DomainsController#update - - Epp::SessionsController#connection_limit_ok? - Epp::SessionsController#login - EppController - EppController#create_full_selectors @@ -172,7 +171,6 @@ DuplicateMethodCall: - Epp::PollsController#ack_poll - Epp::PollsController#poll - Epp::PollsController#req_poll - - Epp::SessionsController#connection_limit_ok? - Epp::SessionsController#ip_white? - Epp::SessionsController#login - Epp::SessionsController#login_params diff --git a/app/controllers/epp/sessions_controller.rb b/app/controllers/epp/sessions_controller.rb index b9884b582..eb460a8d1 100644 --- a/app/controllers/epp/sessions_controller.rb +++ b/app/controllers/epp/sessions_controller.rb @@ -115,14 +115,6 @@ class Epp::SessionsController < EppController true end - def connection_limit_ok? - epp_session_count = EppSession.where(user_id: @api_user.registrar.api_users.ids) - .where('updated_at >= ?', Time.zone.now - 1.second).count - - return false if epp_session_count >= 4 - true - end - def logout unless signed_in? epp_errors << { @@ -151,4 +143,12 @@ class Epp::SessionsController < EppController def resource @api_user end + + def connection_limit_ok? + epp_session_count = EppSession.where(user_id: @api_user.registrar.api_users.ids) + .where('updated_at >= ?', Time.zone.now - 1.second).count + + return false if epp_session_count >= 4 + true + end end diff --git a/doc/controllers_complete.svg b/doc/controllers_complete.svg index 9a0028bb1..68d02cf4a 100644 --- a/doc/controllers_complete.svg +++ b/doc/controllers_complete.svg @@ -71,7 +71,6 @@ Epp::SessionsController -connection_limit_ok? hello ip_white? login