Dont bother rendering status at all if they dont have access

Kronuz · Jan 15, 2012 · 5376454 · 5376454
1 parent 6fa9d41
commit 5376454
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/sentry/templates/sentry/status.html b/sentry/templates/sentry/status.html
@@ -8,11 +8,6 @@
 {% block heading %}{% trans "Status" %}{% endblock %}
 
 {% block content %}
-    {% if restricted %}
-    <div class="span16">
-        <div class="alert-message notice">Restricted to administrators.</div>
-    </div>
-    {% else %}
     <div class="span16">
         <ul class="tabs" data-tabs="tabs">
             <li class="active"><a href="#main">{% trans "Environment" %}</a></li>
@@ -203,5 +198,4 @@ <h2>{{ conf.site_config_title|default:conf.title }} configuration</h2>
             {% endfor %}
         </div>
     </div>
-    {% endif %}
 {% endblock %}
diff --git a/sentry/web/frontend/generic.py b/sentry/web/frontend/generic.py
@@ -38,9 +38,7 @@ def status(request):
     from sentry.processors import Processor
 
     if not request.user.is_staff:
-        return render_to_response('sentry/status.html', {
-            'restricted': True,
-        }, request)
+        return HttpResponseRedirect(reverse('sentry'))
 
     # Deal with the plugins
     site_configs = []