Email_fetcher : Add support for IMAP unencrypted connection

tracim/development.ini.base

@@ -221,6 +221,7 @@ email.reply.imap.port = 993
 email.reply.imap.user = your_imap_user
 email.reply.imap.password = your_imap_password
 email.reply.imap.folder = INBOX
+email.reply.imap.ssl = true
 # Token for communication between mail fetcher and tracim controller
 email.reply.token = mysecuretoken
 # Delay in seconds between each check

tracim/tracim/config/app_cfg.py

@@ -379,6 +379,9 @@ class CFG(object):
         self.EMAIL_REPLY_TOKEN = tg.config.get(
             'email.reply.token',
         )
+        self.EMAIL_REPLY_IMAP_SSL = asbool(tg.config.get(
+            'email.reply.imap.ssl',
+        ))
 
         self.TRACKER_JS_PATH = tg.config.get(
             'js_tracker_path',

tracim/tracim/lib/daemons.py

@@ -171,6 +171,7 @@ class MailFetcherDaemon(Daemon):
             port=cfg.EMAIL_REPLY_IMAP_PORT,
             user=cfg.EMAIL_REPLY_IMAP_USER,
             password=cfg.EMAIL_REPLY_IMAP_PASSWORD,
+            ssl=cfg.EMAIL_REPLY_IMAP_SSL,
             folder=cfg.EMAIL_REPLY_IMAP_FOLDER,
             delay=cfg.EMAIL_REPLY_CHECK_HEARTBEAT,
             # FIXME - G.M - 2017-11-15 - proper tracim url formatting

tracim/tracim/lib/email_fetcher.py

@@ -176,6 +176,7 @@ class MailFetcher(object):
                  port: str,
                  user: str,
                  password: str,
+                 ssl: bool,
                  folder: str,
                  delay: int,
                  endpoint: str,
@@ -189,6 +190,7 @@ class MailFetcher(object):
         :param port: imap connection port
         :param user: user login of mailbox
         :param password: user password of mailbox
+        :param ssl: use imap over ssl connection
         :param folder: mail folder where new mail are fetched
         :param delay: seconds to wait before fetching new mail again
         :param endpoint: tracim http endpoint where decoded mail are send.
@@ -199,6 +201,7 @@ class MailFetcher(object):
         self.port = port
         self.user = user
         self.password = password
+        self.ssl = ssl
         self.folder = folder
         self.delay = delay
         self.endpoint = endpoint
@@ -225,11 +228,14 @@ class MailFetcher(object):
         # Are old connexion properly close this way ?
         if self._connection:
             self._disconnect()
-        # TODO - G.M - 2017-11-15 Support unencrypted connection ?
         # TODO - G.M - 2017-11-23 Support for predefined SSLContext ?
         # without ssl_context param, tracim use default security configuration
         # which is great in most case.
-        self._connection = imaplib.IMAP4_SSL(self.host, self.port)
+        if self.ssl:
+            self._connection = imaplib.IMAP4_SSL(self.host, self.port)
+        else:
+            self._connection = imaplib.IMAP4(self.host, self.port)
+
         try:
             self._connection.login(self.user, self.password)
         except Exception as e: