Ensure is_verified on /_matrix/client/r0/room_keys/keys is a boolean (#7150)

This commit is contained in:
Andrew Morgan 2020-03-27 13:30:22 +00:00 committed by GitHub
parent fbf0782c63
commit 12aa5a7fa7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 2 deletions

1
changelog.d/7150.bugfix Normal file
View file

@ -0,0 +1 @@
Ensure `is_verified` is a boolean in responses to `GET /_matrix/client/r0/room_keys/keys`. Also warn the user if they forgot the `version` query param.

View file

@ -188,7 +188,7 @@ class RoomKeysServlet(RestServlet):
""" """
requester = await self.auth.get_user_by_req(request, allow_guest=False) requester = await self.auth.get_user_by_req(request, allow_guest=False)
user_id = requester.user.to_string() user_id = requester.user.to_string()
version = parse_string(request, "version") version = parse_string(request, "version", required=True)
room_keys = await self.e2e_room_keys_handler.get_room_keys( room_keys = await self.e2e_room_keys_handler.get_room_keys(
user_id, version, room_id, session_id user_id, version, room_id, session_id

View file

@ -146,7 +146,8 @@ class EndToEndRoomKeyStore(SQLBaseStore):
room_entry["sessions"][row["session_id"]] = { room_entry["sessions"][row["session_id"]] = {
"first_message_index": row["first_message_index"], "first_message_index": row["first_message_index"],
"forwarded_count": row["forwarded_count"], "forwarded_count": row["forwarded_count"],
"is_verified": row["is_verified"], # is_verified must be returned to the client as a boolean
"is_verified": bool(row["is_verified"]),
"session_data": json.loads(row["session_data"]), "session_data": json.loads(row["session_data"]),
} }