Changeset 178:37f36ea93838 for wokkel/client.py

Timestamp:

Jan 12, 2013, 4:40:38 PM (9 years ago)

Author:

Ralph Meijer <ralphm@…>

Branch:

default

Message:

Properly encode IDN domain names for SRV lookups.

Before Twisted 12.3.0, the SRV lookup done for outgoing
connections succeeded if passed a unicode string with all-ASCII
code points. A recent change made the DNS code more strict, and
only byte string are accepted as domain name. See
​http://twistedmatrix.com/trac/ticket/6245 for details.

This change makes sure domain names are encoded to their ASCII
Compatible Equivalent (ACE) version before passing the resulting
byte string to twisted.names.srvconnect.SRVConnector, as per
RFC 3490.

Note that while connecting to servers with an IDN domain name
now works properly, authentication using MD5-Digest SASL mechanism
will fail until ​http://twistedmatrix.com/trac/ticket/5066 has been
resolved.

Fixes: #77.

File:

• wokkel/client.py (modified) (2 diffs)

wokkel/client.py

@@ -88 +88 @@
     def __init__(self, jid, password, host=None, port=5222):
         self.jid = jid
-        self.domain = jid.host
+        self.domain = generic.prepareIDNName(jid.host)
         self.host = host
         self.port = port
@@ -183 +183 @@
 
 def clientCreator(factory):
-    domain = factory.authenticator.jid.host
+    domain = generic.prepareIDNName(factory.authenticator.jid.host)
     c = XMPPClientConnector(reactor, domain, factory)
     c.connect()