0
0
Fork 0
mirror of https://github.com/matrix-construct/construct synced 2024-06-02 10:08:56 +02:00

ircd:Ⓜ️ Fix an erroneous conforms check for version 3/4 event_id's.

This commit is contained in:
Jason Volk 2019-08-31 15:36:58 -07:00
parent 5c591582a2
commit a9984d8b7a

View file

@ -401,9 +401,6 @@ ircd::m::vm::conform_check_event_id
|| eval.room_version == "2"
};
if(unaffected)
return;
if(eval.room_version == "3")
if(!event::id::v3::is(event.event_id))
throw error
@ -412,13 +409,15 @@ ircd::m::vm::conform_check_event_id
string_view{event.event_id}
};
if(!event::id::v4::is(event.event_id))
throw error
{
fault::INVALID, "Event ID %s is not sufficient for version %s room",
string_view{event.event_id},
eval.room_version,
};
// note: we check v4 format for all other room versions, including "4"
if(!unaffected && eval.room_version != "3")
if(!event::id::v4::is(event.event_id))
throw error
{
fault::INVALID, "Event ID %s in a version %s room is not a version 4 Event ID.",
string_view{event.event_id},
eval.room_version,
};
}
};