]> cat aescling's git repositories - mastodon.git/commit
Switch to static URIs, new URI format in both protocols for new statuses (#4815)
authorEugen Rochko <eugen@zeonfederated.com>
Wed, 6 Sep 2017 17:01:28 +0000 (19:01 +0200)
committerGitHub <noreply@github.com>
Wed, 6 Sep 2017 17:01:28 +0000 (19:01 +0200)
commite7adbf572a50f77590f889bb4d7cb9efb6fc7036
treea94e9833732205a3907491c0c9106f843aef281a
parent13ffa3c59e1a2727b287b2e6cde47f39c14ae815
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
15 files changed:
app/controllers/accounts_controller.rb
app/lib/ostatus/atom_serializer.rb
app/lib/tag_manager.rb
app/models/status.rb
app/workers/pubsubhubbub/distribution_worker.rb
db/migrate/20170905165803_add_local_to_statuses.rb [new file with mode: 0644]
db/schema.rb
spec/fabricators/status_fabricator.rb
spec/lib/activitypub/activity/delete_spec.rb
spec/lib/activitypub/activity/undo_spec.rb
spec/lib/formatter_spec.rb
spec/lib/ostatus/atom_serializer_spec.rb
spec/lib/tag_manager_spec.rb
spec/models/status_spec.rb
spec/services/fetch_link_card_service_spec.rb