Skip to content

Conversation

@pfefferle
Copy link
Member

@pfefferle pfefferle commented Mar 24, 2025

add_to_outbox only supports generic visibilities like for example ACTIVITYPUB_CONTENT_VISIBILITY_PRIVATE or 'ACTIVITYPUB_CONTENT_VISIBILITY_PUBLIC'. For custom audiences, we would have to add the to, cc, ... fields of the Activity. With the former implementation, we were only able to set the audience for the Activity>Object and these were up-merged to the Activity afterwards. In the case of an Accept of a Follow we were forced to update the Follow-Activity. But because the Follow was sent by a remote platform, adding a new field (to for example) would change the scope and could cause problems for remote verification.

With the rewrite to store the full Activity instead of the Object, we are now able to add the audience to this Activity and keep the Object untouched.

Proposed changes:

  • Do not rely on the Transformer
  • Create the full Activity
  • Add audience to Activity

Other information:

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

Testing instructions:

  • Send Follow Activity to inbox
  • Check Outbox for Accept-Activity

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

Properly set to audience of Activity instead of changing the Follow Object.

@pfefferle pfefferle added the [Type] Bug Something isn't working label Mar 24, 2025
@pfefferle pfefferle self-assigned this Mar 24, 2025
@pfefferle pfefferle changed the title Fix: Pixelfed Follow Fix: (Pixelfed) Follow Mar 25, 2025
@pfefferle pfefferle marked this pull request as ready for review March 25, 2025 09:40
@pfefferle pfefferle requested a review from a team March 25, 2025 09:40
obenland
obenland previously approved these changes Mar 25, 2025
@pfefferle pfefferle merged commit 62f7b48 into trunk Mar 25, 2025
11 checks passed
@pfefferle pfefferle deleted the fix/pixelfed-follow branch March 25, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants