Merge branch 'neilalexander/federationinput2' into neilalexander/federationinput

clientapi/routing/createroom.go

@@ -459,7 +459,6 @@ func createRoom(
 			Kind:         roomserverAPI.KindNew,
 			Event:        event,
 			Origin:       cfg.Matrix.ServerName,
-			AuthEventIDs: event.AuthEventIDs(),
 			SendAsServer: roomserverAPI.DoNotSendToOtherServers,
 		})
 	}

federationapi/routing/join.go

@@ -324,7 +324,6 @@ func SendJoin(
 				{
 					Kind:          api.KindNew,
 					Event:         event.Headered(stateAndAuthChainResponse.RoomVersion),
-					AuthEventIDs:  event.AuthEventIDs(),
 					SendAsServer:  string(cfg.Matrix.ServerName),
 					TransactionID: nil,
 				},

federationapi/routing/leave.go

@@ -275,7 +275,6 @@ func SendLeave(
 			{
 				Kind:          api.KindNew,
 				Event:         event.Headered(verRes.RoomVersion),
-				AuthEventIDs:  event.AuthEventIDs(),
 				SendAsServer:  string(cfg.Matrix.ServerName),
 				TransactionID: nil,
 			},

roomserver/api/input.go

@@ -56,12 +56,6 @@ type InputRoomEvent struct {
 	Event *gomatrixserverlib.HeaderedEvent `json:"event"`
 	// Which server told us about this event.
 	Origin gomatrixserverlib.ServerName `json:"origin"`
-	// List of state event IDs that authenticate this event.
-	// These are likely derived from the "auth_events" JSON key of the event.
-	// But can be different because the "auth_events" key can be incomplete or wrong.
-	// For example many matrix events forget to reference the m.room.create event even though it is needed for auth.
-	// (since synapse allows this to happen we have to allow it as well.)
-	AuthEventIDs []string `json:"auth_event_ids"`
 	// Whether the state is supplied as a list of event IDs or whether it
 	// should be derived from the state at the previous events.
 	HasState bool `json:"has_state"`

roomserver/api/wrapper.go

@@ -36,7 +36,6 @@ func SendEvents(
 			Kind:          kind,
 			Event:         event,
 			Origin:        origin,
-			AuthEventIDs:  event.AuthEventIDs(),
 			SendAsServer:  string(sendAsServer),
 			TransactionID: txnID,
 		}
@@ -66,7 +65,6 @@ func SendEventWithState(
 			Kind:   KindOutlier,
 			Event:  outlier.Headered(event.RoomVersion),
 			Origin: origin,
-			AuthEventIDs: outlier.AuthEventIDs(),
 		})
 	}
 
@@ -79,7 +77,6 @@ func SendEventWithState(
 		Kind:          kind,
 		Event:         event,
 		Origin:        origin,
-		AuthEventIDs:  event.AuthEventIDs(),
 		HasState:      true,
 		StateEventIDs: stateEventIDs,
 	})

roomserver/internal/input/input_missing.go

@@ -180,7 +180,6 @@ func (t *missingStateReq) processEventWithMissingState(
 			Kind:   api.KindOutlier,
 			Event:  outlier.Headered(roomVersion),
 			Origin: t.origin,
-			AuthEventIDs: outlier.AuthEventIDs(),
 		})
 	}
 	// TODO: we could do this concurrently?
@@ -202,7 +201,6 @@ func (t *missingStateReq) processEventWithMissingState(
 		Kind:          api.KindOld,
 		Event:         backwardsExtremity.Headered(roomVersion),
 		Origin:        t.origin,
-		AuthEventIDs:  backwardsExtremity.AuthEventIDs(),
 		HasState:      true,
 		StateEventIDs: stateIDs,
 		SendAsServer:  api.DoNotSendToOtherServers,
@@ -220,7 +218,6 @@ func (t *missingStateReq) processEventWithMissingState(
 			Kind:         api.KindOld,
 			Event:        newEvent.Headered(roomVersion),
 			Origin:       t.origin,
-			AuthEventIDs: newEvent.AuthEventIDs(),
 			SendAsServer: api.DoNotSendToOtherServers,
 		})
 		if err != nil {

roomserver/internal/perform/perform_invite.go

@@ -173,7 +173,6 @@ func (r *Inviter) PerformInvite(
 					Kind:         api.KindNew,
 					Event:        event,
 					Origin:       event.Origin(),
-					AuthEventIDs: event.AuthEventIDs(),
 					SendAsServer: req.SendAsServer,
 				},
 			},

roomserver/internal/perform/perform_join.go

@@ -271,7 +271,6 @@ func (r *Joiner) performJoinRoomByID(
 					{
 						Kind:         rsAPI.KindNew,
 						Event:        event.Headered(buildRes.RoomVersion),
-						AuthEventIDs: event.AuthEventIDs(),
 						SendAsServer: string(r.Cfg.Matrix.ServerName),
 					},
 				},

roomserver/internal/perform/perform_leave.go

@@ -140,7 +140,6 @@ func (r *Leaver) performLeaveRoomByID(
 				Kind:         api.KindNew,
 				Event:        event.Headered(buildRes.RoomVersion),
 				Origin:       event.Origin(),
-				AuthEventIDs: event.AuthEventIDs(),
 				SendAsServer: string(r.Cfg.Matrix.ServerName),
 			},
 		},

roomserver/storage/postgres/events_table.go

@@ -311,7 +311,9 @@ func (s *eventStatements) BulkSelectStateAtEventByID(
 		); err != nil {
 			return nil, err
 		}
-		if result.BeforeStateSnapshotNID == 0 && result.EventTypeNID != types.MRoomCreateNID {
+		// Genuine create events are the only case where it's OK to have no previous state.
+		isCreate := result.EventTypeNID == types.MRoomCreateNID && result.EventStateKeyNID == 1
+		if result.BeforeStateSnapshotNID == 0 && !isCreate {
 			return nil, types.MissingEventError(
 				fmt.Sprintf("storage: missing state for event NID %d", result.EventNID),
 			)

roomserver/storage/sqlite3/events_table.go

@@ -322,7 +322,9 @@ func (s *eventStatements) BulkSelectStateAtEventByID(
 		); err != nil {
 			return nil, err
 		}
-		if result.BeforeStateSnapshotNID == 0 && result.EventTypeNID != types.MRoomCreateNID {
+		// Genuine create events are the only case where it's OK to have no previous state.
+		isCreate := result.EventTypeNID == types.MRoomCreateNID && result.EventStateKeyNID == 1
+		if result.BeforeStateSnapshotNID == 0 && !isCreate {
 			return nil, types.MissingEventError(
 				fmt.Sprintf("storage: missing state for event NID %d", result.EventNID),
 			)

setup/mscs/msc2836/msc2836.go

@@ -645,7 +645,6 @@ func (rc *reqCtx) injectResponseToRoomserver(res *gomatrixserverlib.MSC2836Event
 		ires = append(ires, roomserver.InputRoomEvent{
 			Kind:  roomserver.KindOutlier,
 			Event: outlier.Headered(outlier.Version()),
-			AuthEventIDs: outlier.AuthEventIDs(),
 		})
 	}
 	// we've got the data by this point so use a background context