Skip to content

Keep EMA unread status out of public searches - #306

Merged
akirk merged 1 commit into
mainfrom
fix-private-ema-unread-status
Jul 7, 2026
Merged

Keep EMA unread status out of public searches#306
akirk merged 1 commit into
mainfrom
fix-private-ema-unread-status

Conversation

@akirk

@akirk akirk commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • register the EMA unread DM status as non-public, matching the read status
  • add a regression test that normal WordPress public search SQL does not include ema_unread

Context

Issue #279 shows a normal WordPress front-end search query, not a Mastodon API request. The query included post_status = ema_unread because EMA registered that status as public.

Fixes #279.

Tests

  • composer test -- --filter PublicSearch_Test
  • composer test -- --filter ConversationsEndpoint_Test
  • composer test -- --filter "PublicSearch_Test|ConversationsEndpoint_Test|NotificationsEndpoint_Test"

Note: these focused tests passed before opening the PR. A later accidental shell execution while creating the PR body reran the conversation-related tests and hit an environment-specific OpenSSL random state write error from the ActivityPub plugin.

@akirk
akirk merged commit b0717e0 into main Jul 7, 2026
18 checks passed
@akirk
akirk deleted the fix-private-ema-unread-status branch July 7, 2026 07:23
@akirk akirk mentioned this pull request Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spam?

1 participant