diff options
author | Erik Johnston <erik@matrix.org> | 2016-07-14 16:49:37 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2016-07-14 17:03:24 +0100 |
commit | ebdafd8114d1aed631a3497ad142f79efa9face7 (patch) | |
tree | be0326483c80d7f9e992ffbedd9b939f3ca56fdb /synapse/api/auth.py | |
parent | Check if alias event's state_key matches sender's domain (diff) | |
download | synapse-ebdafd8114d1aed631a3497ad142f79efa9face7.tar.xz |
Check sender signed event
Diffstat (limited to 'synapse/api/auth.py')
-rw-r--r-- | synapse/api/auth.py | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/synapse/api/auth.py b/synapse/api/auth.py index e05defd7d8..e2f40ee65a 100644 --- a/synapse/api/auth.py +++ b/synapse/api/auth.py @@ -63,7 +63,7 @@ class Auth(object): "user_id = ", ]) - def check(self, event, auth_events): + def check(self, event, auth_events, do_sig_check=True): """ Checks if this event is correctly authed. Args: @@ -79,6 +79,13 @@ class Auth(object): if not hasattr(event, "room_id"): raise AuthError(500, "Event has no room_id: %s" % event) + + sender_domain = get_domain_from_id(event.sender) + + # Check the sender's domain has signed the event + if do_sig_check and not event.signatures.get(sender_domain): + raise AuthError(403, "Event not signed by sending server") + if auth_events is None: # Oh, we don't know what the state of the room was, so we # are trusting that this is allowed (at least for now) @@ -87,7 +94,6 @@ class Auth(object): if event.type == EventTypes.Create: room_id_domain = get_domain_from_id(event.room_id) - sender_domain = get_domain_from_id(event.sender) if room_id_domain != sender_domain: raise AuthError( 403, |