diff options
| author | Marc Tamlyn | 2013-02-06 13:05:17 +0000 |
|---|---|---|
| committer | Marc Tamlyn | 2013-02-06 13:05:17 +0000 |
| commit | bd7977eed79bd8fc4d9e73b6d848b0f0cd5a72ec (patch) | |
| tree | 4dc46974f2e1b6e957e611d820329d82efe2eae9 /rest_framework/renderers.py | |
| parent | cc2ec2bbf0aee53d360a81cf338361feca1e8f80 (diff) | |
| download | django-rest-framework-bd7977eed79bd8fc4d9e73b6d848b0f0cd5a72ec.tar.bz2 | |
Purge naked excepts.
Most of these had obvious exceptions which would be thrown. Some I'm not
sure about but they should at least catch only Exception so as not to
ignore SystemExit and other inappropriate Error classes.
Diffstat (limited to 'rest_framework/renderers.py')
| -rw-r--r-- | rest_framework/renderers.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/rest_framework/renderers.py b/rest_framework/renderers.py index 74c7e2c9..960d4849 100644 --- a/rest_framework/renderers.py +++ b/rest_framework/renderers.py @@ -217,7 +217,7 @@ class TemplateHTMLRenderer(BaseRenderer): try: # Try to find an appropriate error template return self.resolve_template(template_names) - except: + except Exception: # Fall back to using eg '404 Not Found' return Template('%d %s' % (response.status_code, response.status_text.title())) @@ -303,7 +303,7 @@ class BrowsableAPIRenderer(BaseRenderer): try: if not view.has_permission(request, obj): return # Don't have permission - except: + except Exception: return # Don't have permission and exception explicitly raise return True |