Skip to content

Fix support for image property on Page type, add support for string#2111

Merged
BentiGorlich merged 2 commits intomainfrom
fix/page-image-string
Apr 23, 2026
Merged

Fix support for image property on Page type, add support for string#2111
BentiGorlich merged 2 commits intomainfrom
fix/page-image-string

Conversation

@BentiGorlich
Copy link
Copy Markdown
Member

  • When a Page ActivityPub object contains only an image property, but no attachment property our code threw an error
  • Add support for either attachment or image property to contain only a string (Ghost does this for example)

With this change an AP-Object like this: https://www.shoptechblog.com/.ghost/activitypub/article/38919605-5b90-4b0b-ad3c-5991a889855c
will properly work and get the image attached

Also connection to #2107

- When a `Page` ActivityPub object contains only an `image` property, but no `attachment` property our code threw an error
- Add support for either `attachment` or `image` property to contain only a string (Ghost does this for example)
@BentiGorlich BentiGorlich self-assigned this Apr 23, 2026
@BentiGorlich BentiGorlich added bug Something isn't working enhancement New feature or request activitypub ActivityPub related issues labels Apr 23, 2026
@BentiGorlich BentiGorlich linked an issue Apr 23, 2026 that may be closed by this pull request
Comment thread src/Service/ActivityPub/Page.php
@BentiGorlich BentiGorlich enabled auto-merge (squash) April 23, 2026 14:17
@BentiGorlich BentiGorlich merged commit 5d6269d into main Apr 23, 2026
9 checks passed
@BentiGorlich BentiGorlich deleted the fix/page-image-string branch April 23, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

activitypub ActivityPub related issues bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: ActivityPub processing fails with as:Public and string to/cc

2 participants