about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2017-01-03 00:44:23 +0100
committerEugen Rochko <eugen@zeonfederated.com>2017-01-03 00:44:23 +0100
commit86be6d48c9cf56cfe5e273169126729d097fe67e (patch)
tree3494fb24c9f6e1eb1de95f4d7ba915fe549736a7
parentf79ba2de83af5d9a1a94415c3ca38a7e7d6f2cfc (diff)
Fix accidental distribution of hidden stream entries to PuSH subscribers
-rw-r--r--app/workers/pubsubhubbub/distribution_worker.rb9
1 files changed, 6 insertions, 3 deletions
diff --git a/app/workers/pubsubhubbub/distribution_worker.rb b/app/workers/pubsubhubbub/distribution_worker.rb
index e88d11be0..d5437bf6b 100644
--- a/app/workers/pubsubhubbub/distribution_worker.rb
+++ b/app/workers/pubsubhubbub/distribution_worker.rb
@@ -7,9 +7,12 @@ class Pubsubhubbub::DistributionWorker
 
   def perform(stream_entry_id)
     stream_entry = StreamEntry.find(stream_entry_id)
-    account      = stream_entry.account
-    renderer     = AccountsController.renderer.new(method: 'get', http_host: Rails.configuration.x.local_domain, https: Rails.configuration.x.use_https)
-    payload      = renderer.render(:show, assigns: { account: account, entries: [stream_entry] }, formats: [:atom])
+
+    return if stream_entry.hidden?
+
+    account  = stream_entry.account
+    renderer = AccountsController.renderer.new(method: 'get', http_host: Rails.configuration.x.local_domain, https: Rails.configuration.x.use_https)
+    payload  = renderer.render(:show, assigns: { account: account, entries: [stream_entry] }, formats: [:atom])
 
     Subscription.where(account: account).active.select('id').find_each do |subscription|
       Pubsubhubbub::DeliveryWorker.perform_async(subscription.id, payload)