Don't add rejections to the state_group, persist all rejections

synapse/storage/events.py

@@ -591,10 +591,11 @@ class EventsStore(SQLBaseStore):
             ],
         )
 
-        if context.rejected:
-            self._store_rejections_txn(
-                txn, event.event_id, context.rejected
-            )
+        for event, context in events_and_contexts:
+            if context.rejected:
+                self._store_rejections_txn(
+                    txn, event.event_id, context.rejected
+                )
 
         self._simple_insert_many_txn(
             txn,

synapse/storage/state.py

@@ -79,7 +79,7 @@ class StateStore(SQLBaseStore):
 
             state_events = dict(context.current_state)
 
-            if event.is_state():
+            if event.is_state() and not context.rejected:
                 state_events[(event.type, event.state_key)] = event
 
             state_group = context.new_state_group_id