Skip to content

Fix: Missing Actor in Outbox-Activities #1564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 10, 2025
Merged

Conversation

pfefferle
Copy link
Member

@pfefferle pfefferle commented Apr 10, 2025

In some cases the Actor of an Activity will not properly set and we wrongly set the inReplyTo for the Activity.

These fixes are cherry picked from #1562

Proposed changes:

  • Add Actor before storing the Activity in the Outbox
  • Add Actor before federating the Activity (for backwards compatibility)
  • Remove reply-to from Activity context

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

  • Publish a post
  • Check outbox item if Activity has an Actor

Changelog entry

  • Automatically create a changelog entry from the details below.
Changelog Entry Details

Significance

  • Patch
  • Minor
  • Major

Type

  • Added - for new features
  • Changed - for changes in existing functionality
  • Deprecated - for soon-to-be removed features
  • Removed - for now removed features
  • Fixed - for any bug fixes
  • Security - in case of vulnerabilities

Message

Ensure that an Activity is properly built before adding it to the Outbox.

Copy link
Member

@obenland obenland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In some cases the Actor of an Activity will not properly set.

Do we know which cases those are?

@pfefferle
Copy link
Member Author

pfefferle commented Apr 10, 2025

If you use the add_to_outbox() and let the function build the Activity, like with add_to_outbox( $activity, 'Announce', $id ).

@pfefferle pfefferle merged commit 1dbcc8e into trunk Apr 10, 2025
11 checks passed
@pfefferle pfefferle deleted the fix/outbox-activities branch April 10, 2025 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants