about summary refs log tree commit diff
path: root/app/services
diff options
context:
space:
mode:
authorysksn <bluewhale1982@gmail.com>2018-12-12 13:54:34 +0900
committerEugen Rochko <eugen@zeonfederated.com>2018-12-12 05:54:34 +0100
commit83975a2394ba6f9a6b8e6612187cb5546d4108ec (patch)
tree8f3e281a92d32861c86f9af71d966d0ccfbc033d /app/services
parent351ed12b2f0314f7dd6c31ef018d84ca2e48ad04 (diff)
Remove redundant (#9503)
Diffstat (limited to 'app/services')
-rw-r--r--app/services/pubsubhubbub/subscribe_service.rb21
1 files changed, 4 insertions, 17 deletions
diff --git a/app/services/pubsubhubbub/subscribe_service.rb b/app/services/pubsubhubbub/subscribe_service.rb
index d2b85b85b..550da6328 100644
--- a/app/services/pubsubhubbub/subscribe_service.rb
+++ b/app/services/pubsubhubbub/subscribe_service.rb
@@ -19,31 +19,18 @@ class Pubsubhubbub::SubscribeService < BaseService
   private
 
   def process_subscribe
-    case subscribe_status
-    when :invalid_topic
+    if account.nil?
       ['Invalid topic URL', 422]
-    when :invalid_callback
+    elsif !valid_callback?
       ['Invalid callback URL', 422]
-    when :callback_not_allowed
+    elsif blocked_domain?
       ['Callback URL not allowed', 403]
-    when :valid
+    else
       confirm_subscription
       ['', 202]
     end
   end
 
-  def subscribe_status
-    if account.nil?
-      :invalid_topic
-    elsif !valid_callback?
-      :invalid_callback
-    elsif blocked_domain?
-      :callback_not_allowed
-    else
-      :valid
-    end
-  end
-
   def confirm_subscription
     subscription = locate_subscription
     Pubsubhubbub::ConfirmationWorker.perform_async(subscription.id, 'subscribe', secret, lease_seconds)