modoboa 2.6.4__py3-none-any.whl → 2.7.0__py3-none-any.whl

modoboa/amavis/tests/test_jobs.py

@@ -0,0 +1,18 @@
+"""Tests cases for RQ jobs."""
+
+from modoboa.lib.tests import ModoTestCase
+
+from modoboa.amavis import factories, jobs, models
+
+
+class JobsTestCase(ModoTestCase):
+
+    databases = "__all__"
+
+    def test_qcleanup(self):
+        factories.create_spam("user@test.com", rs="D")
+        jobs.qcleanup()
+        self.assertEqual(models.Quarantine.objects.count(), 0)
+        self.assertEqual(models.Msgs.objects.count(), 0)
+        self.assertEqual(models.Maddr.objects.count(), 0)
+        self.assertEqual(models.Msgrcpt.objects.count(), 0)

modoboa/amavis/tests/test_viewsets.py

@@ -4,14 +4,13 @@ from unittest import mock
 from rq import SimpleWorker
 
 from django.core import mail
-from django.core.management import call_command
 from django.test import override_settings
 from django.urls import reverse
 
 import django_rq
 
 from modoboa.admin import factories as admin_factories, models as admin_models
-from modoboa.amavis import factories, models
+from modoboa.amavis import factories, jobs, models
 from modoboa.amavis.utils import smart_str
 from modoboa.core import models as core_models
 from modoboa.lib.tests import ModoAPITestCase
@@ -204,7 +203,7 @@ class QuarantineViewSetTestCase(TestDataMixin, ModoAPITestCase):
         self.assertEqual(self.msgrcpt.rs, "p")
 
         # Send notification to admins
-        call_command("amnotify")
+        jobs.amnotify()
         self.assertEqual(len(mail.outbox), 1)
 
     def _test_mark_selection(self, action, status):

modoboa/autoconfig/templates/autoconfig/autoconfig.xml

@@ -9,7 +9,7 @@
       <hostname>{{ connection_settings.imap.HOSTNAME }}</hostname>
       <port>{{ connection_settings.imap.PORT }}</port>
       <socketType>{{ connection_settings.imap.SOCKET_TYPE }}</socketType>
-      <authentication>plain</authentication>
+      <authentication>password-cleartext</authentication>
       <username>{{ emailaddress }}</username>
     </incomingServer>
 
@@ -17,7 +17,7 @@
       <hostname>{{ connection_settings.smtp.HOSTNAME }}</hostname>
       <port>{{ connection_settings.smtp.PORT }}</port>
       <socketType>{{ connection_settings.smtp.SOCKET_TYPE }}</socketType>
-      <authentication>plain</authentication>
+      <authentication>password-cleartext</authentication>
       <username>{{ emailaddress }}</username>
     </outgoingServer>
   </emailProvider>

modoboa/autoconfig/templates/autoconfig/autodiscover.xml

@@ -9,7 +9,16 @@
         <Type>IMAP</Type>
         <Server>{{ connection_settings.imap.HOSTNAME }}</Server>
         <Port>{{ connection_settings.imap.PORT }}</Port>
+        {% if connection_settings.imap.SOCKET_TYPE|upper == "SSL" %}
         <SSL>true</SSL>
+        <TLS>false</TLS>
+        {% elif connection_settings.imap.SOCKET_TYPE|upper == "STARTTLS" %}
+        <SSL>false</SSL>
+        <TLS>true</TLS>
+        {% elif connection_settings.imap.SOCKET_TYPE|lower == "plain" %}
+        <SSL>false</SSL>
+        <TLS>false</TLS>
+        {% endif %}
         <LoginName>{{ emailaddress }}</LoginName>
       </Protocol>
 
@@ -17,8 +26,13 @@
         <Type>SMTP</Type>
         <Server>{{ connection_settings.smtp.HOSTNAME }}</Server>
         <Port>{{ connection_settings.smtp.PORT }}</Port>
+        {% if connection_settings.smtp.SOCKET_TYPE|upper == "SSL" %}
+        <SSL>true</SSL>
+        <TLS>false</TLS>
+        {% elif connection_settings.smtp.SOCKET_TYPE|upper == "STARTTLS" %}
         <SSL>false</SSL>
         <TLS>true</TLS>
+        {% endif %}
         <LoginName>{{ emailaddress }}</LoginName>
       </Protocol>
 

modoboa/autoconfig/tests.py

@@ -10,7 +10,8 @@ class ViewsTestCase(TestCase):
         url = reverse("autoconfig:autoconfig")
 
         resp = self.client.get(url)
-        self.assertEqual(resp.status_code, 404)
+        self.assertEqual(resp.status_code, 200)
+        self.assertIn(b"%EMAILLOCALPART%@%EMAILDOMAIN%", resp.content)
 
         resp = self.client.get(f"{url}?emailaddress=test@test.com")
         self.assertEqual(resp.status_code, 200)

modoboa/autoconfig/views.py

@@ -24,6 +24,10 @@ class ConfigBaseMixin:
 
 
 class AutoConfigView(ConfigBaseMixin, generic.TemplateView):
+    """
+    Format documentation:
+    https://wiki.mozilla.org/Thunderbird:Autoconfiguration:ConfigFileFormat
+    """
 
     http_method_names = ["get"]
     template_name = "autoconfig/autoconfig.xml"
@@ -31,7 +35,7 @@ class AutoConfigView(ConfigBaseMixin, generic.TemplateView):
     def get_context_data(self, **kwargs):
         emailaddress = self.request.GET.get("emailaddress")
         if not emailaddress:
-            raise Http404
+            emailaddress = "%EMAILLOCALPART%@%EMAILDOMAIN%"
         context = super().get_context_data(**kwargs)
         context.update(self.get_common_context(emailaddress))
         return context
@@ -88,14 +92,18 @@ class MobileConfigView(generic.View):
                     "EmailAccountType": "EmailTypeIMAP",
                     "EmailAddress": emailaddress,
                     # incoming
+                    "IncomingMailServerAuthentication": "EmailAuthPassword",
                     "IncomingMailServerHostName": imap_settings["HOSTNAME"],
                     "IncomingMailServerPortNumber": imap_settings["PORT"],
-                    "IncomingMailServerUseSSL": True,
+                    "IncomingMailServerUseSSL": imap_settings["SOCKET_TYPE"].upper()
+                    == "SSL",  # `false` means StartTLS
                     "IncomingMailServerUsername": emailaddress,
                     # outgoing
+                    "OutgoingMailServerAuthentication": "EmailAuthPassword",
                     "OutgoingMailServerHostName": smtp_settings["HOSTNAME"],
                     "OutgoingMailServerPortNumber": smtp_settings["PORT"],
-                    "OutgoingMailServerUseSSL": True,
+                    "OutgoingMailServerUseSSL": smtp_settings["SOCKET_TYPE"].upper()
+                    == "SSL",  # `false` means StartTLS,
                     "OutgoingMailServerUsername": emailaddress,
                     "OutgoingPasswordSameAsIncomingPassword": True,
                 }

modoboa/autoreply/models.py

@@ -38,9 +38,9 @@ class ARmessage(models.Model):
         days = request.localconfig.parameters.get_value("tracking_period")
         context["fromdate"] = localize(self.fromdate)
         fromdate = self.fromdate.isoformat()
-        fromdate, tz = fromdate.split("+")
-        tz = f"+{tz.replace(':', '')}"
-        condition = [("currentdate", ":zone", tz, ":value", "ge", "iso8601", fromdate)]
+        datepart = fromdate[:-6]
+        tz = fromdate[-6:].replace(":", "")
+        condition = [("currentdate", ":zone", tz, ":value", "ge", "iso8601", datepart)]
         if self.untildate:
             context["untildate"] = localize(self.untildate)
             condition.append(
@@ -51,7 +51,7 @@ class ARmessage(models.Model):
                     ":value",
                     "lt",
                     "iso8601",
-                    self.untildate.isoformat().split("+")[0],
+                    self.untildate.isoformat()[:-6],
                 )
             )
         content = self.content % context

modoboa/calendars/backends/caldav_.py

@@ -6,6 +6,7 @@ import uuid
 import caldav
 from caldav.elements import dav, ical
 from caldav import Calendar
+from dateutil.relativedelta import relativedelta
 import vobject
 
 from django.utils import timezone
@@ -53,8 +54,10 @@ class Caldav_Backend(CalendarBackend):
             start = datetime.datetime.combine(
                 vevent.dtstart.value, datetime.time.min
             ).replace(tzinfo=tz)
+            # Small back to make vuetify calendar happy. 'All day' events are generally
+            # created from one day to the day after (even for a 1 day duration...)
             end = datetime.datetime.combine(
-                vevent.dtend.value, datetime.time.min
+                vevent.dtend.value - relativedelta(days=1), datetime.time.min
             ).replace(tzinfo=tz)
         result.update({"allDay": all_day, "start": start, "end": end})
         if "attendee" in vevent.contents:
@@ -80,17 +83,16 @@ class Caldav_Backend(CalendarBackend):
     def create_event(self, data):
         """Create a new event."""
         uid = uuid.uuid4()
-        cal = vobject.iCalendar()
-        cal.add("vevent")
-        cal.vevent.add("uid").value = str(uid)
-        cal.vevent.add("summary").value = data["title"]
         if not data["allDay"]:
-            cal.vevent.add("dtstart").value = data["start"]
-            cal.vevent.add("dtend").value = data["end"]
+            dtstart = data["start"]
+            dtend = data["end"]
         else:
-            cal.vevent.add("dtstart").value = data["start"].date()
-            cal.vevent.add("dtend").value = data["end"].date()
-        self.remote_cal.add_event(cal)
+            dtstart = data["start_date"]
+            # Ensure consistent behavior with other calendar clients (like TB.)
+            dtend = data["end_date"] + relativedelta(days=1)
+        self.remote_cal.save_event(
+            uid=uid, dtstart=dtstart, dtend=dtend, summary=data["title"]
+        )
         return uid
 
     def update_event(self, uid, original_data):
@@ -102,8 +104,8 @@ class Caldav_Backend(CalendarBackend):
         if "title" in data:
             orig_evt.summary.value = data["title"]
         if data.get("allDay"):
-            data["start"] = data["start"].date()
-            data["end"] = data["end"].date()
+            data["start"] = data["start_date"]
+            data["end"] = data["end_date"]
         if "start" in data:
             del orig_evt.contents["dtstart"]
             orig_evt.add("dtstart").value = data["start"]
@@ -141,7 +143,9 @@ class Caldav_Backend(CalendarBackend):
 
     def get_events(self, start, end):
         """Retrieve a list of events."""
-        orig_events = self.remote_cal.date_search(start, end)
+        orig_events = self.remote_cal.search(
+            server_expand=False, start=start, end=end, event=True
+        )
         events = []
         for event in orig_events:
             events.append(self._serialize_event(event))

modoboa/calendars/jobs.py

@@ -0,0 +1,7 @@
+"""Async jobs definition."""
+
+from django.core.management import call_command
+
+
+def generate_rights():
+    call_command("generate_rights")

modoboa/calendars/mocks.py

@@ -49,11 +49,14 @@ class Calendar:
     def add_event(self, data):
         return True
 
+    def save_event(self, *args, **kwargs):
+        return True
+
     def event_by_url(self, url):
         res = objects.Event(url=url, data=EV1, parent=self)
         return res
 
-    def date_search(self, start, end):
+    def search(self, **kwargs):
         return [
             objects.Event(data=EV1, parent=self),
             objects.Event(data=EV2, parent=self),

modoboa/calendars/serializers.py

@@ -116,11 +116,11 @@ class EventSerializer(serializers.Serializer):
 
     id = serializers.CharField(read_only=True)
     title = serializers.CharField()
-    start = serializers.DateTimeField()
-    end = serializers.DateTimeField()
+    start = serializers.DateTimeField(required=False)
+    end = serializers.DateTimeField(required=False)
     allDay = serializers.BooleanField(default=False)
     color = serializers.CharField(read_only=True)
-    description = serializers.CharField(required=False)
+    description = serializers.CharField(required=False, allow_blank=True)
 
     attendees = AttendeeSerializer(many=True, required=False)
 
@@ -147,10 +147,13 @@ class WritableEventSerializer(EventSerializer):
     )
     new_calendar_type = serializers.CharField(required=False)
 
+    start_date = serializers.DateField(required=False)
+    end_date = serializers.DateField(required=False)
+
     def __init__(self, *args, **kwargs):
         """Set calendar list."""
         calendar_type = kwargs.pop("calendar_type")
-        super(EventSerializer, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
         self.update_calendar_field(calendar_type)
 
     def update_calendar_field(self, calendar_type):
@@ -170,11 +173,13 @@ class WritableEventSerializer(EventSerializer):
     def validate(self, data):
         """Make sure dates are present with allDay flag."""
         errors = {}
-        if "allDay" in data:
-            if "start" not in data:
-                errors["start"] = _("This field is required.")
-            if "end" not in data:
-                errors["end"] = _("This field is required.")
+        if data.get("allDay", False):
+            mandatory_fields = ["start_date", "end_date"]
+        else:
+            mandatory_fields = ["start", "end"]
+        for field in mandatory_fields:
+            if not data.get(field):
+                errors[field] = _("This field is required.")
         if errors:
             raise serializers.ValidationError(errors)
         return data

modoboa/calendars/tests.py

@@ -18,6 +18,7 @@ from modoboa.lib.tests import ModoAPITestCase
 from modoboa.admin.factories import populate_database
 
 from . import factories
+from . import jobs
 from . import models
 from . import mocks
 
@@ -106,7 +107,7 @@ class AccessRuleTestCase(ModoAPITestCase):
         self.assertEqual(cfg.get(section, "permissions"), "Rr")
 
         # Call a second time
-        management.call_command("generate_rights", verbosity=False)
+        jobs.generate_rights()
 
     def test_rights_file_generation_with_admin(self):
         self.set_global_parameter(
@@ -428,8 +429,8 @@ class EventViewSetTestCase(TestDataMixin, ModoAPITestCase):
         url = f"/api/v2/user-calendars/{self.calendar.pk}/events/"
         data = {
             "title": "Test event",
-            "start": "2018-03-27T00:00:00Z",
-            "end": "2018-03-28T00:00:00Z",
+            "start_date": "2018-03-27",
+            "end_date": "2018-03-28",
             "allDay": True,
             "color": "#ffdddd",
             "description": "Description",
@@ -461,8 +462,8 @@ class EventViewSetTestCase(TestDataMixin, ModoAPITestCase):
         url = f"/api/v2/user-calendars/{self.calendar.pk}/events/1234/"
         data = {
             "title": "Test event",
-            "start": "2018-03-27T00:00:00Z",
-            "end": "2018-03-28T00:00:00Z",
+            "start_date": "2018-03-27",
+            "end_date": "2018-03-28",
             "allDay": True,
             "color": "#ffdddd",
             "description": "Description",
@@ -483,8 +484,8 @@ class EventViewSetTestCase(TestDataMixin, ModoAPITestCase):
 
         url = f"/api/v2/user-calendars/{self.calendar.pk}/events/1234/"
         data = {
-            "start": "2018-03-27T00:00:00Z",
-            "end": "2018-03-28T00:00:00Z",
+            "start_date": "2018-03-27",
+            "end_date": "2018-03-28",
             "allDay": True,
         }
         response = self.client.patch(url, data=data, format="json")
@@ -496,8 +497,8 @@ class EventViewSetTestCase(TestDataMixin, ModoAPITestCase):
         self.client.post(reverse("core:login"), data)
         data = {
             "title": "Test event",
-            "start": "2018-03-27T00:00:00Z",
-            "end": "2018-03-28T00:00:00Z",
+            "start_date": "2018-03-27",
+            "end_date": "2018-03-28",
             "allDay": True,
             "color": "#ffdddd",
             "description": "Description",

modoboa/calendars/viewsets.py

@@ -21,7 +21,9 @@ from . import serializers
 
 def parse_date_from_iso(value):
     """Return a tz aware datetime parsed from an ISO date."""
-    return dateutil.parser.parse(value)
+    result = dateutil.parser.parse(value)
+    print(result)
+    return result
 
 
 class CheckTokenMixin:

modoboa/contacts/migrations/0008_addressbook_syncing.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.2.9 on 2025-12-12 14:42
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("contacts", "0007_alter_contact_address"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="addressbook",
+            name="syncing",
+            field=models.BooleanField(default=False),
+        ),
+    ]

modoboa/contacts/models.py

@@ -22,6 +22,7 @@ class AddressBook(models.Model):
     name = models.CharField(max_length=50)
     sync_token = models.TextField(blank=True)
     last_sync = models.DateTimeField(null=True)
+    syncing = models.BooleanField(default=False)
     user = models.ForeignKey("core.User", on_delete=models.CASCADE)
     _path = models.TextField()
 

modoboa/contacts/serializers.py

@@ -14,7 +14,7 @@ class AddressBookSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = models.AddressBook
-        fields = ("pk", "name", "url", "last_sync")
+        fields = ("pk", "name", "url", "last_sync", "syncing")
 
 
 class EmailAddressSerializer(serializers.ModelSerializer):
@@ -235,9 +235,12 @@ class UserPreferencesSerializer(serializers.Serializer):
 
     def post_save(self, request):
         """Create remote cal if necessary."""
+        abook = request.user.addressbook_set.first()
         if not self.validated_data["enable_carddav_sync"]:
+            abook.last_sync = None
+            abook.sync_token = ""
+            abook.save(update_fields=["last_sync", "sync_token"])
             return
-        abook = request.user.addressbook_set.first()
         if abook.last_sync:
             return
         tasks.create_cdav_addressbook(abook, request.auth)

modoboa/contacts/tasks.py

@@ -49,12 +49,17 @@ def push_addressbook_to_carddav(request, addressbook):
     addressbook.save(update_fields=["last_sync", "sync_token"])
 
 
-def sync_addressbook_from_cdav(request, addressbook):
+def sync_addressbook_from_cdav(addressbook_id: int, access_token: str):
     """Fetch changes from CardDAV server."""
-    clt = get_cdav_client_from_request(request, addressbook)
+    addressbook = models.AddressBook.objects.get(id=addressbook_id)
+    if addressbook.syncing:
+        return
+    clt = get_cdav_client(addressbook, addressbook.user.email, access_token)
     changes = clt.sync_vcards(addressbook.sync_token)
     if not len(changes["cards"]):
         return
+    addressbook.syncing = True
+    addressbook.save()
     for card in changes["cards"]:
         # UID sometimes embded .vcf extension, sometimes not...
         long_uid = card["href"].split("/")[-1]
@@ -73,7 +78,8 @@ def sync_addressbook_from_cdav(request, addressbook):
             models.Contact.objects.filter(uid__in=[long_uid, short_uid]).delete()
     addressbook.last_sync = timezone.now()
     addressbook.sync_token = changes["token"]
-    addressbook.save(update_fields=["last_sync", "sync_token"])
+    addressbook.syncing = False
+    addressbook.save(update_fields=["last_sync", "sync_token", "syncing"])
 
 
 def push_contact_to_cdav(request, contact):

modoboa/contacts/tests.py

@@ -3,12 +3,14 @@
 import os
 
 import httmock
+from rq import SimpleWorker
 
 from django.core import management
-
 from django.urls import reverse
 from django.utils import timezone
 
+import django_rq
+
 from modoboa.admin import factories as admin_factories
 from modoboa.core import models as core_models
 from modoboa.lib.tests import ModoAPITestCase
@@ -65,13 +67,25 @@ class TestDataMixin:
             )
         self.assertEqual(response.status_code, 200)
 
+    def disable_cdav_sync(self):
+        url = reverse("v2:parameter-user-detail", args=["contacts"])
+        response = self.client.put(
+            url,
+            {
+                "enable_carddav_sync": False,
+                "sync_frequency": 300,
+            },
+            format="json",
+        )
+        self.assertEqual(response.status_code, 200)
+
 
 class ViewsTestCase(TestDataMixin, ModoAPITestCase):
     """Check views."""
 
     def test_user_settings(self):
         """Check that remote collection creation request is sent."""
-        # 1. Addressbook with contacts must be synced manually
+        # 1. Addressbook with contacts must be synced manuelle
         self.client.force_authenticate(self.user)
         self.enable_cdav_sync()
         self.addressbook.refresh_from_db()
@@ -87,6 +101,10 @@ class ViewsTestCase(TestDataMixin, ModoAPITestCase):
         self.enable_cdav_sync()
         abook.refresh_from_db()
         self.assertIsNot(abook.last_sync, None)
+        # And disable sync.
+        self.disable_cdav_sync()
+        abook.refresh_from_db()
+        self.assertIs(abook.last_sync, None)
 
 
 class AddressBookViewSetTestCase(TestDataMixin, ModoAPITestCase):
@@ -118,10 +136,18 @@ class AddressBookViewSetTestCase(TestDataMixin, ModoAPITestCase):
         data = {"username": self.user.username, "password": "toto"}
         response = self.client.post(reverse("core:login"), data)
         self.enable_cdav_sync()
-        self.user.addressbook_set.update(last_sync=timezone.now())
+        last_sync = timezone.now()
+        self.user.addressbook_set.update(last_sync=last_sync)
+        addressbook = self.user.addressbook_set.first()
+        self.assertEqual(addressbook.sync_token, "")
         with httmock.HTTMock(mocks.options_mock, mocks.report_mock, mocks.get_mock):
             response = self.client.get(reverse("api:addressbook-sync-from-cdav"))
+            queue = django_rq.get_queue("modoboa")
+            worker = SimpleWorker([queue], connection=queue.connection)
+            worker.work(burst=True)
         self.assertEqual(response.status_code, 200)
+        addressbook.refresh_from_db()
+        self.assertNotEqual(addressbook.sync_token, "")
 
 
 class CategoryViewSetTestCase(TestDataMixin, ModoAPITestCase):
@@ -167,18 +193,18 @@ class ContactViewSetTestCase(TestDataMixin, ModoAPITestCase):
         url = reverse("api:contact-list")
         response = self.client.get(url)
         self.assertEqual(response.status_code, 200)
-        self.assertEqual(len(response.data), 3)
+        self.assertEqual(len(response.data["results"]), 3)
 
         response = self.client.get(f"{url}?search=homer")
         self.assertEqual(response.status_code, 200)
-        self.assertEqual(len(response.data), 1)
+        self.assertEqual(len(response.data["results"]), 1)
 
     def test_filter_by_category(self):
         """Check filtering."""
         url = reverse("api:contact-list")
         response = self.client.get(f"{url}?category={self.category.name}")
         self.assertEqual(response.status_code, 200)
-        self.assertEqual(len(response.data), 1)
+        self.assertEqual(len(response.data["results"]), 1)
 
     def test_create_contact(self):
         """Create a new contact."""

modoboa/contacts/viewsets.py

@@ -1,10 +1,13 @@
 """Contacts viewsets."""
 
 import django_filters.rest_framework
+import django_rq
 from rest_framework.decorators import action
 from rest_framework import filters, response, viewsets
 from rest_framework.permissions import IsAuthenticated
 
+from modoboa.lib import pagination
+
 from . import models
 from . import serializers
 from . import tasks
@@ -40,7 +43,8 @@ class AddressBookViewSet(viewsets.GenericViewSet):
         if not abook.last_sync:
             return response.Response()
         if request.user.parameters.get_value("enable_carddav_sync", app="contacts"):
-            tasks.sync_addressbook_from_cdav(request, abook)
+            queue = django_rq.get_queue("modoboa")
+            queue.enqueue(tasks.sync_addressbook_from_cdav, abook.pk, str(request.auth))
         return response.Response({})
 
 
@@ -70,10 +74,12 @@ class ContactViewSet(viewsets.ModelViewSet):
     """Contact ViewSet."""
 
     filter_backends = [
+        filters.OrderingFilter,
         filters.SearchFilter,
         django_filters.rest_framework.DjangoFilterBackend,
     ]
     filterset_class = ContactFilter
+    pagination_class = pagination.CustomPageNumberPagination
     permission_classes = [IsAuthenticated]
     search_fields = ("^first_name", "^last_name", "^emails__address")
     serializer_class = serializers.ContactSerializer

modoboa/core/api/v2/serializers.py

@@ -190,19 +190,12 @@ class CoreGlobalParametersSerializer(serializers.Serializer):
     log_maximum_age = serializers.IntegerField(default=365)
     message_history_maximum_age = serializers.IntegerField(default=180)
     items_per_page = serializers.IntegerField(default=30)
-    default_top_redirection = serializers.ChoiceField(
-        default="user", choices=[("user", _("User profile"))], required=False
-    )
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         sms_backend_fields = sms_backends.get_all_backend_serializer_settings()
         for field, definition in sms_backend_fields.items():
             self.fields[field] = definition["type"](**definition["attrs"])
-        # Choices serializer for default_top_redirection field
-        self.fields["default_top_redirection"].choices = (
-            app_settings.enabled_applications()
-        )
         # Populate choices of password_scheme
         self.fields["password_scheme"].choices = app_settings.get_password_scheme()
 

modoboa/core/api/v2/tests.py

@@ -74,7 +74,6 @@ CORE_SETTINGS = {
     "top_notifications_check_interval": 30,
     "log_maximum_age": 365,
     "items_per_page": 30,
-    "default_top_redirection": "user",
 }
 
 
@@ -596,15 +595,6 @@ class FIDOViewSetTestCase(ModoAPITestCase):
         self.assertIn("tokens", resp.json())
 
 
-class CapabilitiesAPITestCase(ModoAPITestCase):
-
-    def test_get(self):
-        url = reverse("v2:capabilities")
-        response = self.client.get(url)
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("rspamd", response.json()["capabilities"])
-
-
 class ThemeAPITestCase(ModoAPITestCase):
 
     def test_get(self):