diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2017-09-06 19:01:28 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-06 19:01:28 +0200 |
commit | e7adbf572a50f77590f889bb4d7cb9efb6fc7036 (patch) | |
tree | a94e9833732205a3907491c0c9106f843aef281a /app/workers/pubsubhubbub/distribution_worker.rb | |
parent | 13ffa3c59e1a2727b287b2e6cde47f39c14ae815 (diff) |
Switch to static URIs, new URI format in both protocols for new statuses (#4815)
* Decouple Status#local? from uri being nil
* Replace on-the-fly URI generation with stored URIs
- Generate URI in after_save hook for local statuses
- Use static value in TagManager when available, fallback to tag format
- Make TagManager use ActivityPub::TagManager to understand new format
- Adjust tests
* Use other heuristic for locality of old statuses, do not perform long query
* Exclude tombstone stream entries from Atom feed
* Prevent nil statuses from landing in Pubsubhubbub::DistributionWorker
* Fix URI not being saved (#4818)
* Add more specs for Status
* Save generated uri immediately
and also fix method order to minimize diff.
* Fix alternate HTML URL in Atom
* Fix tests
* Remove not-null constraint from statuses migration to speed it up
Diffstat (limited to 'app/workers/pubsubhubbub/distribution_worker.rb')
-rw-r--r-- | app/workers/pubsubhubbub/distribution_worker.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/workers/pubsubhubbub/distribution_worker.rb b/app/workers/pubsubhubbub/distribution_worker.rb index 2a5e60fa0e4..524f6849fca 100644 --- a/app/workers/pubsubhubbub/distribution_worker.rb +++ b/app/workers/pubsubhubbub/distribution_worker.rb @@ -6,7 +6,7 @@ class Pubsubhubbub::DistributionWorker sidekiq_options queue: 'push' def perform(stream_entry_ids) - stream_entries = StreamEntry.where(id: stream_entry_ids).includes(:status).reject { |e| e.status&.direct_visibility? } + stream_entries = StreamEntry.where(id: stream_entry_ids).includes(:status).reject { |e| e.status.nil? || e.status.direct_visibility? } return if stream_entries.empty? |