Changelog

Generally speaking one should look up an event by its id. Otherwise
we'll fall back to `object.__hash__` (the instance's address, under
CPython) which is probably not what you want.

Noticed in #15240.

changelog.d/15305.misc

@@ -0,0 +1 @@
+Make EventBase unhashable.

synapse/events/__init__.py

@@ -24,6 +24,7 @@ from typing import (
     Generic,
     Iterable,
     List,
+    NoReturn,
     Optional,
     Sequence,
     Tuple,
@@ -338,6 +339,9 @@ class EventBase(metaclass=abc.ABCMeta):
     type: DictProperty[str] = DictProperty("type")
     user_id: DictProperty[str] = DictProperty("sender")
 
+    def __hash__(self) -> NoReturn:
+        raise NotImplementedError()
+
     @property
     def event_id(self) -> str:
         raise NotImplementedError()