[monitoring/js] Consistent datetime format in RADIUS sessions tab #662

Use Django locale for date formatting in device page RADIUS sessions tab JS, matching Django admin and localization settings.

Fixes #662

Author: Eeshu-Yadav

openwisp_radius/api/urls.py

@@ -88,6 +88,11 @@ def get_api_urls(api_views=None):
                 api_views.radius_accounting,
                 name="radius_accounting_list",
             ),
+            path(
+                "radius/monitoring/sessions/",
+                api_views.monitoring_accounting,
+                name="monitoring_accounting_list",
+            ),
         ]
     else:
         return []

openwisp_radius/api/views.py

@@ -53,6 +53,7 @@
     SmsAttemptCooldownException,
     UserAlreadyVerified,
 )
+from ..integrations.monitoring.views import MonitoringAccountingView
 from ..utils import generate_pdf, get_organization_radius_settings, load_model
 from . import freeradius_views
 from .freeradius_views import AccountingFilter, AccountingViewPagination
@@ -847,3 +848,5 @@ class RadiusAccountingView(ProtectedAPIMixin, FilterByOrganizationManaged, ListA
 
 
 radius_accounting = RadiusAccountingView.as_view()
+
+monitoring_accounting = MonitoringAccountingView.as_view()

openwisp_radius/integrations/monitoring/admin.py

@@ -25,7 +25,7 @@ def get_extra_context(self, pk=None):
         ctx.update(
             {
                 "radius_accounting_api_endpoint": reverse(
-                    "radius:radius_accounting_list"
+                    "radius:monitoring_accounting_list"
                 ),
                 "radius_accounting": reverse(
                     f"admin:{RadiusAccounting._meta.app_label}"

openwisp_radius/integrations/monitoring/serializers.py

@@ -0,0 +1,45 @@
+from django.utils import formats, timezone
+from rest_framework import serializers
+from swapper import load_model
+
+RadiusAccounting = load_model("openwisp_radius", "RadiusAccounting")
+
+
+class MonitoringRadiusAccountingSerializer(serializers.ModelSerializer):
+    """
+    Read-only serializer for RADIUS accounting in monitoring integration
+    that formats datetime fields server-side using Django's localization
+    for consistency with Django admin datetime formatting.
+    """
+
+    start_time = serializers.SerializerMethodField()
+    stop_time = serializers.SerializerMethodField()
+
+    def get_start_time(self, obj):
+        """Format start_time using Django's localization settings"""
+        if not obj.start_time:
+            return None
+        local_time = timezone.localtime(obj.start_time)
+        return formats.date_format(local_time, "DATETIME_FORMAT")
+
+    def get_stop_time(self, obj):
+        """Format stop_time using Django's localization settings"""
+        if not obj.stop_time:
+            return None
+        local_time = timezone.localtime(obj.stop_time)
+        return formats.date_format(local_time, "DATETIME_FORMAT")
+
+    class Meta:
+        model = RadiusAccounting
+        fields = [
+            "session_id",
+            "unique_id",
+            "username",
+            "input_octets",
+            "output_octets",
+            "calling_station_id",
+            "called_station_id",
+            "start_time",
+            "stop_time",
+        ]
+        read_only_fields = fields

openwisp_radius/integrations/monitoring/static/radius-monitoring/js/device-change.js

@@ -14,24 +14,6 @@
     const deviceMac = encodeURIComponent($("#id_mac_address").val()),
       apiEndpoint = `${radiusAccountingApiEndpoint}?called_station_id=${deviceMac}`;
 
-    function getFormattedDateTimeString(dateTimeString) {
-      // Strip the timezone from the dateTimeString.
-      // This is done to show the time in server's timezone
-      // because RadiusAccounting admin also shows the time in server's timezone.
-      let strippedDateTime = new Date(dateTimeString.replace(/[-+]\d{2}:\d{2}$/, ""));
-      const locale = djangoLocale.replace("_", "-");
-      const options = {
-        year: "numeric",
-        month: "short",
-        day: "numeric",
-        hour: "2-digit",
-        minute: "2-digit",
-        second: "2-digit",
-        hour12: false,
-      };
-      return strippedDateTime.toLocaleString(locale, options);
-    }
-
     function fetchRadiusSessions() {
       if ($("#radius-session-tbody").children().length) {
         // Don't fetch if RADIUS sessions are already present
@@ -68,11 +50,8 @@
           );
 
           response.forEach((element, index) => {
-            element.start_time = getFormattedDateTimeString(element.start_time);
             if (!element.stop_time) {
               element.stop_time = `<strong>${onlineMsg}</strong>`;
-            } else {
-              element.stop_time = getFormattedDateTimeString(element.stop_time);
             }
             $("#radius-session-tbody").append(
               `<tr class="form-row has_original dynamic-radiussession_set" id="radiussession_set-${index}">

openwisp_radius/integrations/monitoring/templates/admin/config/radius-monitoring/device/change_form.html

@@ -36,7 +36,6 @@ <h2>{% trans "RADIUS Sessions" %}</h2>
 <script>
     const radiusAccountingApiEndpoint = "{{ radius_accounting_api_endpoint }}";
     const radiusAccountingAdminPath = "{{ radius_accounting }}";
-    const djangoLocale = "{{ LANGUAGE_CODE }}";
 </script>
 {% endif %}
 {% endblock %}

openwisp_radius/integrations/monitoring/views.py

@@ -0,0 +1,29 @@
+from django_filters.rest_framework import DjangoFilterBackend
+from rest_framework.generics import ListAPIView
+from swapper import load_model
+
+from openwisp_radius.api.freeradius_views import (
+    AccountingFilter,
+    AccountingViewPagination,
+)
+from openwisp_users.api.mixins import FilterByOrganizationManaged, ProtectedAPIMixin
+
+from .serializers import MonitoringRadiusAccountingSerializer
+
+RadiusAccounting = load_model("openwisp_radius", "RadiusAccounting")
+
+
+class MonitoringAccountingView(
+    ProtectedAPIMixin, FilterByOrganizationManaged, ListAPIView
+):
+    """
+    API view for RADIUS accounting in monitoring integration.
+    Uses server-side datetime formatting for consistency with Django admin.
+    """
+
+    throttle_scope = "radius_accounting_list"
+    serializer_class = MonitoringRadiusAccountingSerializer
+    pagination_class = AccountingViewPagination
+    filter_backends = (DjangoFilterBackend,)
+    filterset_class = AccountingFilter
+    queryset = RadiusAccounting.objects.all().order_by("-start_time")

tests/openwisp2/sample_radius/api/views.py

@@ -24,6 +24,9 @@
 from openwisp_radius.api.views import (
     ValidatePhoneTokenView as BaseValidatePhoneTokenView,
 )
+from openwisp_radius.integrations.monitoring.views import (
+    MonitoringAccountingView as BaseMonitoringAccountingView,
+)
 
 
 class AuthorizeView(BaseAuthorizeView):
@@ -98,6 +101,10 @@ class RadiusAccountingView(BaseRadiusAccountingView):
     pass
 
 
+class MonitoringAccountingView(BaseMonitoringAccountingView):
+    pass
+
+
 authorize = AuthorizeView.as_view()
 postauth = PostAuthView.as_view()
 accounting = AccountingView.as_view()
@@ -116,3 +123,4 @@ class RadiusAccountingView(BaseRadiusAccountingView):
 change_phone_number = ChangePhoneNumberView.as_view()
 download_rad_batch_pdf = DownloadRadiusBatchPdfView.as_view()
 radius_accounting = RadiusAccountingView.as_view()
+monitoring_accounting = MonitoringAccountingView.as_view()