diff --git a/synapse/handlers/message.py b/synapse/handlers/message.py
index 42dc4d46f3..baf372fdad 100644
--- a/synapse/handlers/message.py
+++ b/synapse/handlers/message.py
@@ -15,10 +15,13 @@
from twisted.internet import defer
-from synapse.api.constants import Membership
+from synapse.api.constants import EventTypes, Membership
from synapse.api.errors import RoomError
from synapse.streams.config import PaginationConfig
from synapse.util.logcontext import PreserveLoggingContext
+
+from synapse.events.validator import EventValidator
+
from ._base import BaseHandler
import logging
@@ -32,7 +35,7 @@ class MessageHandler(BaseHandler):
super(MessageHandler, self).__init__(hs)
self.hs = hs
self.clock = hs.get_clock()
- self.event_factory = hs.get_event_factory()
+ self.validator = EventValidator()
@defer.inlineCallbacks
def get_message(self, msg_id=None, room_id=None, sender_id=None,
@@ -79,7 +82,7 @@ class MessageHandler(BaseHandler):
self.ratelimit(event.user_id)
# TODO(paul): Why does 'event' not have a 'user' object?
user = self.hs.parse_userid(event.user_id)
- assert user.is_mine, "User must be our own: %s" % (user,)
+ assert self.hs.is_mine(user), "User must be our own: %s" % (user,)
snapshot = yield self.store.snapshot_room(event)
@@ -134,19 +137,48 @@ class MessageHandler(BaseHandler):
defer.returnValue(chunk)
@defer.inlineCallbacks
- def store_room_data(self, event=None):
- """ Stores data for a room.
+ def create_and_send_event(self, event_dict):
+ """ Given a dict from a client, create and handle a new event.
+
+ Creates an FrozenEvent object, filling out auth_events, prev_events,
+ etc.
+
+ Adds display names to Join membership events.
+
+ Persists and notifies local clients and federation.
Args:
- event : The room path event
- stamp_event (bool) : True to stamp event content with server keys.
- Raises:
- SynapseError if something went wrong.
+ event_dict (dict): An entire event
"""
+ builder = self.event_builder_factory.new(event_dict)
+
+ self.validator.validate_new(builder)
+
+ if builder.type == EventTypes.Member:
+ membership = builder.content.get("membership", None)
+ if membership == Membership.JOIN:
+ joinee = self.hs.parse_userid(builder.state_key)
+ # If event doesn't include a display name, add one.
+ yield self.distributor.fire(
+ "collect_presencelike_data",
+ joinee,
+ builder.content
+ )
- snapshot = yield self.store.snapshot_room(event)
+ event, context = yield self._create_new_client_event(
+ builder=builder,
+ )
- yield self._on_new_room_event(event, snapshot)
+ if event.type == EventTypes.Member:
+ member_handler = self.hs.get_handlers().room_member_handler
+ yield member_handler.change_membership(event, context)
+ else:
+ yield self.handle_new_client_event(
+ event=event,
+ context=context,
+ )
+
+ defer.returnValue(event)
@defer.inlineCallbacks
def get_room_data(self, user_id=None, room_id=None,
|