mirror of
https://github.com/hanzoai/team.git
synced 2026-09-30 01:57:35 +00:00
* Tighten thread membership cleanup on team membership changes Ensure ThreadMembership rows are cleaned up when a user is removed from or leaves a team, add a defense-in-depth filter on the thread retrieval path so memberships referencing channels the user is no longer a member of are excluded, and add a one-time migration to clean up stale records on existing deployments. https://mattermost.atlassian.net/browse/MM-69008 * Centralize per-channel membership removal and tighten tests Extract the combined channel-member and thread-membership removal into a shared helper used by both removeUserFromChannel and LeaveTeam, so future code paths cannot revoke channel access without dropping the dependent thread state. The channel-leave event is now logged after the combined removal completes. Also drop verbose test header comments and rename a test to a behavior-focused name. * Backfill channel members in thread storetest setups The new ChannelMembers predicate on thread read queries filters out ThreadMembership rows whose user has no ChannelMembers row for the thread's channel. Several existing storetest setups bypassed the normal write path and inserted threads/memberships without channel members. Add the missing channel-member rows so the test data matches the real-world invariant. * Retrigger enterprise CI Pick up enterprise merge e6953d4 (master into MM-69008-thread-membership-team-leave) in the combined Enterprise CI/tests lane, which pins the enterprise SHA at mattermost-side dispatch time. Co-authored-by: Maria A Nunez <[email protected]> --------- Co-authored-by: Mattermost Build <[email protected]> Co-authored-by: Hanzo Dev <[email protected]>
14 lines
525 B
SQL
14 lines
525 B
SQL
-- Drop ThreadMembership rows whose user is no longer a member of the thread's channel.
|
|
DELETE FROM threadmemberships WHERE (postid, userid) IN (
|
|
SELECT
|
|
threadmemberships.postid,
|
|
threadmemberships.userid
|
|
FROM
|
|
threadmemberships
|
|
JOIN threads ON threads.postid = threadmemberships.postid
|
|
LEFT JOIN channelmembers ON channelmembers.userid = threadmemberships.userid
|
|
AND threads.channelid = channelmembers.channelid
|
|
WHERE
|
|
channelmembers.channelid IS NULL
|
|
);
|