Ignore encoding, use message.as_string directly.

The message is already in str type, so we don't care about
encoding. json.dumps will ignore convertion.

3 files changed, 7 insertions, 18 deletions

diff --git a/changes/bug_multiple_encodings b/changes/bug_multiple_encodings
new file mode 100644
index 0000000..3113328
--- /dev/null
+++ b/changes/bug_multiple_encodings
@@ -0,0 +1 @@
+  o Some emails are multipart and each part has its own encoding.
\ No newline at end of file
diff --git a/pkg/requirements.pip b/pkg/requirements.pip
index 4242ad4..692f66f 100644
--- a/pkg/requirements.pip
+++ b/pkg/requirements.pip
@@ -10,6 +10,4 @@ couchdb
 
 leap.common>=0.3.5
 leap.soledad.common>=0.3.0
-leap.keymanager>=0.3.4
-
-cchardet  # we fallback to chardet if this is not available, but it's preferred
+leap.keymanager>=0.3.4
\ No newline at end of file
diff --git a/src/leap/mx/mail_receiver.py b/src/leap/mx/mail_receiver.py
index c90dda9..833c0ff 100644
--- a/src/leap/mx/mail_receiver.py
+++ b/src/leap/mx/mail_receiver.py
@@ -27,18 +27,12 @@ import json
 import email.utils
 import socket
 
-try:
-    import cchardet as chardet
-except ImportError:
-    import chardet
-
 from email import message_from_string
 
 from twisted.application.service import Service
 from twisted.internet import inotify, defer, task
 from twisted.python import filepath, log
 
-from leap.common.mail import get_email_charset
 from leap.soledad.common.document import SoledadDocument
 from leap.soledad.common.crypto import (
     EncryptionSchemes,
@@ -118,12 +112,6 @@ class MailReceiver(Service):
 
         # find message's encoding
         message_as_string = message.as_string()
-        encoding = get_email_charset(
-            message_as_string.decode("utf8", "replace"),
-            default=None)
-        if encoding is None:
-            result = chardet.detect(message_as_string)
-            encoding = result["encoding"]
 
         doc = SoledadDocument(doc_id=str(pyuuid.uuid4()))
 
@@ -133,7 +121,8 @@ class MailReceiver(Service):
             doc.content = {
                 self.INCOMING_KEY: True,
                 ENC_SCHEME_KEY: EncryptionSchemes.NONE,
-                ENC_JSON_KEY: json.dumps(data, encoding=encoding)
+                ENC_JSON_KEY: json.dumps(data,
+                                         ensure_ascii=False)
             }
             return doc
 
@@ -143,7 +132,8 @@ class MailReceiver(Service):
             key = gpg.list_keys().pop()
             # We don't care about the actual address, so we use a
             # dummy one, we just care about the import of the pubkey
-            openpgp_key = openpgp._build_key_from_gpg("dummy@mail.com", key, pubkey)
+            openpgp_key = openpgp._build_key_from_gpg("dummy@mail.com",
+                                                      key, pubkey)
 
             # add X-Leap-Provenance header if message is not encrypted
             if message.get_content_type() != 'multipart/encrypted' and \
@@ -158,7 +148,7 @@ class MailReceiver(Service):
                 self.INCOMING_KEY: True,
                 ENC_SCHEME_KEY: EncryptionSchemes.PUBKEY,
                 ENC_JSON_KEY: str(gpg.encrypt(
-                    json.dumps(data, encoding=encoding),
+                    json.dumps(data, ensure_ascii=False),
                     openpgp_key.fingerprint,
                     symmetric=False))
             }