Commit Graph

22 Commits (d446a1bac4bd737a5af1bf1a69716770dbfb8e8c)

Author SHA1 Message Date
dansup 03e64efd68
Merge pull request #6231 from shleeable/JobBatchv1-StatusPipeline
JobBatch v1- Status pipelines defensive checks
2 weeks ago
Your Name efb9db3eed StatusPipeline defensive checks 2 weeks ago
Shlee f8fbf3a9d5
Bugfix: Regression in NewStatusPipeline.php 2 weeks ago
dansup 0b5ce910b8
Merge branch 'staging' into fix-delay-publish-until-media-processed 2 weeks ago
Anil Kulkarni 8737619b46
Revert "Update NewStatusPipeline, replaces #5706"
This reverts commit 6c13debd56.
4 weeks ago
Anil Kulkarni 5c6e3b9732
Revert "Update NewStatusPipeline, improve fallback"
This reverts commit 863ad8e107.
4 weeks ago
Anil Kulkarni dbd2e17728
Ensure the cloud url is used when publishing a status to activitypub
Posting is done in two steps - first the media is uploaded and that begins a processing pipeline. When the media is finally
processed, the media pipeline sets the cdn_url to the final image (in the cloud if configured).
At any time, the user can click post. This begins a separate NewStatusPipeline to deliver the status over ActivityPub.
This causes a race. If the NewStatusPipeline runs before the media finishes processing, then the status is delivered with
the un-processed (and un-optimized images). If pixelfed is set to use cloud storage, then the ActivityPub message also incorrectly
uses the local media.

This commit fixes the race condition by waiting for all the media to be processed before sending the message over ActivityPub.
A new publish_delayed column is set per-post, indicating whether the
status should be delayed until all media is processed.

Then, the NewStatusPipeline is called twice per post -
When the media finishes processing and when the status is posted.

Both times, the pipeline will check to see if it's valid to post
If so, the pipeline will try to set publish_delayed to false
and only if it is the first time doing so, will it publish the post
4 weeks ago
Anil Kulkarni d819012439
Revert "Update NewStatusPipeline, replaces #5706"
This reverts commit 6c13debd56.
4 weeks ago
Anil Kulkarni 1a245914d8
Revert "Update NewStatusPipeline, improve fallback"
This reverts commit 863ad8e107.
4 weeks ago
Daniel Supernault 863ad8e107
Update NewStatusPipeline, improve fallback 9 months ago
Daniel Supernault 6c13debd56
Update NewStatusPipeline, replaces #5706 9 months ago
Daniel Supernault 6476855519
Update jobs, add retry and timeout attributes 6 years ago
Daniel Supernault c369882b48
Update Redis namespace 6 years ago
Daniel Supernault 7295b1b887
Update StatusPipeline 7 years ago
Daniel Supernault 8edeab11d6
Update StatusPipeline 7 years ago
Daniel Supernault 5b68d147d2
Update StatusPipeline, only deliver if ap is enabled 7 years ago
Daniel Supernault 8235589276
Update StatusPipeline 7 years ago
Daniel Supernault 071feae1d0
Update Status Pipeline 7 years ago
Daniel Supernault 6001039e94
Update NewStatusPipeline job 7 years ago
daniel b8abbdd90f Apply fixes from StyleCI 7 years ago
Daniel Supernault 3f0ad3ffe2 Update StatusController, support multiple uploads 8 years ago
Daniel Supernault ec2bff95b6 Add StatusPipeline 8 years ago