diff options
author | Erik Johnston <erik@matrix.org> | 2019-02-23 14:31:08 +0000 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2019-02-23 14:50:43 +0000 |
commit | 59e011220942e642968c453a36fff4862c753ce3 (patch) | |
tree | 251b6083db13972c87d369fcee94e31c6f772f80 /synapse/federation/federation_client.py | |
parent | Merge pull request #4716 from matrix-org/erikj/pusher_logging (diff) | |
download | synapse-59e011220942e642968c453a36fff4862c753ce3.tar.xz |
MSC 1866 - Use M_UNSUPPORTED_ROOM_VERSION for invite API
Diffstat (limited to 'synapse/federation/federation_client.py')
-rw-r--r-- | synapse/federation/federation_client.py | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/synapse/federation/federation_client.py b/synapse/federation/federation_client.py index 4e4f58b418..58e04d81ab 100644 --- a/synapse/federation/federation_client.py +++ b/synapse/federation/federation_client.py @@ -33,6 +33,7 @@ from synapse.api.constants import ( ) from synapse.api.errors import ( CodeMessageException, + Codes, FederationDeniedError, HttpResponseException, SynapseError, @@ -792,10 +793,25 @@ class FederationClient(FederationBase): defer.returnValue(content) except HttpResponseException as e: if e.code in [400, 404]: + err = e.to_synapse_error() + + # If we receive an error response that isn't a generic error, we + # assume that the remote understands the v2 invite API and this + # is a legitimate error. + if err.errcode != Codes.UNKNOWN: + raise err + + # Otherwise, we assume that the remote server doesn't understand + # the v2 invite API. + if room_version in (RoomVersions.V1, RoomVersions.V2): pass # We'll fall through else: - raise Exception("Remote server is too old") + raise SynapseError( + 400, + "User's homeserver does not support this room version", + Codes.UNSUPPORTED_ROOM_VERSION, + ) elif e.code == 403: raise e.to_synapse_error() else: |