Since empty lookups now return 200/empty list not 404, we can safely log failures as exceptions

1 files changed, 2 insertions, 3 deletions

diff --git a/synapse/appservice/api.py b/synapse/appservice/api.py

index 39b4bff556..e05570cc8b 100644
--- a/synapse/appservice/api.py
+++ b/synapse/appservice/api.py
@@ -78,9 +78,8 @@ class ApplicationServiceApi(SimpleHttpClient):
         try:
             response = yield self.get_json(uri, fields)
             defer.returnValue(response)
-        except Exception:
-            # TODO: would be noisy to log lookup failures, but we want to log
-            # other things. Hrm.
+        except Exception as ex:
+            logger.warning("query_3pu to %s threw exception %s", uri, ex)
             defer.returnValue([])
 
     @defer.inlineCallbacks