Remove different behaviour for fed vs. nonfed

synapse/http/client.py

@@ -76,7 +76,7 @@ class IPBlacklistingResolver(object):
     addresses, preventing DNS rebinding attacks on URL preview.
     """
 
-    def __init__(self, reactor, ip_whitelist, ip_blacklist, federation=False):
+    def __init__(self, reactor, ip_whitelist, ip_blacklist):
         """
         Args:
             reactor (twisted.internet.reactor)
@@ -87,7 +87,6 @@ class IPBlacklistingResolver(object):
         self._reactor = reactor
         self._ip_whitelist = ip_whitelist
         self._ip_blacklist = ip_blacklist
-        self._from_federation = federation
 
     def resolveHostName(self, recv, hostname, portNumber=0):
 

synapse/http/matrixfederationclient.py

@@ -182,7 +182,6 @@ class MatrixFederationHttpClient(object):
         # addresses, to prevent DNS rebinding.
         nameResolver = IPBlacklistingResolver(
             real_reactor, None, hs.config.federation_ip_range_blacklist,
-            federation=True,
         )
 
         @implementer(IReactorPluggableNameResolver)

tests/http/test_fedclient.py

@@ -233,7 +233,7 @@ class FederationClientTests(HomeserverTestCase):
                 fetch_d = cl.get_json("internal:8008", "foo/bar")
 
                 # Nothing happened yet
-                #self.assertNoResult(fetch_d)
+                self.assertNoResult(fetch_d)
 
                 # should have reset logcontext to the sentinel
                 check_logcontext(LoggingContext.sentinel)
@@ -249,7 +249,7 @@ class FederationClientTests(HomeserverTestCase):
         self.pump()
 
         # Nothing has happened yet
-        self.failureResultOf(d, SynapseError)
+        self.assertNoResult(d)
 
         # Check that it was unable to resolve the address
         clients = self.reactor.tcpClients