Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Stop depending on room_id to be returned for children state in the …
Browse files Browse the repository at this point in the history
…hierarchy response. (#12991)

The `room_id` field was removed from MSC2946 before
it was accepted. It was initially kept for backwards compatibility
and should be removed now that the stable form of the API
is used.

This change only stops Synapse from validating that it is returned,
a future PR will remove returning it as part of the response.
  • Loading branch information
clokep authored Jun 10, 2022
1 parent 3d1d510 commit 8160849
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
2 changes: 2 additions & 0 deletions changelog.d/12991.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Fix a bug where non-standard information was required when requesting the `/hierarchy` API over federation. Introduced
in Synapse v1.41.0.
4 changes: 0 additions & 4 deletions synapse/federation/federation_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -1642,10 +1642,6 @@ def _validate_hierarchy_event(d: JsonDict) -> None:
if not isinstance(event_type, str):
raise ValueError("Invalid event: 'event_type' must be a str")

room_id = d.get("room_id")
if not isinstance(room_id, str):
raise ValueError("Invalid event: 'room_id' must be a str")

state_key = d.get("state_key")
if not isinstance(state_key, str):
raise ValueError("Invalid event: 'state_key' must be a str")
Expand Down
2 changes: 1 addition & 1 deletion tests/handlers/test_room_summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ def _assert_hierarchy(
result_room_ids.append(result_room["room_id"])
result_children_ids.append(
[
(cs["room_id"], cs["state_key"])
(result_room["room_id"], cs["state_key"])
for cs in result_room["children_state"]
]
)
Expand Down

0 comments on commit 8160849

Please sign in to comment.