summary refs log tree commit diff
diff options
context:
space:
mode:
authorBrendan Abolivier <babolivier@matrix.org>2021-02-09 18:51:44 +0100
committerGitHub <noreply@github.com>2021-02-09 18:51:44 +0100
commitfb0e14ee9a8cfc9962b2e583cf4837e655d9ca5d (patch)
treeebae0baf447e3bb217812276b5993ea8b4d4f6e9
parentAdd XWiki OIDC provider example. (#9324) (diff)
parentRemove unneeded type constraints on 3rd party protocol lookup responses (diff)
downloadsynapse-fb0e14ee9a8cfc9962b2e583cf4837e655d9ca5d.tar.xz
Merge pull request #9361 from matrix-org/babolivier/third_party_validation
Remove unneeded type constraints on 3rd party protocol lookup responses
-rw-r--r--changelog.d/9361.bugfix1
-rw-r--r--synapse/appservice/api.py3
2 files changed, 1 insertions, 3 deletions
diff --git a/changelog.d/9361.bugfix b/changelog.d/9361.bugfix
new file mode 100644

index 0000000000..4d0477f033 --- /dev/null +++ b/changelog.d/9361.bugfix
@@ -0,0 +1 @@ +Fix a bug causing Synapse to impose the wrong type constraints on fields when processing responses from appservices to `/_matrix/app/v1/thirdparty/user/{protocol}`. diff --git a/synapse/appservice/api.py b/synapse/appservice/api.py
index e366a982b8..11aee50f7a 100644 --- a/synapse/appservice/api.py +++ b/synapse/appservice/api.py
@@ -76,9 +76,6 @@ def _is_valid_3pe_result(r, field): fields = r["fields"] if not isinstance(fields, dict): return False - for k in fields.keys(): - if not isinstance(fields[k], str): - return False return True