2 files changed, 44 insertions, 10 deletions

diff --git a/hyperkitty/lib/__init__.py b/hyperkitty/lib/__init__.py
index 7ae9ee9..4acb7d1 100644
--- a/hyperkitty/lib/__init__.py
+++ b/hyperkitty/lib/__init__.py
@@ -27,6 +27,7 @@ from django.conf import settings
 from django.core.exceptions import SuspiciousOperation
 from django.core.mail import EmailMessage
 
+from hyperkitty.lib import mailman
 
 
 FLASH_MESSAGES = {
@@ -102,22 +103,18 @@ def daterange(start_date, end_date):
         yield start_date + datetime.timedelta(n)
 
 
+class PostingFailed(Exception): pass
+
 def post_to_list(request, mlist, subject, message, headers={}):
     if not mlist:
         # Make sure the list exists to avoid posting to any email addess
         raise SuspiciousOperation("I don't know this mailing-list")
     # Check that the user is subscribed
-    client = Client('%s/3.0' % settings.MAILMAN_REST_SERVER,
-                    settings.MAILMAN_API_USER, settings.MAILMAN_API_PASS)
-    rest_list = client.get_list(mlist.name)
     try:
-        member = rest_list.get_member(request.user.email)
-    except ValueError:
-        # not subscribed yet, subscribe the user without email delivery
-        member = rest_list.subscribe(request.user.email,
-                "%s %s" % (request.user.first_name, request.user.last_name))
-        member.preferences["delivery_status"] = "by_user"
-        member.preferences.save()
+        mailman.subscribe(mlist.name, request.user)
+    except mailman.MailmanConnectionError:
+        raise PostingFailed("Can't connect to Mailman's REST server, "
+                            "your message has not been sent.")
     # send the message
     headers["User-Agent"] = "HyperKitty on %s" % request.build_absolute_uri("/")
     msg = EmailMessage(
diff --git a/hyperkitty/lib/mailman.py b/hyperkitty/lib/mailman.py
new file mode 100644
index 0000000..2bc245b
--- /dev/null
+++ b/hyperkitty/lib/mailman.py
@@ -0,0 +1,37 @@
+#-*- coding: utf-8 -*-
+# Copyright (C) 1998-2012 by the Free Software Foundation, Inc.
+#
+# This file is part of HyperKitty.
+#
+# HyperKitty is free software: you can redistribute it and/or modify it under
+# the terms of the GNU General Public License as published by the Free
+# Software Foundation, either version 3 of the License, or (at your option)
+# any later version.
+#
+# HyperKitty is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+# more details.
+#
+# You should have received a copy of the GNU General Public License along with
+# HyperKitty.  If not, see <http://www.gnu.org/licenses/>.
+#
+# Author: Aurelien Bompard <abompard@fedoraproject.org>
+#
+
+from django.conf import settings
+from mailmanclient import Client, MailmanConnectionError
+
+
+def subscribe(list_address, user):
+    client = Client('%s/3.0' % settings.MAILMAN_REST_SERVER,
+                    settings.MAILMAN_API_USER, settings.MAILMAN_API_PASS)
+    rest_list = client.get_list(list_address)
+    try:
+        member = rest_list.get_member(user.email)
+    except ValueError:
+        # not subscribed yet, subscribe the user without email delivery
+        member = rest_list.subscribe(user.email,
+                "%s %s" % (user.first_name, user.last_name))
+        member.preferences["delivery_status"] = "by_user"
+        member.preferences.save()