Merge pull request #192 from datosgobar/183-analytics-fix

No guardo accesos al panel de admin en analytics

series_tiempo_ar_api/apps/analytics/tests.py

@@ -1,5 +1,6 @@
 #! coding: utf-8
 import json
+import copy
 
 from django.test import TestCase
 from django.urls import reverse
@@ -45,11 +46,13 @@ class AnalyticsViewTests(TestCase):
     }
 
     def test_view_valid_body(self):
+        count_before = Query.objects.count()
         response = self.client.post(reverse('analytics:save'),
                                     json.dumps(self.body),
                                     content_type="application/json")
 
         self.assertEqual(response.status_code, 200)
+        self.assertEqual(Query.objects.count(), count_before + 1)
 
     def test_view_invalid_method(self):
         response = self.client.put(reverse('analytics:save'),
@@ -64,3 +67,15 @@ def test_view_empty_body(self):
                                     content_type='application/json')
 
         self.assertEqual(response.status_code, 400)
+
+    def test_admin_call_not_logged(self):
+        body = copy.deepcopy(self.body)
+        body['request']['uri'] = 'admin/api/dataset/12'
+        count_before = Query.objects.count()
+        response = self.client.post(reverse('analytics:save'),
+                                    json.dumps(body),
+                                    content_type="application/json")
+        self.assertEqual(response.status_code, 200)
+
+        # Count unchanged
+        self.assertEqual(Query.objects.count(), count_before)

series_tiempo_ar_api/apps/analytics/views.py

@@ -19,7 +19,8 @@ def save(request):
     if not req_data:  # Fatal error
         return HttpResponse(status=400)
 
-    if 'api/' not in req_data.get('uri'):
+    uri = req_data.get('uri')
+    if 'admin/' in uri or 'api/' not in uri:
         return HttpResponse()
 
     params = req_data.get('querystring')