Use str to get the exception message

readthedocs · Apr 5, 2018 · 0761a06 · 0761a06
1 parent d557fee
commit 0761a06
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/readthedocs/restapi/views/model_views.py b/readthedocs/restapi/views/model_views.py
@@ -5,6 +5,7 @@
     absolute_import, division, print_function, unicode_literals)
 
 import logging
+from builtins import str
 
 from allauth.socialaccount.models import SocialAccount
 from django.shortcuts import get_object_or_404
@@ -170,10 +171,10 @@ def sync_versions(self, request, **kwargs):  # noqa: D205
                 added_versions.update(ret_set)
             deleted_versions = api_utils.delete_versions(project, data)
         except Exception as e:
-            log.exception('Sync Versions Error: %s', e.message)
+            log.exception('Sync Versions Error: %s', str(e))
             return Response(
                 {
-                    'error': e.message,
+                    'error': str(e),
                 },
                 status=status.HTTP_400_BAD_REQUEST,
             )

diff --git a/readthedocs/restapi/views/task_views.py b/readthedocs/restapi/views/task_views.py
@@ -2,6 +2,7 @@
 
 from __future__ import absolute_import
 import logging
+from builtins import str
 
 from django.core.urlresolvers import reverse
 from rest_framework import decorators, permissions
@@ -32,7 +33,7 @@ def get_status_data(task_name, state, data, error=None):
         'success': state in SUCCESS_STATES,
     }
     if error is not None and isinstance(error, Exception):
-        data['error'] = error.message
+        data['error'] = str(error)
     return data