about summary refs log tree commit diff
path: root/app/services/unsubscribe_service.rb
diff options
context:
space:
mode:
authorDavid Yip <yipdw@member.fsf.org>2017-09-09 23:56:21 -0500
committerDavid Yip <yipdw@member.fsf.org>2017-09-09 23:56:21 -0500
commit67ad4533732f2e5cfc8c7f7ad3abaf7a5eb2647b (patch)
tree011ea44fc94bcff6f8ec4e23c3edf887359243d2 /app/services/unsubscribe_service.rb
parent3dff74eecf5387b92b862893248710d2efb90eec (diff)
parent90712d42933efd9978e4bbae82f81a4650aa4d84 (diff)
Merge tag 'v1.6.0rc4' into sync/upstream-1.6.0rc4
Conflicts:
      app/javascript/mastodon/features/getting_started/index.js
      app/javascript/packs/public.js
      app/javascript/styles/components.scss
Diffstat (limited to 'app/services/unsubscribe_service.rb')
-rw-r--r--app/services/unsubscribe_service.rb13
1 files changed, 8 insertions, 5 deletions
diff --git a/app/services/unsubscribe_service.rb b/app/services/unsubscribe_service.rb
index 865f783bc..b99046712 100644
--- a/app/services/unsubscribe_service.rb
+++ b/app/services/unsubscribe_service.rb
@@ -4,16 +4,19 @@ class UnsubscribeService < BaseService
   def call(account)
     return if account.hub_url.blank?
 
-    @account  = account
-    @response = build_request.perform
+    @account = account
 
-    Rails.logger.debug "PuSH unsubscribe for #{@account.acct} failed: #{@response.status}" unless @response.status.success?
+    begin
+      @response = build_request.perform
+
+      Rails.logger.debug "PuSH unsubscribe for #{@account.acct} failed: #{@response.status}" unless @response.status.success?
+    rescue HTTP::Error, OpenSSL::SSL::SSLError => e
+      Rails.logger.debug "PuSH unsubscribe for #{@account.acct} failed: #{e}"
+    end
 
     @account.secret = ''
     @account.subscription_expires_at = nil
     @account.save!
-  rescue HTTP::Error, OpenSSL::SSL::SSLError
-    Rails.logger.debug "PuSH subscription request for #{@account.acct} could not be made due to HTTP or SSL error"
   end
 
   private