Remove TODO

This commit is contained in:
Andrew Morgan 2019-04-02 10:57:18 +01:00
parent a7d7c5a060
commit fec0c9a074
2 changed files with 2 additions and 1 deletions

View file

@ -146,6 +146,7 @@ class ClientTLSOptionsFactory(object):
# Check if certificate verification has been enabled # Check if certificate verification has been enabled
if (self._config.federation_verify_certificates and if (self._config.federation_verify_certificates and
self._config.federation_certificate_validation_whitelist and
host not in self._config.federation_certificate_validation_whitelist): host not in self._config.federation_certificate_validation_whitelist):
# Require verification # Require verification
return ClientTLSOptions(host, self._options_validate._makeContext()) return ClientTLSOptions(host, self._options_validate._makeContext())

View file

@ -53,7 +53,7 @@ class MatrixFederationAgentTests(TestCase):
self.agent = MatrixFederationAgent( self.agent = MatrixFederationAgent(
reactor=self.reactor, reactor=self.reactor,
tls_client_options_factory=ClientTLSOptionsFactory(#TODO How to deal with None config in tests???), tls_client_options_factory=ClientTLSOptionsFactory(None),
_well_known_tls_policy=TrustingTLSPolicyForHTTPS(), _well_known_tls_policy=TrustingTLSPolicyForHTTPS(),
_srv_resolver=self.mock_resolver, _srv_resolver=self.mock_resolver,
_well_known_cache=self.well_known_cache, _well_known_cache=self.well_known_cache,