Fix improper escaping in Name.to_unicode() method

Escaping before calling encoding.idna cause incorrect conversion.
Now name is escaping after encofing.idna, and only whitespaces

1 files changed, 8 insertions, 3 deletions

diff --git a/dns/name.py b/dns/name.py
index 5f8f213..0d9b4f2 100644
--- a/dns/name.py
+++ b/dns/name.py
@@ -88,8 +88,10 @@ _escaped = {
     '$' : True
     }
 
-def _escapify(label):
+def _escapify(label, whitespaces_only=False):
     """Escape the characters in label which need it.
+    @param whitespaces_only: escapify only special and whitespace (ord < 0x20)
+    characters
     @returns: the escaped string
     @rtype: string"""
     text = ''
@@ -99,7 +101,10 @@ def _escapify(label):
         elif ord(c) > 0x20 and ord(c) < 0x7F:
             text += c
         else:
-            text += '\\%03d' % ord(c)
+            if whitespaces_only and ord(c) >= 0x7F:
+                text += c
+            else:
+                text += '\\%03d' % ord(c)
     return text
 
 def _validate_labels(labels):
@@ -352,7 +357,7 @@ class Name(object):
             l = self.labels[:-1]
         else:
             l = self.labels
-        s = u'.'.join([encodings.idna.ToUnicode(_escapify(x)) for x in l])
+        s = u'.'.join([_escapify(encodings.idna.ToUnicode(x), whitespaces_only=True) for x in l])
         return s
 
     def to_digestable(self, origin=None):