diff options
| author | krombel <krombel@krombel.de> | 2019-10-10 13:59:55 +0200 |
|---|---|---|
| committer | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2019-10-10 12:59:55 +0100 |
| commit | 2efd050c9db2e96fd96535dc9b1c6f54acbd163d (patch) | |
| tree | a51eb9731bebcfb7c786a90afaf23ee3e91432ad /tests/rest/client/v2_alpha | |
| parent | Add snapcraft packaging information (#6084) (diff) | |
| download | synapse-2efd050c9db2e96fd96535dc9b1c6f54acbd163d.tar.xz | |
send 404 as http-status when filter-id is unknown to the server (#2380)
This fixed the weirdness of 400 vs 404 as http status code in the case the filter id is not known by the server. As e.g. matrix-js-sdk expects 404 to catch this situation this leads to unwanted behaviour.
Diffstat (limited to 'tests/rest/client/v2_alpha')
| -rw-r--r-- | tests/rest/client/v2_alpha/test_filter.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/rest/client/v2_alpha/test_filter.py b/tests/rest/client/v2_alpha/test_filter.py index f42a8efbf4..e0e9e94fbf 100644 --- a/tests/rest/client/v2_alpha/test_filter.py +++ b/tests/rest/client/v2_alpha/test_filter.py @@ -92,7 +92,7 @@ class FilterTestCase(unittest.HomeserverTestCase): ) self.render(request) - self.assertEqual(channel.result["code"], b"400") + self.assertEqual(channel.result["code"], b"404") self.assertEquals(channel.json_body["errcode"], Codes.NOT_FOUND) # Currently invalid params do not have an appropriate errcode |