Skip to content

Commit

Permalink
Merge branch 'fix-worker-batch-send' into 'master'
Browse files Browse the repository at this point in the history
Fix using MSC2716 batch sending with event persistence workers

See merge request beeper/synapse!9
  • Loading branch information
Fizzadar committed Nov 3, 2021
2 parents e361394 + c5bf223 commit 04ed203
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 13 deletions.
1 change: 1 addition & 0 deletions changelog.d/11220.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix using MSC2716 batch sending in combination with event persistence workers. Contributed by @tulir at Beeper.
2 changes: 2 additions & 0 deletions synapse/app/generic_worker.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@
)
from synapse.storage.databases.main.presence import PresenceStore
from synapse.storage.databases.main.room import RoomWorkerStore
from synapse.storage.databases.main.room_batch import RoomBatchStore
from synapse.storage.databases.main.search import SearchStore
from synapse.storage.databases.main.session import SessionStore
from synapse.storage.databases.main.stats import StatsStore
Expand Down Expand Up @@ -237,6 +238,7 @@ class GenericWorkerSlavedStore(
SlavedEventStore,
SlavedKeyStore,
RoomWorkerStore,
RoomBatchStore,
DirectoryStore,
SlavedApplicationServiceStore,
SlavedRegistrationStore,
Expand Down
28 changes: 15 additions & 13 deletions synapse/handlers/message.py
Original file line number Diff line number Diff line change
Expand Up @@ -1371,13 +1371,14 @@ async def persist_and_notify_client_event(

if event.type == EventTypes.Member:
if event.content["membership"] == Membership.INVITE:
event.unsigned[
"invite_room_state"
] = await self.store.get_stripped_room_state_from_event_context(
context,
self.room_prejoin_state_types,
membership_user_id=event.sender,
)
if not event.internal_metadata.outlier:
event.unsigned[
"invite_room_state"
] = await self.store.get_stripped_room_state_from_event_context(
context,
self.room_prejoin_state_types,
membership_user_id=event.sender,
)

invitee = UserID.from_string(event.state_key)
if not self.hs.is_mine(invitee):
Expand All @@ -1394,12 +1395,13 @@ async def persist_and_notify_client_event(
event.signatures.update(returned_invite.signatures)

if event.content["membership"] == Membership.KNOCK:
event.unsigned[
"knock_room_state"
] = await self.store.get_stripped_room_state_from_event_context(
context,
self.room_prejoin_state_types,
)
if not event.internal_metadata.outlier:
event.unsigned[
"knock_room_state"
] = await self.store.get_stripped_room_state_from_event_context(
context,
self.room_prejoin_state_types,
)

if event.type == EventTypes.Redaction:
original_event = await self.store.get_event(
Expand Down

0 comments on commit 04ed203

Please sign in to comment.