Use NewEventFromTrustedJSON when getting events from the database

This commit is contained in:
Neil Alexander 2020-03-25 16:54:55 +00:00
parent b54d33a3e8
commit f319de1fbe
2 changed files with 4 additions and 6 deletions

View file

@ -266,9 +266,8 @@ func (d *Database) Events(
if err != nil {
return nil, err
}
// TODO: Use NewEventFromTrustedJSON for efficiency
result.Event, err = gomatrixserverlib.NewEventFromUntrustedJSON(
eventJSON.EventJSON, roomVersion,
result.Event, err = gomatrixserverlib.NewEventFromTrustedJSON(
eventJSON.EventJSON, false, roomVersion,
)
if err != nil {
return nil, err

View file

@ -311,7 +311,6 @@ func (d *Database) Events(
var roomVersion gomatrixserverlib.RoomVersion
result := &results[i]
result.EventNID = eventJSON.EventNID
// TODO: Use NewEventFromTrustedJSON for efficiency
roomNID, err = d.statements.selectRoomNIDForEventNID(ctx, txn, eventJSON.EventNID)
if err != nil {
return err
@ -320,8 +319,8 @@ func (d *Database) Events(
if err != nil {
return err
}
result.Event, err = gomatrixserverlib.NewEventFromUntrustedJSON(
eventJSON.EventJSON, roomVersion,
result.Event, err = gomatrixserverlib.NewEventFromTrustedJSON(
eventJSON.EventJSON, false, roomVersion,
)
if err != nil {
return nil