Merge pull request #100 from MindscapeHQ/cm/apl-123/resolve-crash-on-logging-post-requests

Resolve crash reports not logging post requests in Django

Author: Callum-McKay

python2/tests/middleware/test_django.py

@@ -14,13 +14,20 @@ class DjangoProviderTests(SimpleTestCase):
     def setUp(self):
         request_factory = RequestFactory()
         self.get_request = request_factory.get('/foo')
+        self.post_request = request_factory.post('/foo')
 
-    def test_map_request(self):
+    def test_get_map_request(self):
         provider = Provider()
         request_payload = provider._mapRequest(self.get_request)
         self.assertEqual(request_payload['url'], '/foo')
         self.assertEqual(request_payload['httpMethod'], 'GET')
 
+    def test_post_map_request(self):
+        provider = Provider()
+        request_payload = provider._mapRequest(self.post_request)
+        self.assertEqual(request_payload['url'], '/foo')
+        self.assertEqual(request_payload['httpMethod'], 'POST')
+
     def test_get_django_environment(self):
         provider = Provider()
         environment_payload = provider._get_django_environment()

python3/raygun4py/middleware/django.py

@@ -38,7 +38,7 @@ def _mapRequest(self, request):
             'queryString': dict((key, request.GET[key]) for key in request.GET),
             'form': dict((key, request.POST[key]) for key in request.POST),
             'headers': _headers,
-            'rawData': request.body if hasattr(request, 'body') else getattr(request, 'raw_post_data', {})
+            'rawData': request.data if hasattr(request, 'data') else getattr(request, 'raw_post_data', {})
         }
 
     def _get_django_environment(self):

python3/tests/middleware/test_django.py

@@ -14,13 +14,20 @@ class DjangoProviderTests(SimpleTestCase):
     def setUp(self):
         request_factory = RequestFactory()
         self.get_request = request_factory.get('/foo')
+        self.post_request = request_factory.post('/foo')
 
-    def test_map_request(self):
+    def test_get_map_request(self):
         provider = Provider()
         request_payload = provider._mapRequest(self.get_request)
         self.assertEqual(request_payload['url'], '/foo')
         self.assertEqual(request_payload['httpMethod'], 'GET')
 
+    def test_post_map_request(self):
+        provider = Provider()
+        request_payload = provider._mapRequest(self.post_request)
+        self.assertEqual(request_payload['url'], '/foo')
+        self.assertEqual(request_payload['httpMethod'], 'POST')
+
     def test_get_django_environment(self):
         provider = Provider()
         environment_payload = provider._get_django_environment()