summary refs log tree commit diff
diff options
context:
space:
mode:
authorMark Haines <mark.haines@matrix.org>2014-11-20 18:00:10 +0000
committerMark Haines <mark.haines@matrix.org>2014-11-20 18:00:10 +0000
commit610c2ea1314a8aeb0a5c3b8a06f67a4057e25a4c (patch)
tree52f9e08ad39ca4bf4fc19ff7d94e3794686a99c7
parentMerge pull request #20 from matrix-org/http_client_refactor (diff)
downloadsynapse-610c2ea1314a8aeb0a5c3b8a06f67a4057e25a4c.tar.xz
Fix pep8 and pyflakes warnings
-rw-r--r--synapse/handlers/login.py5
-rw-r--r--synapse/handlers/register.py9
-rw-r--r--synapse/http/client.py7
-rw-r--r--synapse/http/endpoint.py2
-rw-r--r--synapse/http/matrixfederationclient.py6
-rw-r--r--synapse/rest/register.py4
6 files changed, 21 insertions, 12 deletions
diff --git a/synapse/handlers/login.py b/synapse/handlers/login.py
index 979086a3d6..c98ae2cfb5 100644
--- a/synapse/handlers/login.py
+++ b/synapse/handlers/login.py
@@ -23,7 +23,6 @@ import synapse.util.emailutils as emailutils
 
 import bcrypt
 import logging
-import urllib
 
 logger = logging.getLogger(__name__)
 
@@ -101,7 +100,9 @@ class LoginHandler(BaseHandler):
         data = yield httpCli.get_json(
             # TODO FIXME This should be configurable.
             # XXX: ID servers need to use HTTPS
-            "http://%s%s" % ("matrix.org:8090", "/_matrix/identity/api/v1/lookup"),
+            "http://%s%s" % (
+                "matrix.org:8090", "/_matrix/identity/api/v1/lookup"
+            ),
             {
                 'medium': 'email',
                 'address': email
diff --git a/synapse/handlers/register.py b/synapse/handlers/register.py
index 277616ea69..48c326ebf0 100644
--- a/synapse/handlers/register.py
+++ b/synapse/handlers/register.py
@@ -168,7 +168,10 @@ class RegistrationHandler(BaseHandler):
             defer.returnValue(None)
         data = yield httpCli.get_json(
             # XXX: This should be HTTPS
-            "http://%s%s" % (creds['idServer'], "/_matrix/identity/api/v1/3pid/getValidated3pid"),
+            "http://%s%s" % (
+                creds['idServer'],
+                "/_matrix/identity/api/v1/3pid/getValidated3pid"
+            ),
             {'sid': creds['sid'], 'clientSecret': creds['clientSecret']}
         )
 
@@ -183,7 +186,9 @@ class RegistrationHandler(BaseHandler):
         httpCli = SimpleHttpClient(self.hs)
         data = yield httpCli.post_urlencoded_get_json(
             # XXX: Change when ID servers are all HTTPS
-            "http://%s%s" % (creds['idServer'], "/_matrix/identity/api/v1/3pid/bind"),
+            "http://%s%s" % (
+                creds['idServer'], "/_matrix/identity/api/v1/3pid/bind"
+            ),
             {
                 'sid': creds['sid'],
                 'clientSecret': creds['clientSecret'],
diff --git a/synapse/http/client.py b/synapse/http/client.py
index 5c3ddcd83d..048a428905 100644
--- a/synapse/http/client.py
+++ b/synapse/http/client.py
@@ -32,15 +32,16 @@ logger = logging.getLogger(__name__)
 
 class SimpleHttpClient(object):
     """
-    A simple, no-frills HTTP client with methods that wrap up common ways of using HTTP in Matrix
+    A simple, no-frills HTTP client with methods that wrap up common ways of
+    using HTTP in Matrix
     """
     def __init__(self, hs):
         self.hs = hs
         # The default context factory in Twisted 14.0.0 (which we require) is
-        # BrowserLikePolicyForHTTPS which will do regular cert validation 'like a browser'
+        # BrowserLikePolicyForHTTPS which will do regular cert validation
+        # 'like a browser'
         self.agent = Agent(reactor)
 
-
     @defer.inlineCallbacks
     def post_urlencoded_get_json(self, uri, args={}):
         logger.debug("post_urlencoded_get_json args: %s", args)
diff --git a/synapse/http/endpoint.py b/synapse/http/endpoint.py
index f1b14a5657..9c8888f565 100644
--- a/synapse/http/endpoint.py
+++ b/synapse/http/endpoint.py
@@ -28,7 +28,7 @@ logger = logging.getLogger(__name__)
 
 
 def matrix_federation_endpoint(reactor, destination, ssl_context_factory=None,
-                    timeout=None):
+                               timeout=None):
     """Construct an endpoint for the given matrix destination.
 
     Args:
diff --git a/synapse/http/matrixfederationclient.py b/synapse/http/matrixfederationclient.py
index b179c76496..510f07dd7b 100644
--- a/synapse/http/matrixfederationclient.py
+++ b/synapse/http/matrixfederationclient.py
@@ -62,8 +62,8 @@ class MatrixFederationHttpAgent(_AgentBase):
 
 
 class MatrixFederationHttpClient(object):
-    """HTTP client used to talk to other homeservers over the federation protocol.
-    Send client certificates and signs requests.
+    """HTTP client used to talk to other homeservers over the federation
+    protocol. Send client certificates and signs requests.
 
     Attributes:
         agent (twisted.web.client.Agent): The twisted Agent used to send the
@@ -305,4 +305,4 @@ class _JsonProducer(object):
         pass
 
     def stopProducing(self):
-        pass
\ No newline at end of file
+        pass
diff --git a/synapse/rest/register.py b/synapse/rest/register.py
index a51f77034a..f25e23a158 100644
--- a/synapse/rest/register.py
+++ b/synapse/rest/register.py
@@ -261,7 +261,9 @@ class RegisterRestServlet(RestServlet):
         )
 
         if session[LoginType.EMAIL_IDENTITY]:
-            logger.debug("Binding emails %s to %s" % (session["threepidCreds"], user_id))
+            logger.debug("Binding emails %s to %s" % (
+                session["threepidCreds"], user_id)
+            )
             yield handler.bind_emails(user_id, session["threepidCreds"])
 
         result = {